Changeset d40555e for libcfa/src/concurrency

Timestamp:

Mar 9, 2023, 10:58:59 AM (18 months ago)

Author:

caparsons <caparson@…>

Branches:

ADT, ast-experimental, master

Children:

0f275f7

Parents:

dd16dd5

Message:

fixed matrix.cfa test error by reworking globals to avoid spurious dtor calls

File:

• libcfa/src/concurrency/actor.hfa (modified) (1 diff)

libcfa/src/concurrency/actor.hfa

@@ -681 +681 @@
 // assigned at creation to __base_msg_finished to avoid unused message warning
 message __base_msg_finished @= { .allocation_ : Finished };
-struct __DeleteMsg { inline message; } DeleteMsg = __base_msg_finished;
-struct __DestroyMsg { inline message; } DestroyMsg = __base_msg_finished;
-struct __FinishedMsg { inline message; } FinishedMsg = __base_msg_finished;
-void ?{}( __DeleteMsg & this, Allocation status ) { ((message &) this){ status }; }
-void ?{}( __DestroyMsg & this, Allocation status ) { ((message &) this){ status }; }
-void ?{}( __FinishedMsg & this, Allocation status ) { ((message &) this){ status }; }
+struct __DeleteMsg { inline message; };
+struct __DestroyMsg { inline message; };
+struct __FinishedMsg { inline message; };
+
+// These are needed so that the compiler doesn't make a temporary when initializing below
+static inline void ?{}( __DeleteMsg & this, message & other ) { this.allocation_ = other.allocation_; }
+static inline void ?{}( __DestroyMsg & this, message & other ) { this.allocation_ = other.allocation_; }
+static inline void ?{}( __FinishedMsg & this, message & other ) { this.allocation_ = other.allocation_; }
+
+__DeleteMsg DeleteMsg = __base_msg_finished;
+__DestroyMsg DestroyMsg = __base_msg_finished;
+__FinishedMsg FinishedMsg = __base_msg_finished;
 
 Allocation receive( actor & this, __DeleteMsg & msg ) { return Delete; }