From bf2b5e1d5c239a1b4ea812051ec82101e1ce4347 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Thu, 9 Nov 2017 14:27:12 +0000
Subject: [PATCH] tools: remove unused trailing function arguments
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Update tools/doc/html.js and tools/eslint-rules/crypto-check.js to
remove unused trailing function arguments in preparation for enabling a
lint rule to enforce that practice.

PR-URL: https://github.com/nodejs/node/pull/16953
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
---
 tools/doc/html.js                     | 2 +-
 tools/eslint-rules/crypto-check.js    | 2 +-
 tools/eslint-rules/inspector-check.js | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/doc/html.js b/tools/doc/html.js
index 5534dd03b7441a..e25192599593f3 100644
--- a/tools/doc/html.js
+++ b/tools/doc/html.js
@@ -224,7 +224,7 @@ function altDocs(filename) {
   const host = 'https://nodejs.org';
   const href = (v) => `${host}/docs/latest-v${v.num}/api/${filename}.html`;
 
-  function li(v, i) {
+  function li(v) {
     let html = `<li><a href="${href(v)}">${v.num}`;
 
     if (v.lts)
diff --git a/tools/eslint-rules/crypto-check.js b/tools/eslint-rules/crypto-check.js
index b1b2a03f50e3b6..9d24d3355dce7f 100644
--- a/tools/eslint-rules/crypto-check.js
+++ b/tools/eslint-rules/crypto-check.js
@@ -54,7 +54,7 @@ module.exports = function(context) {
     }
   }
 
-  function reportIfMissingCheck(node) {
+  function reportIfMissingCheck() {
     if (hasSkipCall) {
       return;
     }
diff --git a/tools/eslint-rules/inspector-check.js b/tools/eslint-rules/inspector-check.js
index e65dd17d768ac2..bb40a98183250c 100644
--- a/tools/eslint-rules/inspector-check.js
+++ b/tools/eslint-rules/inspector-check.js
@@ -29,7 +29,7 @@ module.exports = function(context) {
     }
   }
 
-  function reportIfMissing(context, node) {
+  function reportIfMissing(context) {
     if (!hasInspectorCheck) {
       missingCheckNodes.forEach((node) => {
         context.report(node, msg);