Fix deletion of argument value caused by forwarding to Delegate.

dok-net · dok-net · commit 79c3816ecf93 · 2021-06-28T12:48:34.000+02:00
diff --git a/cores/esp8266/MultiDelegate.h b/cores/esp8266/MultiDelegate.h
@@ -261,7 +261,7 @@ namespace detail
             do
             {
                 done = current == stop;
-                if (!CallP<Delegate, R, ISQUEUE, P...>::execute(current->mDelegate, std::forward<P...>(args...)))
+                if (!CallP<Delegate, R, ISQUEUE, P...>::execute(current->mDelegate, args...))
                 {
                     // remove callback from stack
                     esp8266::InterruptLock lockAllInterruptsInThisScope;

Original file line number	Diff line number	Diff line change
`@@ -261,7 +261,7 @@ namespace detail`
`261`	`261`	`do`
`262`	`262`	`{`
`263`	`263`	`done = current == stop;`
`264`		`- if (!CallP<Delegate, R, ISQUEUE, P...>::execute(current->mDelegate, std::forward<P...>(args...)))`
	`264`	`+ if (!CallP<Delegate, R, ISQUEUE, P...>::execute(current->mDelegate, args...))`
`265`	`265`	`{`
`266`	`266`	`// remove callback from stack`
`267`	`267`	`esp8266::InterruptLock lockAllInterruptsInThisScope;`