tools: enable eslint no-undef-init rule

This also fixes the three entries that did not pass. PR-URL: nodejs#18831 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Matheus Marchini <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
AdamMajer · Feb 22, 2018 · ff5b56e · ff5b56e
1 parent a926c1a
commit ff5b56e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -130,6 +130,7 @@ module.exports = {
     // http://eslint.org/docs/rules/#variables
     'no-delete-var': 'error',
     'no-undef': 'error',
+    'no-undef-init': 'error',
     'no-unused-vars': ['error', { args: 'none' }],
     'no-use-before-define': ['error', {
       classes: true,

diff --git a/lib/internal/util.js b/lib/internal/util.js
@@ -228,8 +228,8 @@ function getSystemErrorName(err) {
 // getConstructorOf is wrapped into this to save iterations
 function getIdentificationOf(obj) {
   const original = obj;
-  let constructor = undefined;
-  let tag = undefined;
+  let constructor;
+  let tag;
 
   while (obj) {
     if (constructor === undefined) {

diff --git a/test/parallel/test-fs-utimes.js b/test/parallel/test-fs-utimes.js
@@ -88,7 +88,7 @@ function testIt(atime, mtime, callback) {
       expect_errno('futimesSync', fd, ex, 'ENOSYS');
     }
 
-    let err = undefined;
+    let err;
     try {
       fs.utimesSync('foobarbaz', atime, mtime);
     } catch (ex) {