Changes in libcfa/src/concurrency/actor.hfa [1fbf481:b065dbb]

File:

• libcfa/src/concurrency/actor.hfa (modified) (9 diffs)

libcfa/src/concurrency/actor.hfa

@@ -390 +390 @@
 struct actor {
     size_t ticket;                                          // executor-queue handle
-    allocation alloc;                                       // allocation action
+    allocation allocation_;                                         // allocation action
     inline virtual_dtor;
 };
@@ -398 +398 @@
     // member must be called to end it
     DEBUG_ABORT( __actor_executor_ == 0p, "Creating actor before calling start_actor_system() can cause undefined behaviour.\n" );
-    alloc = Nodelete;
+    allocation_ = Nodelete;
     ticket = __get_next_ticket( *__actor_executor_ );
     __atomic_fetch_add( &__num_actors_, 1, __ATOMIC_RELAXED );
@@ -407 +407 @@
 
 static inline void check_actor( actor & this ) {
-    if ( this.alloc != Nodelete ) {
-        switch( this.alloc ) {
+    if ( this.allocation_ != Nodelete ) {
+        switch( this.allocation_ ) {
             case Delete: delete( &this ); break;
             case Destroy:
@@ -427 +427 @@
 
 struct message {
-    allocation alloc;                   // allocation action
+    allocation allocation_;                     // allocation action
     inline virtual_dtor;
 };
 
 static inline void ?{}( message & this ) {
-    this.alloc = Nodelete;
+    this.allocation_ = Nodelete;
 }
 static inline void ?{}( message & this, allocation alloc ) {
-    memcpy( &this.alloc, &alloc, sizeof(allocation) ); // optimization to elide ctor
-    CFA_DEBUG( if( this.alloc == Finished ) this.alloc = Nodelete; )
+    memcpy( &this.allocation_, &alloc, sizeof(allocation) ); // optimization to elide ctor
+    DEBUG_ABORT( this.allocation_ == Finished, "The Finished allocation status is not supported for message types.\n" );
 }
 static inline void ^?{}( message & this ) with(this) {
-    CFA_DEBUG(
-                if ( alloc == Nodelete ) {
-                        printf( "CFA warning (UNIX pid:%ld) : program terminating with message %p allocated but never sent.\n",
-                                        (long int)getpid(), &this );
-                }
-        )
+    CFA_DEBUG( if ( allocation_ == Nodelete ) printf("A message at location %p was allocated but never sent.\n", &this); )
 }
 
 static inline void check_message( message & this ) {
-    switch ( this.alloc ) {                                             // analyze message status
-        case Nodelete: CFA_DEBUG( this.alloc = Finished ); break;
+    switch ( this.allocation_ ) {                                               // analyze message status
+        case Nodelete: CFA_DEBUG( this.allocation_ = Finished ); break;
         case Delete: delete( &this ); break;
         case Destroy: ^?{}( this ); break;
@@ -456 +451 @@
 }
 static inline void set_allocation( message & this, allocation state ) {
-    CFA_DEBUG( if ( state == Nodelete ) state = Finished; )
-    this.alloc = state;
+    this.allocation_ = state;
 }
 
@@ -465 +459 @@
     message * base_msg;
     allocation temp = this.fn( *this.receiver, *this.msg, &base_actor, &base_msg );
-    memcpy( &base_actor->alloc, &temp, sizeof(allocation) ); // optimization to elide ctor
+    base_actor->allocation_ = temp;
     check_message( *base_msg );
     check_actor( *base_actor );
@@ -630 +624 @@
         empty_count = 0; // we found work so reset empty counter
         #endif
-
-        CHECK_TERMINATION;
         
         // potentially reclaim some of the current queue's vector space if it is unused
@@ -677 +669 @@
 
 static inline void stop_actor_system() {
-    park( ); // unparked when actor system is finished
+    park( ); // will be unparked when actor system is finished
 
     if ( !__actor_executor_passed ) delete( __actor_executor_ );
@@ -688 +680 @@
 // Default messages to send to any actor to change status
 // assigned at creation to __base_msg_finished to avoid unused message warning
-message __base_msg_finished @= { .alloc : Finished };
-struct delete_message_t { inline message; } delete_msg = __base_msg_finished;
-struct destroy_msg_t { inline message; } destroy_msg = __base_msg_finished;
-struct finished_msg_t { inline message; } finished_msg = __base_msg_finished;
-
-allocation receive( actor & this, delete_message_t & msg ) { return Delete; }
-allocation receive( actor & this, destroy_msg_t & msg ) { return Destroy; }
-allocation receive( actor & this, finished_msg_t & msg ) { return Finished; }
-
+message __base_msg_finished @= { .allocation_ : Finished };
+struct __delete_msg_t { inline message; } delete_msg = __base_msg_finished;
+struct __destroy_msg_t { inline message; } destroy_msg = __base_msg_finished;
+struct __finished_msg_t { inline message; } finished_msg = __base_msg_finished;
+
+allocation receive( actor & this, __delete_msg_t & msg ) { return Delete; }
+allocation receive( actor & this, __destroy_msg_t & msg ) { return Destroy; }
+allocation receive( actor & this, __finished_msg_t & msg ) { return Finished; }
+