Add and apply style configuration for code formatter Clang-Format.

.clang-format

@@ -0,0 +1,5 @@
+# Style configuration for code formatter Clang-Format (part of LLVM)
+BasedOnStyle: Microsoft
+ReflowComments: false
+ColumnLimit: 0
+

.github/workflows/clang-format-check.yml

@@ -0,0 +1,13 @@
+name: Clang-Format Check
+on: [push, pull_request]
+jobs:
+  formatting-check:
+    name: check style formatting
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - name: Run clang-format style check.
+      uses: jidicula/clang-format-action@v4.11.0
+      with:
+        fallback-style: 'none'
+

time-tracker-pio/include/main.hpp

@@ -3,8 +3,8 @@
 
 namespace main
 {
-void setup(const char * const programIdentificationString);
+void setup(const char *const programIdentificationString);
 void loop();
-}
+} // namespace main
 
 #endif /* APPLICATION_MAIN_HPP_ */

time-tracker-pio/include/pitches.hpp

@@ -93,4 +93,4 @@ constexpr std::uint16_t C8 = 4186;
 constexpr std::uint16_t CS8 = 4435;
 constexpr std::uint16_t D8 = 4699;
 constexpr std::uint16_t DS8 = 4978;
-}
+} // namespace note

time-tracker-pio/src/display.cpp

@@ -1,103 +1,108 @@
 
-#include <Wire.h>
-#include <Arduino.h>
 #include <Adafruit_SSD1306.h>
+#include <Arduino.h>
+#include <Wire.h>
 
 #define SCREEN_WIDTH 128 // OLED display width, in pixels
 #define SCREEN_HEIGHT 64 // OLED display height, in pixels
 
 // Declaration for an SSD1306 display connected to I2C (SDA, SCL pins)
-#define OLED_RESET     -1 // Reset pin # (or -1 if sharing Arduino reset pin)
+#define OLED_RESET -1 // Reset pin # (or -1 if sharing Arduino reset pin)
 Adafruit_SSD1306 display(SCREEN_WIDTH, SCREEN_HEIGHT, &Wire, OLED_RESET);
 
-#define NUMFLAKES     10 // Number of snowflakes in the animation example
+#define NUMFLAKES 10 // Number of snowflakes in the animation example
 
-#define LOGO_HEIGHT   16
-#define LOGO_WIDTH    16
+#define LOGO_HEIGHT 16
+#define LOGO_WIDTH 16
 static constexpr unsigned char PROGMEM logo_bmp[] =
-{ B00000000, B11000000,
-  B00000001, B11000000,
-  B00000001, B11000000,
-  B00000011, B11100000,
-  B11110011, B11100000,
-  B11111110, B11111000,
-  B01111110, B11111111,
-  B00110011, B10011111,
-  B00011111, B11111100,
-  B00001101, B01110000,
-  B00011011, B10100000,
-  B00111111, B11100000,
-  B00111111, B11110000,
-  B01111100, B11110000,
-  B01110000, B01110000,
-  B00000000, B00110000 };
-
-
-#define XPOS   0 // Indexes into the 'icons' array in function below
-#define YPOS   1
+    {B00000000, B11000000,
+     B00000001, B11000000,
+     B00000001, B11000000,
+     B00000011, B11100000,
+     B11110011, B11100000,
+     B11111110, B11111000,
+     B01111110, B11111111,
+     B00110011, B10011111,
+     B00011111, B11111100,
+     B00001101, B01110000,
+     B00011011, B10100000,
+     B00111111, B11100000,
+     B00111111, B11110000,
+     B01111100, B11110000,
+     B01110000, B01110000,
+     B00000000, B00110000};
+
+#define XPOS 0 // Indexes into the 'icons' array in function below
+#define YPOS 1
 #define DELTAY 2
 
 static int8_t icons[NUMFLAKES][3];
 
