Merge pull request #4 from bracci/tidy

Tidy

Colors.h

@@ -15,11 +15,11 @@
 #ifndef COLORS_H
 #define COLORS_H
 
-struct color_t {
+typedef struct color_s {
   byte red;
   byte green;
   byte blue;
-};
+}color_t;
 
 const color_t defaultColors[] PROGMEM = { 
   {250, 255, 200}, 

DCF77Helper.cpp

@@ -21,7 +21,7 @@
  */
 #include "DCF77Helper.h"
 
-//#define DEBUG
+#define DEBUG
 #include "Debug.h"
 
 /**
@@ -39,7 +39,7 @@ DCF77Helper::DCF77Helper() {
 /**
  * Einen neuen Sample hinzufuegen.
  */
-void DCF77Helper::addSample(MyDCF77 dcf77, MyRTC rtc) {
+void DCF77Helper::addSample(MyDCF77* dcf77, MyRTC* rtc) {
   DEBUG_PRINT(F("Adding sample from dcf77: "));
   _zeitstempelDcf77[_cursor]->setFrom(dcf77);
   DEBUG_PRINTLN(_zeitstempelDcf77[_cursor]->asString());

DCF77Helper.h

@@ -32,7 +32,7 @@ class DCF77Helper {
   public:
     DCF77Helper();
 
-    void addSample(MyDCF77 dcf77, MyRTC rtc);
+    void addSample(MyDCF77* dcf77, MyRTC* rtc);
     boolean samplesOk();
 
   private:

Effects.cpp

@@ -21,15 +21,15 @@ void Effects::showTickerString(const char* str2disp, byte tickerSpeed) {
   word matrix [16];
 
   byte strLength = strlen(str2disp);
-  int bufLen;
+  unsigned int bufLen;
   char actChar;
   char lastChar = 'W';
   byte offsetV = 2;
   bool finish = false;
   unsigned int i = 0;
 
   while (!finish) {
-  renderer.clearScreenBuffer(matrix);
+    renderer.clearScreenBuffer(matrix);
     unsigned int shift = 0; // Schiebekorrektur aufgrund variierender Buchstabenbreite
     for (byte k = 0; k < strLength; k++) {
       actChar = str2disp[k];
@@ -67,13 +67,13 @@ void Effects::showIntro() {
   renderer.clearScreenBuffer(matrix);
   for (int j = 0; j < 11; j++) {
     for (byte i = 0; i < 10; i++) {
-      matrix[i] |= 0b1 << 15 - j;
+      matrix[i] |= 0b1 << (15 - j);
     }
     writeToBuffer(matrix, 5);
   }
   for (int j = 0; j < 11; j++) {
     for (int i = 0; i < 10; i++) {
-      matrix[i] ^= 0b1 << 5 + j;
+      matrix[i] ^= 0b1 << (5 + j);
     }
     writeToBuffer(matrix, 5);
   }
@@ -131,14 +131,14 @@ void Effects::showFireWork(byte posX, eColors color) {
   for (byte i = 8; i <= 10; i++) {
     renderer.clearScreenBuffer(matrix);
     for (byte j = 0; j < 10; j++) {
-      matrix[j] |= (pgm_read_word_near(&(effectMasks[i][j])) << 10 - posX) & 0b1111111111100000;
+      matrix[j] |= (pgm_read_word_near(&(effectMasks[i][j])) << (10 - posX)) & 0b1111111111100000;
     }
     writeToBuffer(matrix, 3 + round(10 * (i - 8) / 3), color);
   }
   for (byte i = 0; i <= 10; i++) {
     renderer.clearScreenBuffer(matrix);
     for (byte j = 0; j < 10 - i; j++) {
-      matrix[j + i] |= (pgm_read_word_near(&(effectMasks[12 + i % 2][j])) << 10 - posX) & 0b1111111111100000;
+      matrix[j + i] |= (pgm_read_word_near(&(effectMasks[12 + i % 2][j])) << (10 - posX)) & 0b1111111111100000;
     }
     writeToBuffer(matrix, 20, color);
   }
@@ -183,7 +183,7 @@ void Effects::showBitmap(byte bitmapIdx, byte duration, eColors color) {
   renderer.clearScreenBuffer(matrix);
   for (byte i = 0; i < 10; i++) {
     for (byte j = 0; j < 11; j++) {
-      matrix[i] |= ((pgm_read_word_near(&(bitmaps[bitmapIdx - BITMAP_MIN][j])) >> i) & 0x0001) << 15 - j;
+      matrix[i] |= ((pgm_read_word_near(&(bitmaps[bitmapIdx - BITMAP_MIN][j])) >> i) & 0x0001) << (15 - j);
     }
   }
   writeToBuffer(matrix, 15 * duration, color);
@@ -209,6 +209,8 @@ void Effects::showAnimatedBitmap(byte animatedBitmap, byte duration, eColors col
       showBitmap(BITMAP_SMILEY_WINK, duration, color);
       showBitmap(BITMAP_SMILEY, duration, color);
       break;
+    default:
+      ;
   }
 }
 

Event.cpp

@@ -28,7 +28,7 @@ Event::Event(byte month,
 void Event::show() {
   DEBUG_PRINT(F("Ticker String: "));
   DEBUG_PRINTLN(_txt);
-  if (_txt != "")
+  if (strlen(_txt) != 0)
     Effects::showTickerString(_txt, TICKER_SPEED);
   if (_effect < Effects::BITMAP_MIN) {
     switch (_effect)

LedDriverDefault.cpp

@@ -1,21 +1,21 @@
 /**
- * LedDriverDefault
- * Implementierung auf der Basis 74HC595 und UDN2981A.
- *
- * @mc       Arduino/RBBB
- * @autor    Christian Aschoff / caschoff _AT_ mac _DOT_ com
- * @version  1.4
- * @created  18.1.2013
- * @updated  18.1.2015
- *
- * Versionshistorie:
- * V 1.0:  - Erstellt.
- * V 1.1:  - printSignature() eingefuehrt.
- *         - Benennung verbessert.
- * V 1.2:  - Anpassung Helligkeit 0-100%
- * V 1.3:  - Getter fuer Helligkeit nachgezogen.
- * V 1.4:  - Unterstuetzung fuer die alte Arduino-IDE (bis 1.0.6) entfernt.
- */
+   LedDriverDefault
+   Implementierung auf der Basis 74HC595 und UDN2981A.
+
+   @mc       Arduino/RBBB
+   @autor    Christian Aschoff / caschoff _AT_ mac _DOT_ com
+   @version  1.4
+   @created  18.1.2013
+   @updated  18.1.2015
+
+   Versionshistorie:
+   V 1.0:  - Erstellt.
+   V 1.1:  - printSignature() eingefuehrt.
+           - Benennung verbessert.
+   V 1.2:  - Anpassung Helligkeit 0-100%
+   V 1.3:  - Getter fuer Helligkeit nachgezogen.
+   V 1.4:  - Unterstuetzung fuer die alte Arduino-IDE (bis 1.0.6) entfernt.
+*/
 #include "LedDriverDefault.h"
 #include "Configuration.h"
 
@@ -30,14 +30,14 @@
 #endif
 
 /**
- * Initialisierung.
- *
- * @param data Pin, an dem die Data-Line haengt.
- * @param clock Pin, an dem die Clock-Line haengt.
- * @param latch Pin, an dem die Latch-Line haengt.
- * @param outputEnable Pin, an dem OutputEnable haengt.
- * @param linesToWrite Wieviel Zeilen muessen aus dem Bildspeicher uebernommen werden?
- */
+   Initialisierung.
+
+   @param data Pin, an dem die Data-Line haengt.
+   @param clock Pin, an dem die Clock-Line haengt.
+   @param latch Pin, an dem die Latch-Line haengt.
+   @param outputEnable Pin, an dem OutputEnable haengt.
+   @param linesToWrite Wieviel Zeilen muessen aus dem Bildspeicher uebernommen werden?
+*/
 LedDriverDefault::LedDriverDefault(byte data, byte clock, byte latch, byte outputEnable, byte linesToWrite) {
   _shiftRegister = new ShiftRegister(data, clock, latch);
   _outputEnablePin = outputEnable;
@@ -49,10 +49,10 @@ LedDriverDefault::LedDriverDefault(byte data, byte clock, byte latch, byte outpu
 }
 
 /**
- * init() wird im Hauptprogramm in init() aufgerufen.
- * Hier sollten die LED-Treiber in eine definierten
- * Ausgangszustand gebracht werden.
- */
+   init() wird im Hauptprogramm in init() aufgerufen.
+   Hier sollten die LED-Treiber in eine definierten
+   Ausgangszustand gebracht werden.
+*/
 void LedDriverDefault::init() {
 }
 
@@ -61,11 +61,11 @@ void LedDriverDefault::printSignature() {
 }
 
 /**
- * Den Bildschirm-Puffer auf die LED-Matrix schreiben.
- *
- * @param onChange: TRUE, wenn es Aenderungen in dem Bildschirm-Puffer gab,
- *                  FALSE, wenn es ein Refresh-Aufruf war.
- */
+   Den Bildschirm-Puffer auf die LED-Matrix schreiben.
+
+   @param onChange: TRUE, wenn es Aenderungen in dem Bildschirm-Puffer gab,
+                    FALSE, wenn es ein Refresh-Aufruf war.
+*/
 void LedDriverDefault::writeScreenBufferToMatrix(word matrix[16], boolean onChange, eColors a_color) {
 
   if (!_transitionCompleted && (_transitionCounter > 0)) {
@@ -77,26 +77,26 @@ void LedDriverDefault::writeScreenBufferToMatrix(word matrix[16], boolean onChan
   }
 
   /*************
-  * MATRIX
+    MATRIX
   **************/
 
   if (mode != STD_MODE_NORMAL) {
     _transitionCompleted = true;
   }
 
   /*************
-   * MATRIX
+     MATRIX
   **************/
 
   if (onChange) {
     if ((helperSeconds == 0) && (mode == STD_MODE_NORMAL) && _transitionCompleted && !evtActive) {
       switch (settings.getTransitionMode()) {
         case Settings::TRANSITION_MODE_FADE:
           for (byte i = 0; i < _linesToWrite; i++) {
-            _matrixOld[i] = _matrixNew[i]; //Abbild der aktuellen Matrix in Vorversion r�berkopieren
-            _matrixNew[DISPLAY_SHIFT i] = matrix[i]; //Abbild der neuen Matrix in aktuelle Matrix einkopieren, um sie beim n�chsten �berblenden zu haben
+            _matrixOld[i] = _matrixNew[i]; //Abbild der aktuellen Matrix in Vorversion rüberkopieren
+            _matrixNew[DISPLAY_SHIFT i] = matrix[i]; //Abbild der neuen Matrix in aktuelle Matrix einkopieren, um sie beim nächsten überblenden zu haben
           }
-          _transitionCounter = FADINGCOUNTERLOAD; // �berblendvariable mit Startwert laden
+          _transitionCounter = FADINGCOUNTERLOAD; // Überblendvariable mit Startwert laden
           _transitionCompleted = false;
           break;
         case Settings::TRANSITION_MODE_SLIDE:
@@ -110,13 +110,13 @@ void LedDriverDefault::writeScreenBufferToMatrix(word matrix[16], boolean onChan
           }
           break;
         default:
-          break;
+          ;
       }
     }
     if (_transitionCompleted) {
       for (byte i = 0; i < _linesToWrite; i++) {
         _matrixOld[i] = 0; //Aktuelles Abbild ist nicht verwendbar, weil eventuell keine Uhrzeit
-        _matrixNew[DISPLAY_SHIFT i] = matrix[i]; //Abbild der neuen Matrix in aktuelle Matrix einkopieren, um sie beim n�chsten �berblenden zu haben
+        _matrixNew[DISPLAY_SHIFT i] = matrix[i]; //Abbild der neuen Matrix in aktuelle Matrix einkopieren, um sie beim nächsten überblenden zu haben
       }
     }
   }
@@ -127,6 +127,8 @@ void LedDriverDefault::writeScreenBufferToMatrix(word matrix[16], boolean onChan
         _transitionCounter = SLIDINGCOUNTERLOAD / 50;
         _transitionCompleted = Transitions::nextSlideStep(_matrixNew, matrix);
         break;
+      default:
+        ;
     }
   }
 
@@ -153,7 +155,7 @@ void LedDriverDefault::writeScreenBufferToMatrix(word matrix[16], boolean onChan
 #endif
 
       if (_displayOn == true) {
-        if ((Settings::TRANSITION_MODE_FADE == settings.getTransitionMode()) && !_transitionCompleted) { // �ber OE einschalten und nach PWM-Anteil wieder ausschalten, wenn das Display aktiv ist
+        if ((Settings::TRANSITION_MODE_FADE == settings.getTransitionMode()) && !_transitionCompleted) { // Über OE einschalten und nach PWM-Anteil wieder ausschalten, wenn das Display aktiv ist
           // Alter Zeileninhalt
           // Zeile schreiben...
           _shiftRegister->prepareShiftregisterWrite();
@@ -192,50 +194,50 @@ void LedDriverDefault::writeScreenBufferToMatrix(word matrix[16], boolean onChan
 }
 
 /**
- * Die Helligkeit des Displays anpassen.
- *
- * @param brightnessInPercent Die Helligkeit.
- */
+   Die Helligkeit des Displays anpassen.
+
+   @param brightnessInPercent Die Helligkeit.
+*/
 void LedDriverDefault::setBrightness(byte brightnessInPercent) {
   _brightnessInPercent = brightnessInPercent;
 }
 
 /**
- * Die aktuelle Helligkeit bekommen.
- */
+   Die aktuelle Helligkeit bekommen.
+*/
 byte LedDriverDefault::getBrightness() {
   return _brightnessInPercent;
 }
 
 /**
- * Anpassung der Groesse des Bildspeichers.
- *
- * @param linesToWrite Wieviel Zeilen aus dem Bildspeicher sollen
- *                     geschrieben werden?
- */
+   Anpassung der Groesse des Bildspeichers.
+
+   @param linesToWrite Wieviel Zeilen aus dem Bildspeicher sollen
+                       geschrieben werden?
+*/
 void LedDriverDefault::setLinesToWrite(byte linesToWrite) {
   _linesToWrite = linesToWrite;
 }
 
 /**
- * Das Display ausschalten.
- */
+   Das Display ausschalten.
+*/
 void LedDriverDefault::shutDown() {
   digitalWrite(_outputEnablePin, HIGH);
   _displayOn = false;
 }
 
 /**
- * Das Display einschalten.
- */
+   Das Display einschalten.
+*/
 void LedDriverDefault::wakeUp() {
   digitalWrite(_outputEnablePin, LOW);
   _displayOn = true;
 }
 
 /**
- * Den Dateninhalt des LED-Treibers loeschen.
- */
+   Den Dateninhalt des LED-Treibers loeschen.
+*/
 void LedDriverDefault::clearData() {
   _shiftRegister->prepareShiftregisterWrite();
   _shiftRegister->shiftOut(65535);