update libraries and clang-format

.clang-format

@@ -17,9 +17,10 @@ NamespaceIndentation: All
 AlignAfterOpenBracket: AlwaysBreak
 AlignConsecutiveBitFields: Consecutive
 AlignConsecutiveMacros: AcrossEmptyLines
-AlignTrailingComments: true
 AllowShortCaseLabelsOnASingleLine: true
 AllowShortIfStatementsOnASingleLine: AllIfsAndElse
+AlignTrailingComments:
+  OverEmptyLines: 1
 
 # spacing
 BitFieldColonSpacing: Both

.github/workflows/clang-format.yaml

@@ -5,9 +5,10 @@ jobs:
     name: Formatting Check
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
+    - name: Checkout repo
+      uses: actions/checkout@v3
     - name: Run clang-format style check for C/C++/Protobuf programs.
-      uses: jidicula/clang-format-action@v4.6.2
+      uses: jidicula/clang-format-action@v4.11.0
       with:
-        clang-format-version: '13'
+        clang-format-version: '16'
         check-path: 'src'

platformio.ini

@@ -13,4 +13,4 @@ platform = atmelavr
 board = uno
 framework = arduino
 monitor_speed = 115200
-lib_deps = waspinator/AccelStepper@^1.61
+lib_deps = waspinator/AccelStepper@1.64

src/drivers/stepper_x27_driver.cpp

@@ -36,7 +36,7 @@ namespace stepper_x27_driver {
 
     stepper_x27_go_home();
     stepper_handle->disableOutputs();
-    operator delete(stepper_handle);
+    delete stepper_handle;
     stepper_handle = nullptr;
 
     return true;

src/drivers/stepper_x27_driver.h

@@ -10,17 +10,17 @@ namespace stepper_x27_driver {
   typedef struct {
     AccelStepper::MotorInterfaceType mode; // Stepper wire mode
 
-    uint8_t pin1; // Pin 1
-    uint8_t pin2; // Pin 2
-    uint8_t pin3; // Pin 3
-    uint8_t pin4; // Pin 4
-
-    uint32_t start_offset; // Stepper start offset
-    uint32_t full_range;   // Stepper full range (angle)
-    uint32_t short_range;  // Stepper main range (angle) (0 for full range)
-    uint32_t steps;        // Stepper total steps
-    uint32_t speed;        // Stepper max speed
-    uint32_t acceleration; // Stepper acceleration
+    uint8_t pin1;                          // Pin 1
+    uint8_t pin2;                          // Pin 2
+    uint8_t pin3;                          // Pin 3
+    uint8_t pin4;                          // Pin 4
+
+    uint32_t start_offset;                 // Stepper start offset
+    uint32_t full_range;                   // Stepper full range (angle)
+    uint32_t short_range;                  // Stepper main range (angle) (0 for full range)
+    uint32_t steps;                        // Stepper total steps
+    uint32_t speed;                        // Stepper max speed
+    uint32_t acceleration;                 // Stepper acceleration
   } stepper_x27_cfg;
 
   /**

src/main.cpp

@@ -8,6 +8,7 @@
 #include <saab_can/utilities/can_reader_util.h>
 #include <stdint.h>
 #include <stdio.h>
+#include <stdlib.h>
 
 #define MAIN_TAG          "MAIN"
 #define PACKET_TIMEOUT_MS 5000
@@ -43,7 +44,8 @@ void setup() {
     delete can_handle;
   }
 
-  sc_can_reader_subscription_t sub1 = {.id = "SUB_1", .frame_id = GAUGE_CAN_ID, .cb = handle_gauge_value_frame};
+  sc_can_reader_subscription_t sub1 = {.id = {}, .frame_id = GAUGE_CAN_ID, .cb = handle_gauge_value_frame};
+  snprintf(sub1.id, SC_CAN_RDR_SUB_ID_LENGTH, "SUB_1");
   sc_can_reader_subscribe(sub1);
 
   stepper_x27_driver::stepper_x27_cfg config = {};