lib,src: fix a few typos in comments

PR-URL: #46835
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>

Author: tniessen

lib/_http_client.js

@@ -779,8 +779,8 @@ function responseOnTimeout() {
   res.emit('timeout');
 }
 
-// This function is necessary in the case where we receive the entire reponse
-// from server before we finish sending out the request
+// This function is necessary in the case where we receive the entire response
+// from the server before we finish sending out the request.
 function requestOnFinish() {
   const req = this;
 

lib/_tls_wrap.js

@@ -426,7 +426,7 @@ function onerror(err) {
     // so self._tlsError will return null instead of actual error
 
     // Set closing the socket after emitting an event since the socket needs to
-    // be accessible when the `tlsClientError` event is emmited.
+    // be accessible when the `tlsClientError` event is emitted.
     owner._closeAfterHandlingError = true;
     owner.destroy(err);
   } else if (owner._tlsOptions?.isServer &&

src/base_object.h

@@ -250,7 +250,7 @@ inline T* Unwrap(v8::Local<v8::Value> obj) {
 // reset to nullptr once the BaseObject is destroyed.
 // The API matches std::shared_ptr closely. However, this class is not thread
 // safe, that is, we can't have different BaseObjectPtrImpl instances in
-// different threads refering to the same BaseObject instance.
+// different threads referring to the same BaseObject instance.
 template <typename T, bool kIsWeak>
 class BaseObjectPtrImpl final {
  public:

src/dataqueue/queue.h

@@ -117,7 +117,7 @@ namespace node {
 //
 // For idempotent DataQueues, any number of readers can be created and
 // pull concurrently from the same DataQueue. The DataQueue can be read
-// multiple times. Succesful reads should always produce the same result.
+// multiple times. Successful reads should always produce the same result.
 // If, for whatever reason, the implementation cannot ensure that the
 // data read will remain the same, the read must fail with an error status.
 //

src/node_contextify.cc

@@ -246,7 +246,7 @@ BaseObjectPtr<ContextifyContext> ContextifyContext::New(
     const ContextOptions& options) {
   HandleScope scope(env->isolate());
   // This only initializes part of the context. The primordials are
-  // only initilaized when needed because even deserializing them slows
+  // only initialized when needed because even deserializing them slows
   // things down significantly and they are only needed in rare occasions
   // in the vm contexts.
   if (InitializeContextRuntime(v8_context).IsNothing()) {