Changeset 025278e for src/libcfa/concurrency/monitor

Timestamp:

Nov 2, 2017, 4:38:32 PM (7 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, deferred_resn, demangler, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, resolv-new, with_gc

Children:

513daec

Parents:

8fc45b7

Message:

Changed comments to be above fields, to help source control diff readability

File:

• src/libcfa/concurrency/monitor (modified) (3 diffs)

src/libcfa/concurrency/monitor

@@ -39 +39 @@
 }
 
-// static inline int ?<?(monitor_desc* lhs, monitor_desc* rhs) {
-//      return ((intptr_t)lhs) < ((intptr_t)rhs);
-// }
-
 struct monitor_guard_t {
         monitor_desc ** m;
@@ -74 +70 @@
 
 struct __condition_criterion_t {
-        bool ready;                                             //Whether or not the criterion is met (True if met)
-        monitor_desc * target;                          //The monitor this criterion concerns
-        struct __condition_node_t * owner;              //The parent node to which this criterion belongs
-        __condition_criterion_t * next;         //Intrusive linked list Next field
+        // Whether or not the criterion is met (True if met)
+        bool ready;
+
+        // The monitor this criterion concerns
+        monitor_desc * target;
+
+        // The parent node to which this criterion belongs
+        struct __condition_node_t * owner;
+
+        // Intrusive linked list Next field
+        __condition_criterion_t * next;
+
 };
 
 struct __condition_node_t {
-        thread_desc * waiting_thread;                   //Thread that needs to be woken when all criteria are met
-        __condition_criterion_t * criteria;     //Array of criteria (Criterions are contiguous in memory)
-        unsigned short count;                           //Number of criterions in the criteria
-        __condition_node_t * next;                      //Intrusive linked list Next field
-        uintptr_t user_info;                            //Custom user info accessible before signalling
+        // Thread that needs to be woken when all criteria are met
+        thread_desc * waiting_thread;
+
+        // Array of criteria (Criterions are contiguous in memory)
+        __condition_criterion_t * criteria;
+
+        // Number of criterions in the criteria
+        unsigned short count;
+
+        // Intrusive linked list Next field
+        __condition_node_t * next;
+
+        // Custom user info accessible before signalling
+        uintptr_t user_info;
+
 };
 
@@ -102 +116 @@
 
 struct condition {
-        __condition_blocked_queue_t blocked;    //Link list which contains the blocked threads as-well as the information needed to unblock them
-        monitor_desc ** monitors;                       //Array of monitor pointers (Monitors are NOT contiguous in memory)
-        unsigned short monitor_count;                   //Number of monitors in the array
+        // Link list which contains the blocked threads as-well as the information needed to unblock them
+        __condition_blocked_queue_t blocked;
+
+        // Array of monitor pointers (Monitors are NOT contiguous in memory)
+        monitor_desc ** monitors;
+
+        // Number of monitors in the array
+        unsigned short monitor_count;
+
 };