TemporaryCallback has to be strictly reserved for input arguments, or else it may crash.

Author: Dawoodoz

Source/DFPSR/base/TemporaryCallback.h

@@ -33,7 +33,8 @@ namespace dsr {
 //   Either pass a lambda allocated on the stack or declare a new lambda for the implicit constructor.
 //   Use as an input argument to high level functions when the function will not be saved for later.
 //   Do not save a TemporaryCallback for later, because then it will refer to stack memory that is no longer available.
-//     If TemporaryCallback outlives a lambda's closure allocated as a nameless structure on the stack, .
+//     If TemporaryCallback outlives a lambda's closure allocated as a nameless structure on the stack, the stack memory will be deallocated before use.
+//   If you store the callback then it is safer to use StorableCallback, because deleting the copy constructor can not prevent all memory errors.
 template<typename T>
 class TemporaryCallback;
 template<typename RESULT, typename... ARGS>

Source/test/tests/ListTest.cpp

@@ -1,7 +1,7 @@
 
 #include "../testTools.h"
 #include "../../DFPSR/collection/List.h"
-#include "../../DFPSR/base/TemporaryCallback.h"
+#include "../../DFPSR/base/StorableCallback.h"
 
 static void targetByReference(List<int32_t> &target, int32_t value) {
 	target.push(value);
@@ -50,7 +50,7 @@ START_TEST(List)
 		ASSERT_EQUAL(integers.length(), 2);
 		ASSERT_EQUAL(integers[0], 5);
 		ASSERT_EQUAL(integers[1], 86);
-		TemporaryCallback<void(int32_t value)> method([&integers](int32_t value) {
+		StorableCallback<void(int32_t value)> method([&integers](int32_t value) {
 			integers.push(value);
 		});
 		method(24);