Minor fixes ♻️

Author: AntonioBerna

.gitignore

@@ -16,31 +16,31 @@ bool button_b = true;
 bool button_c = true;
 
 void setup() {
-Serial.begin(9600);
-// Initialize Modulino I2C communication
-Modulino.begin();
-// Detect and connect to buttons module
-buttons.begin();
-// Turn on the LEDs above buttons A, B, and C
-buttons.setLeds(true, true, true);
+  Serial.begin(9600);
+  // Initialize Modulino I2C communication
+  Modulino.begin();
+  // Detect and connect to buttons module
+  buttons.begin();
+  // Turn on the LEDs above buttons A, B, and C
+  buttons.setLeds(true, true, true);
 }
 void loop() {
-// Check for new button events, returns true when button state changes
-if (buttons.update()) {
+  // Check for new button events, returns true when button state changes
+  if (buttons.update()) {
     // Check which button was pressed (0=A, 1=B, 2=C)
     // Also toggle the corresponding LED, for each of the three buttons
     if (buttons.isPressed(0)) {
-    Serial.println("Button A pressed!");
-    button_a = !button_a;
+      Serial.println("Button A pressed!");
+      button_a = !button_a;
     } else if (buttons.isPressed(1)) {
-    Serial.println("Button B pressed!");
-    button_b = !button_b;
+      Serial.println("Button B pressed!");
+      button_b = !button_b;
     } else if (buttons.isPressed(2)) {
-    Serial.println("Button C pressed!");
-    button_c = !button_c;
+      Serial.println("Button C pressed!");
+      button_c = !button_c;
     }
-
+    
     // Update the LEDs above buttons, depending on the variables value
     buttons.setLeds(button_a, button_b, button_c);
-}
+  }
 }

examples/Modulino_Buttons/Buttons_Basic/Buttons_Basic.ino

@@ -256,27 +256,12 @@ class ModulinoKnob : public Module {
     }
     return ret;
   }
-  bool update() {
+  int16_t get() {
     uint8_t buf[3];
     auto res = read(buf, 3);
     if (res == false) {
       return 0;
     }
-    get(buf);
-    return 1;
-  }
-  int16_t get(uint8_t * buf = nullptr) {
-    if (buf == nullptr) {
-      buf = (uint8_t*)malloc(3);
-      if (buf == nullptr) {
-        return 0;
-      }
-      auto res = read(buf, 3);
-      if (res == false) {
-        _pressed = false;
-        return 0;
-      }
-    }
     _pressed = (buf[2] != 0);
     int16_t ret = buf[0] | (buf[1] << 8);
     return ret;