Invented void* detachInterruptArg() that returns the argument given in attachInterruptArg().

This is sufficient for resource management - the function has static duration anyway.

Author: dok-net

cores/esp32/esp32-hal-gpio.c

@@ -75,7 +75,7 @@ typedef struct {
     voidFuncPtr fn;
     void* arg;
 } InterruptHandle_t;
-static InterruptHandle_t __pinInterruptHandlers[GPIO_PIN_COUNT] = {0,};
+static InterruptHandle_t __pinInterruptHandlers[GPIO_PIN_COUNT] = { {0,0}, };
 
 #include "driver/rtc_io.h"
 
@@ -275,15 +275,16 @@ extern void __detachInterrupt(uint8_t pin)
     esp_intr_enable(gpio_intr_handle);
 }
 
-extern InterruptHandle_t* __getInterruptHandler(uint8_t pin) {
-	return (pin < GPIO_PIN_COUNT) ? &__pinInterruptHandlers[pin] : NULL;
+extern void* __detachInterruptArg(uint8_t pin) {
+	void* arg = (pin < GPIO_PIN_COUNT) ? __pinInterruptHandlers[pin].arg : NULL;
+	__detachInterrupt(pin);
+	return arg;
 }
 
-
 extern void pinMode(uint8_t pin, uint8_t mode) __attribute__ ((weak, alias("__pinMode")));
 extern void digitalWrite(uint8_t pin, uint8_t val) __attribute__ ((weak, alias("__digitalWrite")));
 extern int digitalRead(uint8_t pin) __attribute__ ((weak, alias("__digitalRead")));
 extern void attachInterrupt(uint8_t pin, voidFuncPtr handler, int mode) __attribute__ ((weak, alias("__attachInterrupt")));
+extern void detachInterrupt(uint8_t pin) __attribute__((weak, alias("__detachInterrupt")));
 extern void attachInterruptArg(uint8_t pin, voidFuncPtrArg handler, void * arg, int mode) __attribute__ ((weak, alias("__attachInterruptArg")));
-extern void detachInterrupt(uint8_t pin) __attribute__ ((weak, alias("__detachInterrupt")));
-
+extern void* detachInterruptArg(uint8_t pin) __attribute__((weak, alias("__detachInterruptArg")));

cores/esp32/esp32-hal-gpio.h

@@ -79,8 +79,9 @@ void digitalWrite(uint8_t pin, uint8_t val);
 int digitalRead(uint8_t pin);
 
 void attachInterrupt(uint8_t pin, void (*)(void), int mode);
-void attachInterruptArg(uint8_t pin, void (*)(void*), void * arg, int mode);
 void detachInterrupt(uint8_t pin);
+void attachInterruptArg(uint8_t pin, void (*)(void*), void* arg, int mode);
+void* detachInterruptArg(uint8_t pin);
 
 #ifdef __cplusplus
 }

libraries/ESP32/examples/GPIO/FunctionalInterrupt/FunctionalInterrupts.cpp

@@ -2,38 +2,6 @@
 #include "Schedule.h"
 #include "Arduino.h"
 
