lib: fix typos

Original-PR-URL: #13741 Original-Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Original-Reviewed-By: Roman Reiss <me@silverwind.io> Original-Reviewed-By: James M Snell <jasnell@gmail.com> Original-Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Original-Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> PR-URL: #14044 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
nodejs · danbev · Mar 24, 2017 · Mar 24, 2017 · May 9, 2017 · May 18, 2017
commit d133e33dbfb994ed9de3778866f7960aa7e729b0
diff --git a/lib/_tls_wrap.js b/lib/_tls_wrap.js
@@ -402,7 +402,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;
 
@@ -705,7 +705,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 -
@@ -984,7 +984,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.

diff --git a/lib/buffer.js b/lib/buffer.js
@@ -58,7 +58,7 @@ function alignPool() {
 }
 
 /**
- * The Buffer() construtor is "soft deprecated" ... that is, it is deprecated
+ * The Buffer() constructor is "soft deprecated" ... that is, it is deprecated
  * in the 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
@@ -124,7 +124,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);

diff --git a/lib/fs.js b/lib/fs.js
@@ -1651,7 +1651,7 @@ fs.realpathSync = function realpathSync(p, options) {
     }
   }
 
-  // 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) {
@@ -1765,7 +1765,7 @@ fs.realpath = function realpath(p, options, callback) {
     }
   }
 
-  // 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

diff --git a/lib/internal/socket_list.js b/lib/internal/socket_list.js
@@ -5,7 +5,7 @@ module.exports = {SocketListSend, SocketListReceive};
 const EventEmitter = require('events');
 const util = require('util');
 
-// This object keep track of the socket there are sended
+// This object keeps track of the sockets that are sent
 function SocketListSend(slave, key) {
   EventEmitter.call(this);
 
@@ -54,7 +54,7 @@ SocketListSend.prototype.getConnections = function getConnections(callback) {
   });
 };
 
-// This object keep track of the socket there are received
+// This object keeps track of the sockets there are received
 function SocketListReceive(slave, key) {
   EventEmitter.call(this);
 
@@ -95,7 +95,7 @@ util.inherits(SocketListReceive, EventEmitter);
 SocketListReceive.prototype.add = function(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--;
 

diff --git a/lib/repl.js b/lib/repl.js
@@ -216,7 +216,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');
   }
 
@@ -817,7 +817,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

diff --git a/lib/timers.js b/lib/timers.js
@@ -367,7 +367,7 @@ exports.setTimeout = function(callback, after, arg1, arg2, arg3) {
 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)
@@ -454,7 +454,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;

diff --git a/lib/tls.js b/lib/tls.js
@@ -10,7 +10,7 @@ const Buffer = require('buffer').Buffer;
 
 // 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;

diff --git a/lib/tty.js b/lib/tty.js
@@ -50,7 +50,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(process.env.NODE_TTY_UNSAFE_ASYNC !== '1');

diff --git a/lib/url.js b/lib/url.js
@@ -436,7 +436,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.