doc,src,test: fix typos in code comments

Correct miscellaneous spelling and grammar typos in code comments and
documentation. No code identifiers or runtime logic were modified.

Signed-off-by: Noah Webber <noah.webber@pmg.com>
Assisted-by: Gemini

Author: aguashui

benchmark/fixtures/coverage-many-branches.js

@@ -30,7 +30,7 @@ class CoveredClass {
   }
 }
 
-// Excercise coverage of functions:
+// Exercise coverage of functions:
 function add(x, y) {
   const mt = new CoveredClass(x, y);
   return mt.add();

benchmark/misc/startup-cli-version.js

@@ -8,7 +8,7 @@ const path = require('path');
 // checked into the source code. We use --version because the output
 // tends to be minimal and fewer operations are done to generate
 // these so that the startup cost is still dominated by a more
-// indispensible part of the CLI.
+// indispensable part of the CLI.
 // NOTE: not all tools are present in tarball hence need to filter
 const availableCli = [
   'tools/eslint/node_modules/eslint/bin/eslint.js',

lib/internal/bootstrap/realm.js

@@ -120,7 +120,7 @@ const legacyWrapperList = new SafeSet([
   'util',
 ]);
 
-// The code bellow assumes that the two lists must not contain any modules
+// The code below assumes that the two lists must not contain any modules
 // beginning with "internal/".
 // Modules that can only be imported via the node: scheme.
 const schemelessBlockList = new SafeSet([

lib/internal/main/embedding.js

@@ -121,7 +121,7 @@ function embedderRunESM(content, filename) {
   const wrap = compileSourceTextModule(resourceName, content, kEmbedder, context);
 
   // TODO(joyeecheung): we may want to return the v8::Module via a vm.SourceTextModule
-  // when vm.SourceTextModule stablizes, or put it in an out parameter.
+  // when vm.SourceTextModule stabilizes, or put it in an out parameter.
   return wrap.getNamespace();
 }
 

lib/internal/modules/esm/translators.js

@@ -264,7 +264,7 @@ function createCJSModuleWrap(url, translateContext, parentURL) {
           !ObjectPrototypeHasOwnProperty(exports, exportName)) {
         continue;
       }
-      // We might trigger a getter -> dont fail.
+      // We might trigger a getter -> don't fail.
       let value;
       try {
         value = exports[exportName];

lib/internal/quic/quic.js

@@ -1537,7 +1537,7 @@ async function consumeSyncSource(handle, stream, source) {
       stream.destroy(err);
     } else {
       // If the stream is already destroyed, rethrow the error to avoid
-      // silently swallowing it. Tho in practice this shouldn't happen.
+      // silently swallowing it. Though in practice this shouldn't happen.
       throw err;
     }
   }
@@ -4926,7 +4926,7 @@ class QuicEndpoint {
     // As QuicSessions are closed they are expected to remove themselves
     // from the sessions collection. Just in case they don't, let's force
     // it by resetting the set so we don't leak memory. Let's emit a warning,
-    // tho, if the set is not empty at this point as that would indicate a
+    // though, if the set is not empty at this point as that would indicate a
     // bug in Node.js that should be fixed.
     if (inner.sessions.size > 0) {
       process.emitWarning(

src/cppgc_helpers.h

@@ -90,7 +90,7 @@ class CppgcMixin : public cppgc::GarbageCollectedMixin, public MemoryRetainer {
   // during Realm shutdown. The destruction of the wrappers would happen later,
   // when the final garbage collection is triggered when CppHeap is torn down as
   // part of the Isolate teardown. If subclasses of CppgcMixin wish to perform
-  // cleanups that depend on the Realm during destruction, they should implment
+  // cleanups that depend on the Realm during destruction, they should implement
   // it in a Clean() override, and then call this->Finalize() from their
   // destructor. Outside of Finalize(), subclasses should avoid calling
   // into JavaScript or perform any operation that can trigger garbage

src/dataqueue/queue.cc

@@ -943,11 +943,11 @@ class FdEntry final : public EntryImpl {
     uv_fs_t req = uv_fs_t();
     auto cleanup = OnScopeLeave([&] { uv_fs_req_cleanup(&req); });
     // TODO(jasnell): Note the use of a sync fs call here is a bit unfortunate.
-    // Doing this asynchronously creates a bit of a race condition tho, a file
-    // could be unmodified when we call the operation but then by the time the
-    // async callback is triggered to give us that answer the file is modified.
-    // While such silliness is still possible here, the sync call at least makes
-    // it less likely to hit the race.
+    // Doing this asynchronously creates a bit of a race condition though, a
+    // file could be unmodified when we call the operation but then by the time
+    // the async callback is triggered to give us that answer the file is
+    // modified. While such silliness is still possible here, the sync call at
+    // least makes it less likely to hit the race.
     if (uv_fs_fstat(nullptr, &req, fd, nullptr) < 0) return true;
     return entry->is_modified(req.statbuf);
   }

src/fs_event_wrap.cc

@@ -221,7 +221,7 @@ void FSEventWrap::OnEvent(uv_fs_event_t* handle, const char* filename,
     // TODO(@jasnell): Historically, this code has failed to correctly
     // propagate any error returned by the StringBytes::Encode method,
     // and would instead just crash the process. That behavior is preserved
-    // here but should be looked at. Preferrably errors would be handled
+    // here but should be looked at. Preferably errors would be handled
     // correctly here.
     TryCatch try_catch(env->isolate());
     MaybeLocal<Value> fn =

src/node.cc

@@ -516,7 +516,7 @@ void ResetSignalHandlers() {
       // The only bad handler value we can inherit from before exec is SIG_IGN
       // (any actual function pointer is reset to SIG_DFL during exec).
       // If that's the case, we want to reset it back to SIG_DFL.
-      // However, it's also possible that an embeder (or an LD_PRELOAD-ed
+      // However, it's also possible that an embedder (or an LD_PRELOAD-ed
       // library) has set up own signal handler for own purposes
       // (e.g. profiling). If that's the case, we want to keep it intact.
       struct sigaction old;