-#if defined(ESP8266)
-
-// Duplicate typedefs from core_esp8266_wiring_digital.cpp
-// Keep in sync
-typedef void (*voidFuncPtr)(void);
-typedef void (*voidFuncPtrArg)(void*);
-
-typedef struct {
-	uint8_t mode;
-	voidFuncPtr fn;
-	void* arg;
-} interrupt_handler_t;
-
-// Helper functions for Functional interrupt routines
-extern "C" interrupt_handler_t* __getInterruptHandler(uint8_t pin);
-
-#elif defined(ESP32)
-
-// Duplicate typedefs from esp32-hal-gpio.c
-// Keep in sync
-typedef void (*voidFuncPtr)(void);
-typedef void (*voidFuncPtrArg)(void*);
-typedef struct {
-	voidFuncPtr fn;
-	void* arg;
-} InterruptHandle_t;
-
-// Helper functions for Functional interrupt routines
-extern "C" InterruptHandle_t* __getInterruptHandler(uint8_t pin);
-
-#endif
-
 void ICACHE_RAM_ATTR interruptFunctional(void* arg)
 {
 	ArgStructure* localArg = static_cast<ArgStructure*>(arg);
@@ -44,32 +12,30 @@ void ICACHE_RAM_ATTR interruptFunctional(void* arg)
 	}
 	if (localArg->functionInfo->reqScheduledFunction)
 	{
-		schedule_function(std::bind(localArg->functionInfo->reqScheduledFunction,InterruptInfo(*(localArg->interruptInfo))));
+		schedule_function(
+			[reqScheduledFunction = localArg->functionInfo->reqScheduledFunction,
+				interruptInfo = *localArg->interruptInfo]() { reqScheduledFunction(interruptInfo); });
 	}
 	if (localArg->functionInfo->reqFunction)
 	{
 	  localArg->functionInfo->reqFunction();
 	}
 }
 
-   void cleanupFunctional(void* arg)
-   {
-	 ArgStructure* localArg = static_cast<ArgStructure*>(arg);
-	 delete localArg;
-   }
+void cleanupFunctional(void* arg)
+{
+	ArgStructure* localArg = static_cast<ArgStructure*>(arg);
+	delete localArg;
+}
 
 void attachInterrupt(uint8_t pin, std::function<void(void)> intRoutine, int mode)
 {
 	// use the local interrupt routine which takes the ArgStructure as argument
 
-#if defined(ESP8266)
-	interrupt_handler_t* handler = __getInterruptHandler(pin);
-#elif defined(ESP32)
-	InterruptHandle_t* handler = __getInterruptHandler(pin);
-#endif
-	if (handler->arg)
+	void* localArg = detachInterruptArg(pin);
+	if (localArg)
 	{
-		cleanupFunctional(handler->arg);
+		cleanupFunctional(localArg);
 	}
 
 	FunctionInfo* fi = new FunctionInfo;
@@ -78,19 +44,15 @@ void attachInterrupt(uint8_t pin, std::function<void(void)> intRoutine, int mode
 	ArgStructure* as = new ArgStructure;
 	as->functionInfo = fi;
 
-	::attachInterruptArg (pin, static_cast<voidFuncPtrArg>(interruptFunctional), as, mode);
+	attachInterruptArg (pin, interruptFunctional, as, mode);
 }
 
 void attachScheduledInterrupt(uint8_t pin, std::function<void(InterruptInfo)> scheduledIntRoutine, int mode)
 {
-#if defined(ESP8266)
-	interrupt_handler_t* handler = __getInterruptHandler(pin);
-#elif defined(ESP32)
-	InterruptHandle_t* handler = __getInterruptHandler(pin);
-#endif
-	if (handler->arg)
+	void* localArg = detachInterruptArg(pin);
+	if (localArg)
 	{
-		cleanupFunctional(handler->arg);
+		cleanupFunctional(localArg);
 	}
 
 	InterruptInfo* ii = new InterruptInfo(pin);
@@ -102,20 +64,14 @@ void attachScheduledInterrupt(uint8_t pin, std::function<void(InterruptInfo)> sc
 	as->interruptInfo = ii;
 	as->functionInfo = fi;
 
-	::attachInterruptArg (pin, static_cast<voidFuncPtrArg>(interruptFunctional), as, mode);
+	attachInterruptArg(pin, interruptFunctional, as, mode);
 }
 
 void detachFunctionalInterrupt(uint8_t pin)
 {
-#if defined(ESP8266)
-	interrupt_handler_t* handler = __getInterruptHandler(pin);
-#elif defined(ESP32)
-	InterruptHandle_t* handler = __getInterruptHandler(pin);
-#endif
-	if (handler->arg)
+	void* localArg = detachInterruptArg(pin);
+	if (localArg)
 	{
-		cleanupFunctional(handler->arg);
+		cleanupFunctional(localArg);
 	}
-	::detachInterrupt (pin);
 }
-