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]>

Author: bl-ue

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{
 
 structPropInfo{
  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{
 inlinevoidSetWeak(){
 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.
 staticvoidSecondPassCallback(
 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.
 #defineV(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)
 boolis_match(const SocketAddress& other) const;
 
 // Compares this SocketAddress to the given other SocketAddress.