src: fix typos

PR-URL: #38845
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Khaidi Chu <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anto Aravinth <[email protected]>
Reviewed-By: Zeyu Yang <[email protected]>

src/base_object.h

@@ -191,7 +191,7 @@ class BaseObject : public MemoryRetainer {
     // Indicates whether MakeWeak() has been called.
     bool wants_weak_jsobj = false;
     // Indicates whether Detach() has been called. If that is the case, this
-    // object will be destryoed once the strong pointer count drops to zero.
+    // object will be destroyed once the strong pointer count drops to zero.
     bool is_detached = false;
     // Reference to the original BaseObject. This is used by weak pointers.
     BaseObject* self = nullptr;

src/cares_wrap.cc

@@ -1809,7 +1809,7 @@ void SetLocalAddress(const FunctionCallbackInfo<Value>& args) {
       return;
     }
   } else {
-    // No second arg specifed
+    // No second arg specified
     if (type0 == 4) {
       memset(&addr1, 0, sizeof(addr1));
       ares_set_local_ip6(channel->cares_channel(), addr1);

src/debug_utils.cc

@@ -355,7 +355,7 @@ void PrintLibuvHandleInformation(uv_loop_t* loop, FILE* stream) {
     void* first_field = nullptr;
     // `handle->data` might be any value, including `nullptr`, or something
     // cast from a completely different type; therefore, check that it’s
-    // dereferencable first.
+    // dereferenceable first.
     if (sym_ctx->IsMapped(handle->data))
       first_field = *reinterpret_cast<void**>(handle->data);
 

src/env.h

@@ -908,7 +908,7 @@ class CleanupHookCallback {
 
 struct PropInfo {
   std::string name;     // name for debugging
-  size_t id;            // In the list - in case there are any empty entires
+  size_t id;            // In the list - in case there are any empty entries
   SnapshotIndex index;  // In the snapshot
 };
 

src/js_native_api_v8.cc

@@ -248,7 +248,7 @@ class RefBase : protected Finalizer, RefTracker {
       delete reference;
     } else {
       // defer until finalizer runs as
-      // it may alread be queued
+      // it may already be queued
       reference->_delete_self = true;
     }
   }
@@ -416,7 +416,7 @@ class Reference : public RefBase {
   inline void SetWeak() {
     if (_secondPassParameter == nullptr) {
       // This means that the Reference has already been processed
-      // by the second pass calback, so its already been Finalized, do
+      // by the second pass callback, so its already been Finalized, do
       // nothing
       return;
     }
@@ -455,9 +455,9 @@ class Reference : public RefBase {
   // second pass callback task. We have to make sure that parameter is kept
   // alive until the second pass callback is been invoked. In order to do
   // this and still allow our code to Finalize/delete the Reference during
-  // shutdown we have to use a seperately allocated parameter instead
+  // shutdown we have to use a separately allocated parameter instead
   // of a parameter within the Reference object itself. This is what
-  // is stored in _secondPassParameter and it is alocated in the
+  // is stored in _secondPassParameter and it is allocated in the
   // constructor for the Reference.
   static void SecondPassCallback(
       const v8::WeakCallbackInfo<SecondPassCallParameterRef>& data) {

src/node.cc

@@ -1033,7 +1033,7 @@ InitializationResult InitializeOncePerProcess(int argc, char** argv) {
   // command. If the path to this file is incorrect no error will be reported.
   //
   // For Node.js this will mean that EntropySource will be called by V8 as part
-  // of its initalization process, and EntropySource will in turn call
+  // of its initialization process, and EntropySource will in turn call
   // CheckEntropy. CheckEntropy will call RAND_status which will now always
   // return 0, leading to an endless loop and the node process will appear to
   // hang/freeze.

src/node_http2.cc

@@ -3189,7 +3189,7 @@ void Initialize(Local<Object> target,
 
   Local<Object> constants = Object::New(isolate);
 
-  // This does alocate one more slot than needed but it's not used.
+  // This does allocate one more slot than needed but it's not used.
 #define V(name) FIXED_ONE_BYTE_STRING(isolate, #name),
   Local<Value> error_code_names[] = {
     HTTP2_ERROR_CODES(V)

src/node_i18n.cc

@@ -485,7 +485,7 @@ void ConverterObject::Decode(const FunctionCallbackInfo<Value>& args) {
     }
     ret = ToBufferEndian(env, &result);
     if (omit_initial_bom && !ret.IsEmpty()) {
-      // Peform `ret = ret.slice(2)`.
+      // Perform `ret = ret.slice(2)`.
       CHECK(ret.ToLocalChecked()->IsUint8Array());
       Local<Uint8Array> orig_ret = ret.ToLocalChecked().As<Uint8Array>();
       ret = Buffer::New(env,

src/node_messaging.cc

@@ -162,7 +162,7 @@ MaybeLocal<Value> Message::Deserialize(Environment* env,
       // If we gather a list of all message ports, and this transferred object
       // is a message port, add it to that list. This is a bit of an odd case
       // of special handling for MessagePorts (as opposed to applying to all
-      // transferables), but it's required for spec compliancy.
+      // transferables), but it's required for spec compliance.
       DCHECK((*port_list)->IsArray());
       Local<Array> port_list_array = port_list->As<Array>();
       Local<Object> obj = host_objects[i]->object();
@@ -749,7 +749,7 @@ void MessagePort::OnMessage(MessageProcessingMode mode) {
       // interruption that were already present when the OnMessage() call was
       // first triggered, but at least 1000 messages because otherwise the
       // overhead of repeatedly triggering the uv_async_t instance becomes
-      // noticable, at least on Windows.
+      // noticeable, at least on Windows.
       // (That might require more investigation by somebody more familiar with
       // Windows.)
       TriggerAsync();

src/node_sockaddr.h

@@ -98,7 +98,7 @@ class SocketAddress : public MemoryRetainer {
   // for this one. The addresses are a match if:
   // 1. They are the same family and match identically
   // 2. They are different family but match semantically (
-  //     for instance, an IPv4 addres in IPv6 notation)
+  //     for instance, an IPv4 address in IPv6 notation)
   bool is_match(const SocketAddress& other) const;
 
   // Compares this SocketAddress to the given other SocketAddress.

src/util.h

@@ -404,7 +404,7 @@ class MaybeStackBuffer {
     buf_[length] = T();
   }
 
-  // Make derefencing this object return nullptr.
+  // Make dereferencing this object return nullptr.
   // This method can be called multiple times throughout the lifetime of the
   // buffer, but once this has been called AllocateSufficientStorage() cannot
   // be used.