Add assertions against fibers waiting on themselves

rmrfslashhome · Dec 21, 2024 · e701ac2 · e701ac2
1 parent bdaf1ad
commit e701ac2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/sync/Context.mpl b/sync/Context.mpl
@@ -62,7 +62,9 @@ Context: [{
 
   wait: [
     done? ~ [
-      [data.waitedBy nil?] "multiple attempts to wait on the same fiber" assert
+      [data.waitedBy nil?          ] "multiple attempts to wait on the same fiber" assert
+      [data.fiber currentFiber is ~] "attempt to wait on self"                     assert
+
       @currentFiber @data.!waitedBy
       canceled? [
         cancel

diff --git a/sync/ContextGroup.mpl b/sync/ContextGroup.mpl
@@ -79,7 +79,9 @@ ContextGroup: [{
     ] when
 
     [fibers.empty? ~] [
-      [fibers.first.waitedBy nil?] "multiple attempts to wait on the same fiber" assert
+      [fibers.first.waitedBy nil?          ] "multiple attempts to wait on the same fiber" assert
+      [fibers.first.fiber currentFiber is ~] "attempt to wait on self"                     assert
+
       @currentFiber @fibers.@first.!waitedBy
       canceled? ~ [
         [currentFiber.@func @defaultCancelFunc is] "invalid cancelation function" assert