tools: remove unused trailing function arguments

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: #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>

Author: Trott

tools/doc/html.js

@@ -224,7 +224,7 @@ function altDocs(filename) {
 consthost='https://nodejs.org';
 consthref=(v)=>`${host}/docs/latest-v${v.num}/api/${filename}.html`;
 
-functionli(v,i){
+functionli(v){
 lethtml=`<li><a href="${href(v)}">${v.num}`;
 
 if(v.lts)

tools/eslint-rules/crypto-check.js

@@ -54,7 +54,7 @@ module.exports = function(context) {
 }
 }
 
-functionreportIfMissingCheck(node){
+functionreportIfMissingCheck(){
 if(hasSkipCall){
 return;
 }

tools/eslint-rules/inspector-check.js

@@ -29,7 +29,7 @@ module.exports = function(context) {
 }
 }
 
-functionreportIfMissing(context,node){
+functionreportIfMissing(context){
 if(!hasInspectorCheck){
 missingCheckNodes.forEach((node)=>{
 context.report(node,msg);