23 files changed, 72 insertions, 74 deletions
diff --git a/lib/_http_agent.js b/lib/_http_agent.js
index 17f6856bc7..8632da4572 100644
--- a/lib/_http_agent.js
+++ b/lib/_http_agent.js
@@ -162,7 +162,7 @@ Agent.prototype.addRequest = function addRequest(req, options, port/* legacy */,
   var sockLen = freeLen + this.sockets[name].length;
 
   if (freeLen) {
-    // we have a free socket, so use that.
+    // We have a free socket, so use that.
     var socket = this.freeSockets[name].shift();
     // Guard against an uninitialized or user supplied Socket.
     if (socket._handle && typeof socket._handle.asyncReset === 'function') {
@@ -357,7 +357,7 @@ function setRequestSocket(agent, req, socket) {
     return;
   }
   socket.setTimeout(req.timeout);
-  // reset timeout after response end
+  // Reset timeout after response end
   req.once('response', (res) => {
     res.once('end', () => {
       if (socket.timeout !== agentTimeout) {
diff --git a/lib/_http_common.js b/lib/_http_common.js
index 7ecef8b83d..a83c82c9d4 100644
--- a/lib/_http_common.js
+++ b/lib/_http_common.js
@@ -147,7 +147,7 @@ function parserOnMessageComplete() {
     stream.push(null);
   }
 
-  // force to read the next incoming message
+  // Force to read the next incoming message
   readStart(parser.socket);
 }
 
diff --git a/lib/_http_server.js b/lib/_http_server.js
index adf2543cfe..7a26759618 100644
--- a/lib/_http_server.js
+++ b/lib/_http_server.js
@@ -594,7 +594,7 @@ function resOnFinish(req, res, socket, state, server) {
       state.keepAliveTimeoutSet = true;
     }
   } else {
-    // start sending the next message
+    // Start sending the next message
     var m = state.outgoing.shift();
     if (m) {
       m.assignSocket(socket);
diff --git a/lib/_stream_duplex.js b/lib/_stream_duplex.js
index 7551af7f4e..83aac732e9 100644
--- a/lib/_stream_duplex.js
+++ b/lib/_stream_duplex.js
@@ -124,7 +124,7 @@ Object.defineProperty(Duplex.prototype, 'destroyed', {
     return this._readableState.destroyed && this._writableState.destroyed;
   },
   set(value) {
-    // we ignore the value if the stream
+    // We ignore the value if the stream
     // has not been initialized yet
     if (this._readableState === undefined ||
         this._writableState === undefined) {
diff --git a/lib/_stream_readable.js b/lib/_stream_readable.js
index b3a1e1933a..4e162b97f7 100644
--- a/lib/_stream_readable.js
+++ b/lib/_stream_readable.js
@@ -182,7 +182,7 @@ Object.defineProperty(Readable.prototype, 'destroyed', {
     return this._readableState.destroyed;
   },
   set(value) {
-    // we ignore the value if the stream
+    // We ignore the value if the stream
     // has not been initialized yet
     if (!this._readableState) {
       return;
@@ -600,7 +600,7 @@ function maybeReadMore_(stream, state) {
     debug('maybeReadMore read 0');
     stream.read(0);
     if (len === state.length)
-      // didn't get any data, stop spinning.
+      // Didn't get any data, stop spinning.
       break;
   }
   state.readingMore = false;
@@ -741,7 +741,7 @@ Readable.prototype.pipe = function(dest, pipeOpts) {
     src.unpipe(dest);
   }
 
-  // tell the dest that it's being piped to
+  // Tell the dest that it's being piped to
   dest.emit('pipe', src);
 
   // Start the flow if it hasn't been started already.
@@ -793,7 +793,7 @@ Readable.prototype.unpipe = function(dest) {
     return this;
   }
 
-  // slow case. multiple pipe destinations.
+  // Slow case with multiple pipe destinations.
 
   if (!dest) {
     // remove all.
@@ -896,7 +896,7 @@ function updateReadableListening(self) {
     // the upcoming resume will not flow.
     state.flowing = true;
 
-    // crude way to check if we should resume
+    // Crude way to check if we should resume
   } else if (self.listenerCount('data') > 0) {
     self.resume();
   }
@@ -1008,7 +1008,7 @@ Readable.prototype.wrap = function(stream) {
     }
   }
 
-  // proxy certain important events.
+  // Proxy certain important events.
   for (var n = 0; n < kProxyEvents.length; n++) {
     stream.on(kProxyEvents[n], this.emit.bind(this, kProxyEvents[n]));
   }
@@ -1070,7 +1070,7 @@ Object.defineProperty(Readable.prototype, 'readableFlowing', {
   }
 });
 
-// exposed for testing purposes only.
+// Exposed for testing purposes only.
 Readable._fromList = fromList;
 
 Object.defineProperty(Readable.prototype, 'readableLength', {
@@ -1096,7 +1096,7 @@ function fromList(n, state) {
   if (state.objectMode)
     ret = state.buffer.shift();
   else if (!n || n >= state.length) {
-    // read it all, truncate the list
+    // Read it all, truncate the list
     if (state.decoder)
       ret = state.buffer.join('');
     else if (state.buffer.length === 1)
diff --git a/lib/_stream_writable.js b/lib/_stream_writable.js
index 4c60bf7d7a..e5b11d06a1 100644
--- a/lib/_stream_writable.js
+++ b/lib/_stream_writable.js
@@ -434,7 +434,7 @@ function onwriteError(stream, state, sync, er, cb) {
     cb(er);
     stream._writableState.errorEmitted = true;
     errorOrDestroy(stream, er);
-    // this can emit finish, but finish must
+    // This can emit finish, but finish must
     // always follow error
     finishMaybe(stream, state);
   }
@@ -589,7 +589,7 @@ Writable.prototype.end = function(chunk, encoding, cb) {
     this.uncork();
   }
 
-  // ignore unnecessary end() calls.
+  // Ignore unnecessary end() calls.
   if (!state.ending)
     endWritable(this, state, cb);
 
@@ -697,7 +697,7 @@ Object.defineProperty(Writable.prototype, 'destroyed', {
     return this._writableState.destroyed;
   },
   set(value) {
-    // we ignore the value if the stream
+    // We ignore the value if the stream
     // has not been initialized yet
     if (!this._writableState) {
       return;
diff --git a/lib/dgram.js b/lib/dgram.js
index e48c117eed..ceda826530 100644
--- a/lib/dgram.js
+++ b/lib/dgram.js
@@ -293,7 +293,7 @@ Socket.prototype.bind = function(port_, address_ /* , callback */) {
       });
     } else {
       if (!state.handle)
-        return; // handle has been closed in the mean time
+        return; // Handle has been closed in the mean time
 
       const err = state.handle.bind(ip, port || 0, flags);
       if (err) {
diff --git a/lib/domain.js b/lib/domain.js
index 76b27dc058..24cc406132 100644
--- a/lib/domain.js
+++ b/lib/domain.js
@@ -286,7 +286,7 @@ Domain.prototype.exit = function() {
   var index = stack.lastIndexOf(this);
   if (index === -1) return;
 
-  // exit all domains until this one.
+  // Exit all domains until this one.
   stack.splice(index);
 
   exports.active = stack[stack.length - 1];
@@ -301,7 +301,7 @@ Domain.prototype.add = function(ee) {
   if (ee.domain === this)
     return;
 
-  // has a domain already - remove it first.
+  // Has a domain already - remove it first.
   if (ee.domain)
     ee.domain.remove(ee);
 
diff --git a/lib/fs.js b/lib/fs.js
index e8ba400a4b..e5abc59b59 100644
--- a/lib/fs.js
+++ b/lib/fs.js
@@ -368,7 +368,7 @@ function readFileSync(path, options) {
     } while (bytesRead !== 0 && pos < size);
   } else {
     do {
-      // the kernel lies about many files.
+      // The kernel lies about many files.
       // Go ahead and try to read some bytes.
       buffer = Buffer.allocUnsafe(8192);
       bytesRead = tryReadSync(fd, isUserFd, buffer, 0, 8192);
@@ -1414,7 +1414,7 @@ function realpathSync(p, options) {
   const knownHard = Object.create(null);
   const original = p;
 
-  // current character position in p
+  // Current character position in p
   let pos;
   // The partial path so far, including a trailing slash if any
   let current;
@@ -1481,7 +1481,7 @@ function realpathSync(p, options) {
         continue;
       }
 
-      // read the link if it wasn't read before
+      // Read the link if it wasn't read before
       // dev/ino always return 0 on windows, so skip the check.
       let linkTarget = null;
       let id;
@@ -1506,7 +1506,7 @@ function realpathSync(p, options) {
       if (!isWindows) seenLinks[id] = linkTarget;
     }
 
-    // resolve the link, then start over
+    // Resolve the link, then start over
     p = pathModule.resolve(resolvedLink, p.slice(pos));
 
     // Skip over roots
@@ -1554,7 +1554,7 @@ function realpath(p, options, callback) {
   const seenLinks = Object.create(null);
   const knownHard = Object.create(null);
 
-  // current character position in p
+  // Current character position in p
   let pos;
   // The partial path so far, including a trailing slash if any
   let current;
@@ -1580,7 +1580,7 @@ function realpath(p, options, callback) {
   // Walk down the path, swapping out linked path parts for their real
   // values
   function LOOP() {
-    // stop if scanned past end of path
+    // Stop if scanned past end of path
     if (pos >= p.length) {
       return callback(null, encodeRealpathResult(p, options));
     }
@@ -1620,9 +1620,9 @@ function realpath(p, options, callback) {
       return process.nextTick(LOOP);
     }
 
-    // stat & read the link if not read before
-    // call gotTarget as soon as the link target is known
-    // dev/ino always return 0 on windows, so skip the check.
+    // Stat & read the link if not read before.
+    // Call `gotTarget()` as soon as the link target is known.
+    // `dev`/`ino` always return 0 on windows, so skip the check.
     let id;
     if (!isWindows) {
       const dev = stats.dev.toString(32);
@@ -1649,7 +1649,7 @@ function realpath(p, options, callback) {
   }
 
   function gotResolvedLink(resolvedLink) {
-    // resolve the link, then start over
+    // Resolve the link, then start over
     p = pathModule.resolve(resolvedLink, p.slice(pos));
     current = base = splitRoot(p);
     pos = current.length;
diff --git a/lib/internal/child_process.js b/lib/internal/child_process.js
index 4414b002ae..be90d23f2a 100644
--- a/lib/internal/child_process.js
+++ b/lib/internal/child_process.js
@@ -97,7 +97,7 @@ const handleConversion = {
       if (!socket._handle)
         return;
 
-      // if the socket was created by net.Server
+      // If the socket was created by net.Server
       if (socket.server) {
         // The worker should keep track of the socket
         message.key = socket.server._connectionKey;
@@ -169,7 +169,7 @@ const handleConversion = {
       // If the socket was created by net.Server we will track the socket
       if (message.key) {
 
-        // add socket to connections list
+        // Add socket to connections list
         var socketList = getSocketList('got', this, message.key);
         socketList.add({
           socket: socket
@@ -561,7 +561,7 @@ function setupChannel(target, channel) {
     }
   };
 
-  // object where socket lists will live
+  // Object where socket lists will live
   channel.sockets = { got: {}, send: {} };
 
   // handlers will go through this
@@ -670,7 +670,7 @@ function setupChannel(target, channel) {
       options = { swallowErrors: options };
     }
 
-    // package messages with a handle object
+    // Package messages with a handle object
     if (handle) {
       // This message will be handled by an internalMessage event handler
       message = {
diff --git a/lib/internal/fs/utils.js b/lib/internal/fs/utils.js
index fb36a173c3..8a9d4d3572 100644
--- a/lib/internal/fs/utils.js
+++ b/lib/internal/fs/utils.js
@@ -393,7 +393,7 @@ function toUnixTimestamp(time, name = 'time') {
     return time;
   }
   if (util.isDate(time)) {
-    // convert to 123.456 UNIX timestamp
+    // Convert to 123.456 UNIX timestamp
     return time.getTime() / 1000;
   }
   throw new ERR_INVALID_ARG_TYPE(name, ['Date', 'Time in seconds'], time);
diff --git a/lib/internal/fs/watchers.js b/lib/internal/fs/watchers.js
index 7bc6f2ac97..a785b8ded8 100644
--- a/lib/internal/fs/watchers.js
+++ b/lib/internal/fs/watchers.js
@@ -118,7 +118,7 @@ function FSWatcher() {
       if (this._handle !== null) {
         // We don't use this.close() here to avoid firing the close event.
         this._handle.close();
-        this._handle = null;  // make the handle garbage collectable
+        this._handle = null;  // Make the handle garbage collectable
       }
       const error = errors.uvException({
         errno: status,
@@ -185,7 +185,7 @@ FSWatcher.prototype.close = function() {
     return;
   }
   this._handle.close();
-  this._handle = null;  // make the handle garbage collectable
+  this._handle = null;  // Make the handle garbage collectable
   process.nextTick(emitCloseNT, this);
 };
 
diff --git a/lib/internal/http2/compat.js b/lib/internal/http2/compat.js
index 286bc51683..cad6efc733 100644
--- a/lib/internal/http2/compat.js
+++ b/lib/internal/http2/compat.js
@@ -130,7 +130,7 @@ function onStreamEnd() {
 }
 
 function onStreamError(error) {
-  // this is purposefully left blank
+  // This is purposefully left blank
   //
   // errors in compatibility mode are
   // not forwarded to the request
diff --git a/lib/internal/http2/core.js b/lib/internal/http2/core.js
index 21f780c733..c17980993a 100644
--- a/lib/internal/http2/core.js
+++ b/lib/internal/http2/core.js
@@ -1732,7 +1732,7 @@ class Http2Stream extends Duplex {
     this.emit('timeout');
   }
 
-  // true if the HEADERS frame has been sent
+  // True if the HEADERS frame has been sent
   get headersSent() {
     return !!(this[kState].flags & STREAM_FLAGS_HEADERS_SENT);
   }
@@ -1742,7 +1742,7 @@ class Http2Stream extends Duplex {
     return !!(this[kState].flags & STREAM_FLAGS_ABORTED);
   }
 
-  // true if dealing with a HEAD request
+  // True if dealing with a HEAD request
   get headRequest() {
     return !!(this[kState].flags & STREAM_FLAGS_HEAD_REQUEST);
   }
diff --git a/lib/internal/modules/cjs/loader.js b/lib/internal/modules/cjs/loader.js
index def440da63..f5dff9b5c1 100644
--- a/lib/internal/modules/cjs/loader.js
+++ b/lib/internal/modules/cjs/loader.js
@@ -335,7 +335,7 @@ Module._findPath = function(request, paths, isMain) {
       }
 
       if (!filename) {
-        // try it with each of the extensions
+        // Try it with each of the extensions
         if (exts === undefined)
           exts = Object.keys(Module._extensions);
         filename = tryExtensions(basePath, exts, isMain);
@@ -378,7 +378,7 @@ var nmLen = nmChars.length;
 if (isWindows) {
   // 'from' is the __dirname of the module.
   Module._nodeModulePaths = function(from) {
-    // guarantee that 'from' is absolute.
+    // Guarantee that 'from' is absolute.
     from = path.resolve(from);
 
     // note: this approach *only* works when the path is guaranteed
@@ -422,7 +422,7 @@ if (isWindows) {
 } else { // posix
   // 'from' is the __dirname of the module.
   Module._nodeModulePaths = function(from) {
-    // guarantee that 'from' is absolute.
+    // Guarantee that 'from' is absolute.
     from = path.resolve(from);
     // Return early not only to avoid unnecessary work, but to *avoid* returning
     // an array of two items for a root: [ '//node_modules', '/node_modules' ]
@@ -836,7 +836,7 @@ Module._extensions['.node'] = function(module, filename) {
     const moduleURL = pathToFileURL(filename);
     manifest.assertIntegrity(moduleURL, content);
   }
-  // be aware this doesn't use `content`
+  // Be aware this doesn't use `content`
   return process.dlopen(module, path.toNamespacedPath(filename));
 };
 
diff --git a/lib/internal/readline.js b/lib/internal/readline.js
index f1716f1f25..b05c2d575a 100644
--- a/lib/internal/readline.js
+++ b/lib/internal/readline.js
@@ -244,7 +244,7 @@ function* emitKeys(stream) {
          */
         const cmdStart = s.length - 1;
 
-        // skip one or two leading digits
+        // Skip one or two leading digits
         if (ch >= '0' && ch <= '9') {
           s += (ch = yield);
 
@@ -386,7 +386,7 @@ function* emitKeys(stream) {
       // carriage return
       key.name = 'return';
     } else if (ch === '\n') {
-      // enter, should have been called linefeed
+      // Enter, should have been called linefeed
       key.name = 'enter';
     } else if (ch === '\t') {
       // tab
diff --git a/lib/internal/stream_base_commons.js b/lib/internal/stream_base_commons.js
index 31291e751d..9c3b60e584 100644
--- a/lib/internal/stream_base_commons.js
+++ b/lib/internal/stream_base_commons.js
@@ -162,7 +162,7 @@ function onStreamRead(arrayBuffer) {
     return stream.destroy(errnoException(nread, 'read'));
   }
 
-  // defer this until we actually emit end
+  // Defer this until we actually emit end
   if (stream._readableState.endEmitted) {
     if (stream[kMaybeDestroy])
       stream[kMaybeDestroy]();
@@ -170,7 +170,7 @@ function onStreamRead(arrayBuffer) {
     if (stream[kMaybeDestroy])
       stream.on('end', stream[kMaybeDestroy]);
 
-    // push a null to signal the end of data.
+    // Push a null to signal the end of data.
     // Do it before `maybeDestroy` for correct order of events:
     // `end` -> `close`
     stream.push(null);
diff --git a/lib/internal/streams/async_iterator.js b/lib/internal/streams/async_iterator.js
index 810132c0cb..0d22e4efee 100644
--- a/lib/internal/streams/async_iterator.js
+++ b/lib/internal/streams/async_iterator.js
@@ -85,10 +85,9 @@ const ReadableStreamAsyncIteratorPrototype = Object.setPrototypeOf({
       });
     }
 
-    // if we have multiple next() calls
-    // we will wait for the previous Promise to finish
-    // this logic is optimized to support for await loops,
-    // where next() is only called once at a time
+    // If we have multiple next() calls we will wait for the previous Promise to
+    // finish. This logic is optimized to support for await loops, where next()
+    // is only called once at a time.
     const lastPromise = this[kLastPromise];
     let promise;
 
@@ -96,7 +95,7 @@ const ReadableStreamAsyncIteratorPrototype = Object.setPrototypeOf({
       promise = new Promise(wrapForNext(lastPromise, this));
     } else {
       // Fast path needed to support multiple this.push()
-      // without triggering the next() queue
+      // without triggering the next() queue.
       const data = this[kStream].read();
       if (data !== null) {
         return Promise.resolve(createIterResult(data, false));
@@ -136,7 +135,7 @@ const createReadableStreamAsyncIterator = (stream) => {
       value: stream._readableState.endEmitted,
       writable: true
     },
-    // the function passed to new Promise
+    // The function passed to new Promise
     // is cached so we avoid allocating a new
     // closure at every run
     [kHandlePromise]: {
diff --git a/lib/net.js b/lib/net.js
index e46dea4928..2af6fedfd3 100644
--- a/lib/net.js
+++ b/lib/net.js
@@ -535,7 +535,7 @@ Socket.prototype._read = function(n) {
     debug('_read wait for connection');
     this.once('connect', () => this._read(n));
   } else if (!this._handle.reading) {
-    // not already reading, start the flow
+    // Not already reading, start the flow
     debug('Socket._read readStart');
     this._handle.reading = true;
     var err = this._handle.readStart();
@@ -1049,7 +1049,7 @@ Socket.prototype.unref = function() {
 function afterConnect(status, handle, req, readable, writable) {
   var self = handle[owner_symbol];
 
-  // callback may come after call to destroy
+  // Callback may come after call to destroy
   if (self.destroyed) {
     return;
   }
diff --git a/lib/readline.js b/lib/readline.js
index 46fc304770..585128fe02 100644
--- a/lib/readline.js
+++ b/lib/readline.js
@@ -768,7 +768,7 @@ Interface.prototype._moveCursor = function(dx) {
 
   var newPos = this._getCursorPos();
 
-  // check if cursors are in the same line
+  // Check if cursors are in the same line
   if (oldPos.rows === newPos.rows) {
     var diffCursor = this.cursor - oldcursor;
     var diffWidth;
@@ -837,11 +837,11 @@ Interface.prototype._ttyWrite = function(s, key) {
         }
         break;
 
-      case 'u': // delete from current to start of line
+      case 'u': // Delete from current to start of line
         this._deleteLineLeft();
         break;
 
-      case 'k': // delete from current to end of line
+      case 'k': // Delete from current to end of line
         this._deleteLineRight();
         break;
 
@@ -901,12 +901,12 @@ Interface.prototype._ttyWrite = function(s, key) {
         }
         break;
 
-      case 'w': // delete backwards to a word boundary
+      case 'w': // Delete backwards to a word boundary
       case 'backspace':
         this._deleteWordLeft();
         break;
 
-      case 'delete': // delete forward to a word boundary
+      case 'delete': // Delete forward to a word boundary
         this._deleteWordRight();
         break;
 
@@ -936,7 +936,7 @@ Interface.prototype._ttyWrite = function(s, key) {
         this._deleteWordRight();
         break;
 
-      case 'backspace': // delete backwards to a word boundary
+      case 'backspace': // Delete backwards to a word boundary
         this._deleteWordLeft();
         break;
     }
@@ -972,7 +972,7 @@ Interface.prototype._ttyWrite = function(s, key) {
         break;
 
       case 'left':
-        // obtain the code point to the left
+        // Obtain the code point to the left
         this._moveCursor(-charLengthLeft(this.line, this.cursor));
         break;
 
diff --git a/lib/sys.js b/lib/sys.js
index 87f40fc6d5..1292a38e9c 100644
--- a/lib/sys.js
+++ b/lib/sys.js
@@ -21,9 +21,8 @@
 
 'use strict';
 
-// the sys module was renamed to 'util'.
-// this shim remains to keep old programs working.
-// sys is deprecated and shouldn't be used
+// The sys module was renamed to 'util'. This shim remains to keep old programs
+// working. `sys` is deprecated and shouldn't be used.
 
 module.exports = require('util');
 process.emitWarning('sys is deprecated. Use util instead.',
diff --git a/lib/url.js b/lib/url.js
index 78dcbe7130..391d6c2723 100644
--- a/lib/url.js
+++ b/lib/url.js
@@ -80,12 +80,12 @@ const unsafeProtocol = new SafeSet([
   'javascript',
   'javascript:'
 ]);
-// protocols that never have a hostname.
+// Protocols that never have a hostname.
 const hostlessProtocol = new SafeSet([
   'javascript',
   'javascript:'
 ]);
-// protocols that always contain a // bit.
+// Protocols that always contain a // bit.
 const slashedProtocol = new SafeSet([
   'http',
   'http:',
@@ -367,7 +367,7 @@ Url.prototype.parse = function parse(url, parseQueryString, slashesDenoteHost) {
     if (this.hostname.length > hostnameMaxLen) {
       this.hostname = '';
     } else {
-      // hostnames are always lower case.
+      // Hostnames are always lower case.
       this.hostname = this.hostname.toLowerCase();
     }
 
@@ -396,8 +396,8 @@ Url.prototype.parse = function parse(url, parseQueryString, slashesDenoteHost) {
     }
   }
 
-  // now rest is set to the post-host stuff.
-  // chop off any delim chars.
+  // Now rest is set to the post-host stuff.
+  // Chop off any delim chars.
   if (!unsafeProtocol.has(lowerProto)) {
     // First, make 100% sure that any "autoEscape" chars get
     // escaped, even if encodeURIComponent doesn't think they
@@ -810,7 +810,7 @@ Url.prototype.resolveObject = function resolveObject(relative) {
     result.search = relative.search;
     result.query = relative.query;
     srcPath = relPath;
-    // fall through to the dot-handling below.
+    // Fall through to the dot-handling below.
   } else if (relPath.length) {
     // it's relative
     // throw away the existing file, and take the new path instead.
diff --git a/lib/zlib.js b/lib/zlib.js
index 3bc82e36a8..09c3419b4d 100644
--- a/lib/zlib.js
+++ b/lib/zlib.js
@@ -61,7 +61,7 @@ const {
   BROTLI_OPERATION_FINISH
 } = constants;
 
-// translation table for return codes.
+// Translation table for return codes.
 const codes = {
   Z_OK: constants.Z_OK,
   Z_STREAM_END: constants.Z_STREAM_END,
@@ -153,8 +153,8 @@ function zlibBufferSync(engine, buffer) {
 
 function zlibOnError(message, errno, code) {
   var self = this[owner_symbol];
-  // there is no way to cleanly recover.
-  // continuing only obscures problems.
+  // There is no way to cleanly recover.
+  // Continuing only obscures problems.
   _close(self);
   self._hadError = true;