fix #495: VM emulator ignores Sys.wait

use javascript setTimeout instead of busy wait loop
nand2tetris · Dec 10, 2024 · 3d44ae7 · 3d44ae7
1 parent 36e768d
commit 3d44ae7
Showing 1 changed file with 5 additions and 17 deletions.
diff --git a/simulator/src/vm/os/sys.ts b/simulator/src/vm/os/sys.ts
@@ -53,25 +53,13 @@ export class SysLib {
       this.error(ERRNO.SYS_WAIT_DURATION_NOT_POSITIVE);
       return;
     }
-    this.block();
-
-    let waited = 0;
-
-    const loop = (delta: number) => {
-      if (this.cancelWait) {
-        return;
-      }
 
-      waited += delta;
-
-      if (waited >= ms) {
-        this.release();
-      } else {
-        this.animationFrameId = requestAnimationFrame(loop);
-      }
-    };
+    this.block();
 
-    loop(0);
+    (async () => {
+      await new Promise(x => setTimeout(x, ms));
+      this.release();
+    })();
   }
 
   halt() {