ran make format

Author: tgd1975

include/config.h

@@ -10,7 +10,8 @@
  * Each hardware package provides its own config.cpp with the concrete
  * pin assignments for that board.
  */
-struct HardwareConfig {
+struct HardwareConfig
+{
     // LED pins
     uint8_t ledBluetooth; /**< Bluetooth status LED pin */
     uint8_t ledPower;     /**< Power indicator LED pin */

lib/PedalLogic/include/bank_manager.h

@@ -1,75 +1,77 @@
 #pragma once
-#include <memory>
-#include <array>
-#include "send.h"
 #include "i_led_controller.h"
+#include "send.h"
+#include <array>
+#include <memory>
 
 /**
  * @class BankManager
  * @brief Manages multiple banks of button configurations with LED feedback
- * 
+ *
  * This class handles the three configurable button banks, allowing different
  * actions to be assigned to each button in each bank. It also manages the
  * visual feedback through LED indicators to show the currently active bank.
  */
-class BankManager {
+class BankManager
+{
 public:
-    static constexpr uint8_t NUM_BANKS   = 3; /**< Number of available button banks */
+    static constexpr uint8_t NUM_BANKS = 3;   /**< Number of available button banks */
     static constexpr uint8_t NUM_BUTTONS = 4; /**< Number of buttons per bank */
 
     /**
      * @brief Constructs a BankManager with LED controllers
-     * 
+     *
      * @param led1 LED controller for bank 1 indicator
-     * @param led2 LED controller for bank 2 indicator  
+     * @param led2 LED controller for bank 2 indicator
      * @param led3 LED controller for bank 3 indicator
      */
     BankManager(ILEDController& led1, ILEDController& led2, ILEDController& led3);
 
     /**
      * @brief Adds an action to a specific bank and button
-     * 
+     *
      * @param bank Bank index (0-2)
      * @param button Button index (0-3)
      * @param action Unique pointer to the action to be executed
      */
     void addAction(uint8_t bank, uint8_t button, std::unique_ptr<Send> action);
-    
+
     /**
      * @brief Gets the action associated with a bank and button
-     * 
+     *
      * @param bank Bank index (0-2)
      * @param button Button index (0-3)
      * @return Pointer to the Send action, or nullptr if none assigned
      */
     Send* getAction(uint8_t bank, uint8_t button);
-    
+
     /**
      * @brief Switches to the next bank and updates LED indicators
-     * 
+     *
      * @return Index of the new current bank (0-2)
      */
     uint8_t switchBank();
-    
+
     /**
      * @brief Gets the currently active bank index
-     * 
+     *
      * @return Current bank index (0-2)
      */
     uint8_t getCurrentBank() const;
 
 private:
     /**
      * @brief Updates LED indicators to reflect current bank
-     * 
+     *
      * Turns on the LED corresponding to the current bank and
      * turns off the LEDs for other banks.
      */
     void updateLEDs();
-    
-    std::array<std::array<std::unique_ptr<Send>, NUM_BUTTONS>, NUM_BANKS> banks; /**< 2D array storing actions for all banks/buttons */
+
+    std::array<std::array<std::unique_ptr<Send>, NUM_BUTTONS>, NUM_BANKS>
+        banks;               /**< 2D array storing actions for all banks/buttons */
     uint8_t currentBank = 0; /**< Currently active bank index */
-    ILEDController& led1; /**< Reference to bank 1 LED controller */
-    ILEDController& led2; /**< Reference to bank 2 LED controller */
-    ILEDController& led3; /**< Reference to bank 3 LED controller */
+    ILEDController& led1;    /**< Reference to bank 1 LED controller */
+    ILEDController& led2;    /**< Reference to bank 2 LED controller */
+    ILEDController& led3;    /**< Reference to bank 3 LED controller */
 };

lib/PedalLogic/include/event_dispatcher.h

@@ -1,48 +1,49 @@
 #pragma once
-#include <functional>
 #include <array>
 #include <cstdint>
+#include <functional>
 
 /**
  * @class EventDispatcher
  * @brief Manages event callbacks for button presses
- * 
+ *
  * This class provides a simple event dispatching system that maps
  * button indices to callback functions. It supports 5 buttons:
  * - 0-3: Action buttons A-D
  * - 4: Bank selection button
  */
-class EventDispatcher {
+class EventDispatcher
+{
 public:
     /**
      * @brief Callback function type for button events
-     * 
+     *
      * Function signature for event handlers (no parameters, void return)
      */
     using EventCallback = std::function<void()>;
-    
+
     /**
      * @brief Registers a callback function for a button
-     * 
+     *
      * @param button Button index (0-4)
      * @param callback Function to call when button is pressed
      */
     void registerHandler(uint8_t button, EventCallback callback);
-    
+
     /**
      * @brief Executes the callback for the specified button
-     * 
+     *
      * @param button Button index (0-4) to dispatch
      */
     void dispatch(uint8_t button);
-    
+
     /**
      * @brief Clears all registered event handlers
-     * 
+     *
      * Resets all callbacks to empty functions.
      */
     void clearHandlers();
-    
+
 private:
     std::array<EventCallback, 5> handlers{}; /**< Array of callbacks for buttons 0-4 */
 };

lib/PedalLogic/include/i_ble_keyboard.h

@@ -3,64 +3,65 @@
 
 /**
  * @brief Type alias for media key reports
- * 
+ *
  * Represents a 2-byte media key report for BLE keyboard
  */
 using MediaKeyReport = uint8_t[2];
 
 // Key constants: define only when BleKeyboard.h has not been included,
 // so that firmware TUs that include both headers don't get redefinitions.
 #ifndef ESP32_BLE_KEYBOARD_H
-const uint8_t KEY_UP_ARROW    = 0xDA;    /**< USB HID key code for up arrow */
-const uint8_t KEY_DOWN_ARROW  = 0xD9;    /**< USB HID key code for down arrow */
-const uint8_t KEY_LEFT_ARROW  = 0xD8;    /**< USB HID key code for left arrow */
-const uint8_t KEY_RIGHT_ARROW = 0xD7;   /**< USB HID key code for right arrow */
+const uint8_t KEY_UP_ARROW = 0xDA;    /**< USB HID key code for up arrow */
+const uint8_t KEY_DOWN_ARROW = 0xD9;  /**< USB HID key code for down arrow */
+const uint8_t KEY_LEFT_ARROW = 0xD8;  /**< USB HID key code for left arrow */
+const uint8_t KEY_RIGHT_ARROW = 0xD7; /**< USB HID key code for right arrow */
 
 static constexpr uint8_t KEY_MEDIA_STOP[2] = {4, 0}; /**< Media key report for stop */
 #endif
 
 /**
  * @class IBleKeyboard
  * @brief Interface for Bluetooth LE keyboard functionality
- * 
+ *
  * Abstract interface that defines the contract for BLE keyboard operations.
  * Allows for dependency injection and mocking in tests.
  */
-class IBleKeyboard {
+class IBleKeyboard
+{
 public:
     virtual ~IBleKeyboard() = default;
-    
+
     /**
      * @brief Initializes the BLE keyboard
-     * 
+     *
      * Starts BLE advertising and sets up the keyboard service.
      */
     virtual void begin() = 0;
-    
+
     /**
      * @brief Checks if BLE keyboard is connected to a host
-     * 
+     *
      * @return true if connected, false otherwise
      */
     virtual bool isConnected() = 0;
-    
+
     /**
      * @brief Sends a single key press
-     * 
+     *
      * @param key USB HID key code to send
      */
     virtual void write(uint8_t key) = 0;
-    
+
     /**
      * @brief Sends a media key report
-     * 
+     *
      * @param key Media key report (2-byte array)
      */
     virtual void write(const MediaKeyReport key) = 0;
-    
+
     /**
      * @brief Sends a text string
-     * 
+     *
      * @param text Null-terminated string to send as keyboard input
      */
     virtual void print(const char* text) = 0;

lib/PedalLogic/include/i_button.h

@@ -8,7 +8,8 @@
  * of the concrete hardware ISR mechanism. Implementations live in the
  * platform-specific hardware packages.
  */
-class IButton {
+class IButton
+{
 public:
     virtual ~IButton() = default;
 

lib/PedalLogic/include/i_button_controller.h

@@ -3,24 +3,25 @@
 /**
  * @class IButtonController
  * @brief Interface for button input functionality
- * 
+ *
  * Abstract interface that defines the contract for button operations.
  * Allows for hardware abstraction and dependency injection.
  */
-class IButtonController {
+class IButtonController
+{
 public:
     virtual ~IButtonController() = default;
-    
+
     /**
      * @brief Initializes the button hardware
-     * 
+     *
      * Configures GPIO pins and any necessary hardware resources.
      */
     virtual void setup() = 0;
-    
+
     /**
      * @brief Reads the current button state
-     * 
+     *
      * @return true if button is pressed, false otherwise
      */
     virtual bool read() = 0;

lib/PedalLogic/include/i_led_controller.h

@@ -4,31 +4,32 @@
 /**
  * @class ILEDController
  * @brief Interface for LED control functionality
- * 
+ *
  * Abstract interface that defines the contract for LED operations.
  * Allows for hardware abstraction and dependency injection.
  */
-class ILEDController {
+class ILEDController
+{
 public:
     virtual ~ILEDController() = default;
-    
+
     /**
      * @brief Initializes the LED with optional initial state
-     * 
+     *
      * @param initialState Initial state (0 = off, non-zero = on)
      */
     virtual void setup(uint32_t initialState = 0) = 0;
-    
+
     /**
      * @brief Sets the LED state
-     * 
+     *
      * @param state true to turn on, false to turn off
      */
     virtual void setState(bool state) = 0;
-    
+
     /**
      * @brief Toggles the LED state
-     * 
+     *
      * If LED is on, turns it off. If off, turns it on.
      */
     virtual void toggle() = 0;

lib/PedalLogic/include/pedal_config.h

@@ -4,10 +4,10 @@
 
 /**
  * @brief Configures the button banks with default actions
- * 
+ *
  * Sets up the three button banks with predefined actions for the guitar pedal.
  * This function defines what each button does in each bank.
- * 
+ *
  * @param bankManager Reference to the BankManager instance
  * @param keyboard Pointer to the BLE keyboard interface
  */