Apply suggestions from code review

wordsmith comments Co-authored-by: Geoffrey Booth <[email protected]> Co-authored-by: Antoine du Hamel <[email protected]>
nodejs · Oct 5, 2022 · 9836c67 · 9836c67
1 parent e1b50cb
commit 9836c67
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/internal/modules/esm/public_loader_proxy.js b/lib/internal/modules/esm/public_loader_proxy.js
@@ -11,8 +11,8 @@ let debug = require('internal/util/debuglog').debuglog('ESMLoader (public)', (fn
 const commsChannel = new SharedArrayBuffer(2048);
 /**
  * The lock/unlock segment of the shared memory. Atomics require this to be a Int32Array. This
- * segment is used to tell the main to sleep when the worker is worker, and vice verse (for the
- * worker to sleep whilst the main is working).
+ * segment is used to tell the main to sleep when the worker is processing, and vice versa (for the
+ * worker to sleep whilst the main thread is processing).
  * 0 → main sleeps
  * 1 → worker sleeps
  */

diff --git a/lib/internal/modules/esm/worker.js b/lib/internal/modules/esm/worker.js
@@ -6,7 +6,7 @@ const { commsChannel } = workerData;
 // lock = 0 → main sleeps
 // lock = 1 → worker sleeps
 const lock = new Int32Array(commsChannel, 0, 4); // Required by Atomics
-const requestResponseData = new Uint8Array(commsChannel, 4, 2044); // for TextEncoder/Decoder
+const requestResponseData = new Uint8Array(commsChannel, 4, 2044); // for v8.serialize/deserialize
 
 const publicESMLoader = new ESMLoader();