-void testanimate() {
-  int8_t f;
+void testanimate()
+{
+    int8_t f;
 
     display.clearDisplay(); // Clear the display buffer
 
     // Draw each snowflake:
-    for(f=0; f< NUMFLAKES; f++) {
-      display.drawBitmap(icons[f][XPOS], icons[f][YPOS], logo_bmp, LOGO_WIDTH, LOGO_HEIGHT, WHITE);
+    for (f = 0; f < NUMFLAKES; f++)
+    {
+        display.drawBitmap(icons[f][XPOS], icons[f][YPOS], logo_bmp, LOGO_WIDTH, LOGO_HEIGHT, WHITE);
     }
 
     display.display(); // Show the display buffer on the screen
 
     // Then update coordinates of each flake...
-    for(f=0; f< NUMFLAKES; f++) {
-      icons[f][YPOS] += icons[f][DELTAY];
-      // If snowflake is off the bottom of the screen...
-      if (icons[f][YPOS] >= display.height()) {
-        // Reinitialize to a random position, just off the top
-        icons[f][XPOS]   = random(1 - LOGO_WIDTH, display.width());
-        icons[f][YPOS]   = -LOGO_HEIGHT;
-        icons[f][DELTAY] = random(1, 6);
-      }
+    for (f = 0; f < NUMFLAKES; f++)
+    {
+        icons[f][YPOS] += icons[f][DELTAY];
+        // If snowflake is off the bottom of the screen...
+        if (icons[f][YPOS] >= display.height())
+        {
+            // Reinitialize to a random position, just off the top
+            icons[f][XPOS] = random(1 - LOGO_WIDTH, display.width());
+            icons[f][YPOS] = -LOGO_HEIGHT;
+            icons[f][DELTAY] = random(1, 6);
+        }
     }
 }
 
+void setup_display()
+{
 
-void setup_display() {
-
-  // SSD1306_SWITCHCAPVCC = generate display voltage from 3.3V internally
-  if(!display.begin(SSD1306_SWITCHCAPVCC, 0x3C)) {
-    Serial.println(F("SSD1306 allocation failed"));
-    for(;;); // Don't proceed, loop forever
-  }
+    // SSD1306_SWITCHCAPVCC = generate display voltage from 3.3V internally
+    if (!display.begin(SSD1306_SWITCHCAPVCC, 0x3C))
+    {
+        Serial.println(F("SSD1306 allocation failed"));
+        for (;;)
+            ; // Don't proceed, loop forever
+    }
 
-  // Show initial display buffer contents on the screen --
-  // the library initializes this with an Adafruit splash screen.
-  display.display();
-  delay(2000); // Pause for 2 seconds
+    // Show initial display buffer contents on the screen --
+    // the library initializes this with an Adafruit splash screen.
+    display.display();
+    delay(2000); // Pause for 2 seconds
 
-  // Clear the buffer
-  //display.clearDisplay();
+    // Clear the buffer
+    //display.clearDisplay();
 
-  // Show the display buffer on the screen. You MUST call display() after
-  // drawing commands to make them visible on screen!
-  display.display();
+    // Show the display buffer on the screen. You MUST call display() after
+    // drawing commands to make them visible on screen!
+    display.display();
 
     int8_t f;
 
-  // Initialize 'snowflake' positions
-  for(f=0; f< NUMFLAKES; f++) {
-    icons[f][XPOS]   = random(1 - LOGO_WIDTH, display.width());
-    icons[f][YPOS]   = -LOGO_HEIGHT;
-    icons[f][DELTAY] = random(1, 6);
-    Serial.print(F("x: "));
-    Serial.print(icons[f][XPOS], DEC);
-    Serial.print(F(" y: "));
-    Serial.print(icons[f][YPOS], DEC);
-    Serial.print(F(" dy: "));
-    Serial.println(icons[f][DELTAY], DEC);
-  }
-
+    // Initialize 'snowflake' positions
+    for (f = 0; f < NUMFLAKES; f++)
+    {
+        icons[f][XPOS] = random(1 - LOGO_WIDTH, display.width());
+        icons[f][YPOS] = -LOGO_HEIGHT;
+        icons[f][DELTAY] = random(1, 6);
+        Serial.print(F("x: "));
+        Serial.print(icons[f][XPOS], DEC);
+        Serial.print(F(" y: "));
+        Serial.print(icons[f][YPOS], DEC);
+        Serial.print(F(" dy: "));
+        Serial.println(icons[f][DELTAY], DEC);
+    }
 }

time-tracker-pio/src/main.cpp

@@ -3,101 +3,102 @@
 #include "pitches.hpp"
 //#include <ESP32Tone.h>
 #include <Arduino.h>
-#include <cstdint>
 #include <cmath>
+#include <cstdint>
 
 namespace OutputShiftRegister
 {
-  constexpr int pin_data = 14;
-  constexpr int pin_latch = 13;
-  constexpr int pin_clock = 5;
+constexpr int pin_data = 14;
+constexpr int pin_latch = 13;
+constexpr int pin_clock = 5;
 
-  void setup()
-  {
+void setup()
+{
     pinMode(pin_data, OUTPUT);
     pinMode(pin_latch, OUTPUT);
     pinMode(pin_clock, OUTPUT);
-  }
+}
 
-  void setRegister(const uint8_t data)
-  {
+void setRegister(const uint8_t data)
+{
     digitalWrite(pin_latch, LOW);
     shiftOut(pin_data, pin_clock, LSBFIRST, data);
     digitalWrite(pin_latch, HIGH);
-  }
 }
+} // namespace OutputShiftRegister
 
 namespace InputShiftRegister
 {
-  constexpr int pin_data = 34;
-  constexpr int pin_latch = 12;
-  constexpr int pin_clock = 23;
+constexpr int pin_data = 34;
+constexpr int pin_latch = 12;
+constexpr int pin_clock = 23;
 
-  void setup()
-  {
+void setup()
+{
     pinMode(pin_data, INPUT);
     pinMode(pin_latch, OUTPUT);
     pinMode(pin_clock, OUTPUT);
-  }
+}
 
-  typedef decltype(shiftIn(0,0,LSBFIRST)) RegisterType;
+typedef decltype(shiftIn(0, 0, LSBFIRST)) RegisterType;
 
-  RegisterType getRegister()
-  {
+RegisterType getRegister()
+{
     digitalWrite(pin_latch, LOW);
     delayMicroseconds(1); // at least 200ns
     digitalWrite(pin_latch, HIGH);
     return shiftIn(pin_data, pin_clock, LSBFIRST);
-  }
 }
+} // namespace InputShiftRegister
 
 /**
  *
  * @return 1-8 or 0 in case of no event
  */
 static std::uint8_t getEvent()
 {
-  static InputShiftRegister::RegisterType oldValue = 0;
-  const InputShiftRegister::RegisterType newValue = InputShiftRegister::getRegister();
-  std::uint8_t result = 0;
-  if(newValue != 0 && oldValue == 0)
-  {
-    result = std::log2(newValue<<1);
-  }
-  oldValue = newValue;
-  return result;
+    static InputShiftRegister::RegisterType oldValue = 0;
+    const InputShiftRegister::RegisterType newValue = InputShiftRegister::getRegister();
+    std::uint8_t result = 0;
+    if (newValue != 0 && oldValue == 0)
+    {
+        result = std::log2(newValue << 1);
+    }
+    oldValue = newValue;
+    return result;
 }
 
 constexpr int pin_buzzer = 4;
 
-namespace main{
-void setup(char const * programIdentificationString)
+namespace main
 {
-  // put your setup code here, to run once:
-  pinMode(pin_buzzer, OUTPUT);
-  OutputShiftRegister::setup();
-  InputShiftRegister::setup();
-  setup_display();
-  Serial.begin(115200);
-  delay(100);
-  Serial.flush();
-  delay(100);
-  Serial.printf("\n begin program '%s'\n", programIdentificationString);
+void setup(char const *programIdentificationString)
+{
+    // put your setup code here, to run once:
+    pinMode(pin_buzzer, OUTPUT);
+    OutputShiftRegister::setup();
+    InputShiftRegister::setup();
+    setup_display();
+    Serial.begin(115200);
+    delay(100);
+    Serial.flush();
+    delay(100);
+    Serial.printf("\n begin program '%s'\n", programIdentificationString);
 }
 void loop()
 {
-  const auto event = getEvent();
-  if (event)
-  {
-    constexpr std::uint16_t notes[] = { note::C4, note::G3, note::A3, note::B3, note::D1, note::E1, note::F1, note::G1 };
-    OutputShiftRegister::setRegister(1 << (event - 1));
-    tone(pin_buzzer, notes[event-1], 250);
-  }
-  else
-  {
-    OutputShiftRegister::setRegister(0);
-  }
-  delay(100);
-  testanimate(); // Animate bitmaps
-}
+    const auto event = getEvent();
+    if (event)
+    {
+        constexpr std::uint16_t notes[] = {note::C4, note::G3, note::A3, note::B3, note::D1, note::E1, note::F1, note::G1};
+        OutputShiftRegister::setRegister(1 << (event - 1));
+        tone(pin_buzzer, notes[event - 1], 250);
+    }
+    else
+    {
+        OutputShiftRegister::setRegister(0);
+    }
+    delay(100);
+    testanimate(); // Animate bitmaps
 }
+} // namespace main

time-tracker-pio/src/time-tracker-src.ino

@@ -4,10 +4,10 @@ const auto programIdentificationString = __FILE__ " compiled at " __DATE__ " " _
 
 void setup()
 {
-  main::setup(programIdentificationString);
+    main::setup(programIdentificationString);
 }
 
 void loop()
 {
-  main::loop();
+    main::loop();
 }