lib: remove use of array destructuring

PR-URL: #36818
Reviewed-By: Rich Trott <rtrott@gmail.com>

Author: aduh95

lib/_http_server.js

@@ -415,7 +415,7 @@ Server.prototype.setTimeout = function setTimeout(msecs, callback) {
 Server.prototype[EE.captureRejectionSymbol] = function(err, event, ...args) {
   switch (event) {
     case 'request':
-      const [ , res] = args;
+      const { 1: res } = args;
       if (!res.headersSent && !res.writableEnded) {
         // Don't leak headers.
         ArrayPrototypeForEach(res.getHeaderNames(),

lib/assert.js

@@ -335,15 +335,15 @@ function getErrMessage(message, fn) {
       }
       fd = openSync(filename, 'r', 0o666);
       // Reset column and message.
-      [column, message] = getCode(fd, line, column);
+      ({ 0: column, 1: message } = getCode(fd, line, column));
       // Flush unfinished multi byte characters.
       decoder.end();
     } else {
       for (let i = 0; i < line; i++) {
         code = StringPrototypeSlice(code,
                                     StringPrototypeIndexOf(code, '\n') + 1);
       }
-      [column, message] = parseCode(code, column);
+      ({ 0: column, 1: message } = parseCode(code, column));
     }
     // Always normalize indentation, otherwise the message could look weird.
     if (StringPrototypeIncludes(message, '\n')) {

lib/events.js

@@ -315,7 +315,7 @@ function enhanceStackTrace(err, own) {
   const errStack = err.stack.split('\n').slice(1);
   const ownStack = own.stack.split('\n').slice(1);
 
-  const [ len, off ] = identicalSequenceRange(ownStack, errStack);
+  const { 0: len, 1: off } = identicalSequenceRange(ownStack, errStack);
   if (len > 0) {
     ownStack.splice(off + 1, len - 2,
                     '    [... lines matching original stack trace ...]');

lib/fs.js

@@ -2073,7 +2073,8 @@ function copyFileSync(src, dest, mode) {
 function lazyLoadStreams() {
   if (!ReadStream) {
     ({ ReadStream, WriteStream } = require('internal/fs/streams'));
-    [ FileReadStream, FileWriteStream ] = [ ReadStream, WriteStream ];
+    FileReadStream = ReadStream;
+    FileWriteStream = WriteStream;
   }
 }
 

lib/internal/bootstrap/loaders.js

@@ -198,7 +198,7 @@ class NativeModule {
   // To be called during pre-execution when --expose-internals is on.
   // Enables the user-land module loader to access internal modules.
   static exposeInternals() {
-    for (const [id, mod] of NativeModule.map) {
+    for (const { 0: id, 1: mod } of NativeModule.map) {
       // Do not expose this to user land even with --expose-internals.
       if (id !== loaderId) {
         mod.canBeRequiredByUsers = true;

lib/internal/cluster/master.js

@@ -23,7 +23,8 @@ const cluster = new EventEmitter();
 const intercom = new EventEmitter();
 const SCHED_NONE = 1;
 const SCHED_RR = 2;
-const [ minPort, maxPort ] = [ 1024, 65535 ];
+const minPort = 1024;
+const maxPort = 65535;
 const { validatePort } = require('internal/validators');
 
 module.exports = cluster;

lib/internal/cluster/round_robin_handle.js

@@ -93,10 +93,11 @@ RoundRobinHandle.prototype.remove = function(worker) {
 
 RoundRobinHandle.prototype.distribute = function(err, handle) {
   ArrayPrototypePush(this.handles, handle);
-  const [ workerEntry ] = this.free;
+  // eslint-disable-next-line node-core/no-array-destructuring
+  const [ workerEntry ] = this.free; // this.free is a SafeMap
 
   if (ArrayIsArray(workerEntry)) {
-    const [ workerId, worker ] = workerEntry;
+    const { 0: workerId, 1: worker } = workerEntry;
     this.free.delete(workerId);
     this.handoff(worker);
   }

lib/internal/console/constructor.js

@@ -525,7 +525,7 @@ const consoleMethods = {
           length++;
         }
       } else {
-        for (const [k, v] of tabularData) {
+        for (const { 0: k, 1: v } of tabularData) {
           ArrayPrototypePush(keys, _inspect(k));
           ArrayPrototypePush(values, _inspect(v));
           length++;

lib/internal/crypto/keys.js

@@ -86,12 +86,12 @@ for (const m of [[kKeyEncodingPKCS1, 'pkcs1'], [kKeyEncodingPKCS8, 'pkcs8'],
 // which requires the KeyObject base class to be implemented in C++.
 // The creation requires a callback to make sure that the NativeKeyObject
 // base class cannot exist without the other KeyObject implementations.
-const [
-  KeyObject,
-  SecretKeyObject,
-  PublicKeyObject,
-  PrivateKeyObject,
-] = createNativeKeyObjectClass((NativeKeyObject) => {
+const {
+  0: KeyObject,
+  1: SecretKeyObject,
+  2: PublicKeyObject,
+  3: PrivateKeyObject,
+} = createNativeKeyObjectClass((NativeKeyObject) => {
   // Publicly visible KeyObject class.
   class KeyObject extends NativeKeyObject {
     constructor(type, handle) {

lib/internal/errors.js

@@ -429,7 +429,7 @@ const captureLargerStackTrace = hideStackFrames(
  * @returns {Error}
  */
 const uvException = hideStackFrames(function uvException(ctx) {
-  const [code, uvmsg] = uvErrmapGet(ctx.errno) || uvUnmappedError;
+  const { 0: code, 1: uvmsg } = uvErrmapGet(ctx.errno) || uvUnmappedError;
   let message = `${code}: ${ctx.message || uvmsg}, ${ctx.syscall}`;
 
   let path;
@@ -485,7 +485,7 @@ const uvException = hideStackFrames(function uvException(ctx) {
  */
 const uvExceptionWithHostPort = hideStackFrames(
   function uvExceptionWithHostPort(err, syscall, address, port) {
-    const [code, uvmsg] = uvErrmapGet(err) || uvUnmappedError;
+    const { 0: code, 1: uvmsg } = uvErrmapGet(err) || uvUnmappedError;
     const message = `${syscall} ${code}: ${uvmsg}`;
     let details = '';
 
@@ -1243,7 +1243,8 @@ E('ERR_MANIFEST_ASSERT_INTEGRITY',
     }" does not match the expected integrity.`;
     if (realIntegrities.size) {
       const sri = ArrayPrototypeJoin(
-        ArrayFrom(realIntegrities.entries(), ([alg, dgs]) => `${alg}-${dgs}`),
+        ArrayFrom(realIntegrities.entries(),
+                  ({ 0: alg, 1: dgs }) => `${alg}-${dgs}`),
         ' '
       );
       msg += ` Integrities found are: ${sri}`;