lib: fix typos

Original-PR-URL: #13741
Original-Reviewed-By: Colin Ihrig <[email protected]>
Original-Reviewed-By: Roman Reiss <[email protected]>
Original-Reviewed-By: James M Snell <[email protected]>
Original-Reviewed-By: Luigi Pinca <[email protected]>
Original-Reviewed-By: Gibson Fahnestock <[email protected]>

PR-URL: #14044
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>

lib/_tls_wrap.js

@@ -420,7 +420,7 @@ TLSSocket.prototype._init = function(socket, wrap) {
 
   // lib/net.js expect this value to be non-zero if write hasn't been flushed
   // immediately
-  // TODO(indutny): rewise this solution, it might be 1 before handshake and
+  // TODO(indutny): revise this solution, it might be 1 before handshake and
   // represent real writeQueueSize during regular writes.
   ssl.writeQueueSize = 1;
 
@@ -723,7 +723,7 @@ TLSSocket.prototype.getProtocol = function() {
 // outcomes:
 //
 //   A) verifyError returns null meaning the client's certificate is signed
-//   by one of the server's CAs. The server know's the client idenity now
+//   by one of the server's CAs. The server now knows the client's identity
 //   and the client is authorized.
 //
 //   B) For some reason the client's certificate is not acceptable -
@@ -993,7 +993,7 @@ function normalizeConnectArgs(listArgs) {
   var cb = args[1];
 
   // If args[0] was options, then normalize dealt with it.
-  // If args[0] is port, or args[0], args[1]  is host,port, we need to
+  // If args[0] is port, or args[0], args[1] is host, port, we need to
   // find the options and merge them in, normalize's options has only
   // the host/port/path args that it knows about, not the tls options.
   // This means that options.host overrides a host arg.

lib/buffer.js

@@ -119,7 +119,7 @@ function showFlaggedDeprecation() {
   if (bufferWarn) {
     // This is a *pending* deprecation warning. It is not emitted by
     // default unless the --pending-deprecation command-line flag is
-    // used or the NODE_PENDING_DEPRECATION=1 envvar is set.
+    // used or the NODE_PENDING_DEPRECATION=1 env var is set.
     process.emitWarning(bufferWarning, 'DeprecationWarning', 'DEP0005');
     bufferWarn = false;
   }
@@ -131,7 +131,7 @@ const doFlaggedDeprecation =
     function() {};
 
 /**
- * The Buffer() construtor is deprecated in documentation and should not be
+ * The Buffer() constructor is deprecated in documentation and should not be
  * used moving forward. Rather, developers should use one of the three new
  * factory APIs: Buffer.from(), Buffer.allocUnsafe() or Buffer.alloc() based on
  * their specific needs. There is no runtime deprecation because of the extent
@@ -231,7 +231,7 @@ Buffer.alloc = function(size, fill, encoding) {
     // Since we are filling anyway, don't zero fill initially.
     // Only pay attention to encoding if it's a string. This
     // prevents accidentally sending in a number that would
-    // be interpretted as a start offset.
+    // be interpreted as a start offset.
     if (typeof encoding !== 'string')
       encoding = undefined;
     return createUnsafeBuffer(size).fill(fill, encoding);

lib/fs.js

@@ -1620,7 +1620,7 @@ fs.realpathSync = function realpathSync(p, options) {
     knownHard[base] = true;
   }
 
-  // walk down the path, swapping out linked pathparts for their real
+  // walk down the path, swapping out linked path parts for their real
   // values
   // NB: p.length changes.
   while (pos < p.length) {
@@ -1747,7 +1747,7 @@ fs.realpath = function realpath(p, options, callback) {
     process.nextTick(LOOP);
   }
 
-  // walk down the path, swapping out linked pathparts for their real
+  // walk down the path, swapping out linked path parts for their real
   // values
   function LOOP() {
     // stop if scanned past end of path

lib/internal/socket_list.js

@@ -2,7 +2,7 @@
 
 const EventEmitter = require('events');
 
-// This object keep track of the socket there are sended
+// This object keeps track of the sockets that are sent
 class SocketListSend extends EventEmitter {
   constructor(child, key) {
     super();
@@ -52,7 +52,7 @@ class SocketListSend extends EventEmitter {
 }
 
 
-// This object keep track of the socket there are received
+// This object keeps track of the sockets that are received
 class SocketListReceive extends EventEmitter {
   constructor(child, key) {
     super();
@@ -93,7 +93,7 @@ class SocketListReceive extends EventEmitter {
   add(obj) {
     this.connections++;
 
-    // Notify previous owner of socket about its state change
+    // Notify the previous owner of the socket about its state change
     obj.socket.once('close', () => {
       this.connections--;
 

lib/repl.js

@@ -137,7 +137,7 @@ function REPLServer(prompt,
 
   if (breakEvalOnSigint && eval_) {
     // Allowing this would not reflect user expectations.
-    // breakEvalOnSigint affects only the behaviour of the default eval().
+    // breakEvalOnSigint affects only the behavior of the default eval().
     throw new Error('Cannot specify both breakEvalOnSigint and eval for REPL');
   }
 
@@ -710,7 +710,7 @@ REPLServer.prototype.complete = function() {
 function complete(line, callback) {
   // There may be local variables to evaluate, try a nested REPL
   if (this.bufferedCommand !== undefined && this.bufferedCommand.length) {
-    // Get a new array of inputed lines
+    // Get a new array of inputted lines
     var tmp = this.lines.slice();
     // Kill off all function declarations to push all local variables into
     // global scope

lib/timers.js

@@ -467,7 +467,7 @@ exports.setTimeout = setTimeout;
 function createSingleTimeout(callback, after, args) {
   after *= 1; // coalesce to number or NaN
   if (!(after >= 1 && after <= TIMEOUT_MAX))
-    after = 1; // schedule on next tick, follows browser behaviour
+    after = 1; // schedule on next tick, follows browser behavior
 
   var timer = new Timeout(after, callback, args);
   if (process.domain)
@@ -556,7 +556,7 @@ exports.setInterval = function(callback, repeat, arg1, arg2, arg3) {
 function createRepeatTimeout(callback, repeat, args) {
   repeat *= 1; // coalesce to number or NaN
   if (!(repeat >= 1 && repeat <= TIMEOUT_MAX))
-    repeat = 1; // schedule on next tick, follows browser behaviour
+    repeat = 1; // schedule on next tick, follows browser behavior
 
   var timer = new Timeout(repeat, callback, args);
   timer._repeat = repeat;

lib/tls.js

@@ -32,7 +32,7 @@ const { isUint8Array } = process.binding('util');
 
 // Allow {CLIENT_RENEG_LIMIT} client-initiated session renegotiations
 // every {CLIENT_RENEG_WINDOW} seconds. An error event is emitted if more
-// renegotations are seen. The settings are applied to all remote client
+// renegotiations are seen. The settings are applied to all remote client
 // connections.
 exports.CLIENT_RENEG_LIMIT = 3;
 exports.CLIENT_RENEG_WINDOW = 600;

lib/tty.js

@@ -77,7 +77,7 @@ function WriteStream(fd) {
 
   // Prevents interleaved or dropped stdout/stderr output for terminals.
   // As noted in the following reference, local TTYs tend to be quite fast and
-  // this behaviour has become expected due historical functionality on OS X,
+  // this behavior has become expected due historical functionality on OS X,
   // even though it was originally intended to change in v1.0.2 (Libuv 1.2.1).
   // Ref: https://github.com/nodejs/node/pull/1771#issuecomment-119351671
   this._handle.setBlocking(true);

lib/url.js

@@ -449,7 +449,7 @@ function autoEscapeStr(rest) {
   var lastEscapedPos = 0;
   for (var i = 0; i < rest.length; ++i) {
     // Manual switching is faster than using a Map/Object.
-    // `escaped` contains substring up to the last escaped cahracter.
+    // `escaped` contains substring up to the last escaped character.
     switch (rest.charCodeAt(i)) {
       case 9:   // '\t'
         // Concat if there are ordinary characters in the middle.