DigitalOut run() refactor

Author: cotestatnt

README.md

@@ -49,12 +49,18 @@ if (button1.longClick(true)) {
 ```
 Use the callback function for edge transition
 ```C++
-void buttonPressed() {
+void buttonPressed(void* arg) {
+  DigitalIn* btn = (DigitalIn*) arg;
+  Serial.print("GPIO ");
+  Serial.print(btn->getPin());
   Serial.println("Button pressed");   
 }
 
-void buttonReleased() {
-  Serial.println("Button released");   
+void buttonReleased(void* arg) {
+  DigitalIn* btn = (DigitalIn*) arg;
+  Serial.print("GPIO ");
+  Serial.print(btn->getPin());
+  Serial.println(" - Button released");   
 }
 
 void setup() {

library.properties

@@ -1,5 +1,5 @@
-name=DigitalSignalsArduino
-version=0.0.5
+name=DigitalSignals Arduino
+version=0.1.0
 author=Tolentino Cotesta <cotestatnt@yahoo.com>
 maintainer=Tolentino Cotesta <cotestatnt@yahoo.com>
 sentence=Arduino Digital Signal library

src/DigitalIn.cpp

@@ -8,7 +8,7 @@ DigitalIn::DigitalIn(IOPinName _pin, IOPinMode _mode, uint32_t _time) {
   m_shortPressCnt = LONG_CLICK;
 
   m_bounceTime = _time;
-  m_deadTime = 2*m_bounceTime;
+  m_deadTime = 2 * m_bounceTime;
   m_pin = _pin;
   m_mode = _mode;
   pinMode(m_pin, _mode);
@@ -27,7 +27,7 @@ void DigitalIn::mode(IOPinMode pull) {
   pinMode(m_pin, pull);
 }
 
-void DigitalIn::setLongClickTime(uint32_t newTime){
+void DigitalIn::setLongClickTime(uint32_t newTime) {
   m_longClickTime = newTime;
 }
 

src/DigitalIn.h

@@ -8,8 +8,8 @@ typedef int IOPinMode;
 
 typedef void(*function_cb)(void*);
 
-
-/** A digital input, used for reading the state of a pin
+/*
+  A digital input, used for reading the state of a pin
 */
 class DigitalIn {
 
@@ -38,28 +38,32 @@ class DigitalIn {
 
     DigitalIn(IOPinName _pin, IOPinMode _mode = INPUT_PULLUP, uint32_t _time = 70);
 
-    void pressCallback(function_cb fn)  {onPressed(fn);}
-    void releaseCallback(function_cb fn) {onReleased(fn);}
+    void pressCallback(function_cb fn)  {
+      onPressed(fn);
+    }
+    void releaseCallback(function_cb fn) {
+      onReleased(fn);
+    }
 
     void onPressed(function_cb fn);
     void onReleased(function_cb fn);
 
     /** Set the input pin mode
-     *
-     *  @param pull INPUT, INPUT_PULLUP, OUTPUT
-     */
+
+        @param pull INPUT, INPUT_PULLUP, OUTPUT
+    */
     void mode(IOPinMode pull);
 
     /** Set the long click time different from default
-     *
-     *  @param newTime
-     */
+
+        @param newTime
+    */
     void setLongClickTime(uint32_t newTime);
 
     /** Set the double click time different from default
-     *
-     *  @param newTime
-     */
+
+        @param newTime
+    */
     void setDoubleClickTime(uint32_t newTime);
 
     /** Get current status of output */
@@ -75,24 +79,29 @@ class DigitalIn {
     bool doubleClick();
 
     /** Check for long click (return true on release confgurable)
-     *
-     *  @param waitRelease true, false
-     */
+
+        @param waitRelease true, false
+    */
     bool longClick(bool waitRelease = false);
 
     /** An operator shorthand for read status
-     * \sa DigitalIn::read()
-     * @code
-     *      DigitalIn  button(BUTTON1);
-     *      DigitalOut led(LED1);
-     *      led = button;   // Equivalent to led.write(button.read())
-     * @endcode
-     */
+       \sa DigitalIn::read()
+       @code
+            DigitalIn  button(BUTTON1);
+            DigitalOut led(LED1);
+            led = button;   // Equivalent to led.write(button.read())
+       @endcode
+    */
     operator bool() {
       update();
       return isActive() && m_pressedDuration > m_bounceTime;
     }
 
+    operator int() {
+      update();
+      return isActive() && m_pressedDuration > m_bounceTime;
+    }
+
     /** Get assigned GPIO number */
     int getPin() {
       return m_pin;