test,util: remove lint workarounds

Remove assignments to `SIMD` that are only to pacify ESLint. Instead, use either `global.SIMD` or provide an comment letting ESLint know in cases where `SIMD` is guaranteed to be a defined global identifier. PR-URL: nodejs#10785 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Teddy Katz <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
italoacasas · Jan 27, 2017 · 84bf04b · 84bf04b
1 parent 77cbc26
commit 84bf04b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/lib/util.js b/lib/util.js
@@ -27,16 +27,13 @@ var simdFormatters;
 if (typeof global.SIMD === 'object' && global.SIMD !== null) {
   simdFormatters = new Map();
 
-  const make = (extractLane, count) => {
-    return (ctx, value, recurseTimes, visibleKeys, keys) => {
+  const make =
+    (extractLane, count) => (ctx, value, recurseTimes, visibleKeys, keys) => {
       const output = new Array(count);
       for (var i = 0; i < count; i += 1)
         output[i] = formatPrimitive(ctx, extractLane(value, i));
       return output;
     };
-  };
-
-  const SIMD = global.SIMD;  // Pacify eslint.
 
   const countPerType = {
     Bool16x8: 8,
@@ -52,7 +49,7 @@ if (typeof global.SIMD === 'object' && global.SIMD !== null) {
   };
 
   for (const key in countPerType) {
-    const type = SIMD[key];
+    const type = global.SIMD[key];
     simdFormatters.set(type, make(type.extractLane, countPerType[key]));
   }
 }

diff --git a/test/parallel/test-util-inspect-simd.js b/test/parallel/test-util-inspect-simd.js
@@ -1,12 +1,11 @@
 // Flags: --harmony_simd
+/* global SIMD */
 'use strict';
 
 require('../common');
 const assert = require('assert');
 const inspect = require('util').inspect;
 
-const SIMD = global.SIMD;  // Pacify eslint.
-
 assert.strictEqual(
     inspect(SIMD.Bool16x8()),
     'Bool16x8 [ false, false, false, false, false, false, false, false ]');