@jslint-org/jslint 2021.11.20 → 2022.3.30

package/jslint.mjs

@@ -94,6 +94,8 @@
 /*jslint beta, node*/
 
 /*property
+    fud_stmt,
+    is_fart,
     mode_conditional,
     JSLINT_BETA, NODE_V8_COVERAGE, a, all, argv, arity, artifact,
     assertErrorThrownAsync, assertJsonEqual, assertOrThrow, assign, async, b,
@@ -107,7 +109,7 @@
     example_list, exec, execArgv, exit, export_dict, exports, expression, extra,
     file, fileList, fileURLToPath, filter, finally, flag, floor, for, forEach,
     formatted_message, free, freeze, from, froms,
-    fsWriteFileWithParents, fud, functionName, function_list, function_stack,
+    fsWriteFileWithParents, functionName, function_list, function_stack,
     functions, get, getset, github_repo, global, global_dict, global_list,
     holeList, htmlEscape, id, identifier, import, import_list, inc, indent2,
     index, indexOf, init, initial, isArray, isBlockCoverage, isHole, isNaN,
@@ -115,13 +117,13 @@
     jslint_charset_ascii, jslint_cli, jslint_edition, jslint_phase1_split,
     jslint_phase2_lex, jslint_phase3_parse, jslint_phase4_walk,
     jslint_phase5_whitage, jslint_report, json, jstestDescribe, jstestIt,
-    jstestOnExit, keys, label, lbp, led, length, level, line, lineList,
+    jstestOnExit, keys, label, lbp, led_infix, length, level, line, lineList,
     line_list, line_offset, line_source, lines, linesCovered, linesTotal, live,
     log, long, loop, m, main, map, margin, match, max, message, meta, min,
     mkdir, modeCoverageIgnoreFile, modeIndex, mode_cli, mode_json, mode_module,
     mode_noop, mode_property, mode_shebang, mode_stop, module, moduleFsInit,
     moduleName, module_list, name, names, node, noop, now,
-    nr, nud, objectDeepCopyWithKeysSorted, ok, on, open, opening, option,
+    nr, nud_prefix, objectDeepCopyWithKeysSorted, ok, on, open, opening, option,
     option_dict, order, package_name, padEnd, padStart, parameters, parent,
     parentIi, parse, pathname, platform, pop, processArgv, process_argv,
     process_env, process_exit, process_version, promises, property,
@@ -165,11 +167,96 @@ let jslint_charset_ascii = (
     + "@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\\]^_"
     + "`abcdefghijklmnopqrstuvwxyz{|}~\u007f"
 );
-let jslint_edition = "v2021.11.20";
+let jslint_edition = "v2022.3.30";
 let jslint_export;                      // The jslint object to be exported.
 let jslint_fudge = 1;                   // Fudge starting line and starting
                                         // ... column to 1.
 let jslint_import_meta_url = "";        // import.meta.url used by cli.
+let jslint_rgx_cap = (
+    /^[A-Z]/
+);
+let jslint_rgx_crlf = (
+    /\n|\r\n?/
+);
+let jslint_rgx_digits_bits = (
+    /^[01_]*/
+);
+let jslint_rgx_digits_decimals = (
+    /^[0-9_]*/
+);
+let jslint_rgx_digits_hexs = (
+    /^[0-9A-F_]*/i
+);
+let jslint_rgx_digits_octals = (
+    /^[0-7_]*/
+);
+let jslint_rgx_directive = (
+    /^(jslint|property|global)\s+(.*)$/
+);
+let jslint_rgx_directive_part = (
+    /([a-zA-Z$_][a-zA-Z0-9$_]*)(?::\s*(true|false))?,?\s*|$/g
+);
+let jslint_rgx_identifier = (
+    /^([a-zA-Z_$][a-zA-Z0-9_$]*)$/
+);
+let jslint_rgx_json_number = (
+
+// https://datatracker.ietf.org/doc/html/rfc7159#section-6
+// number = [ minus ] int [ frac ] [ exp ]
+
+    /^-?(?:0|[1-9]\d*)(?:\.\d*)?(?:[eE][\-+]?\d+)?$/
+);
+let jslint_rgx_mega = (
+
+// Vim-hack - vim-editor has trouble parsing naked '`' in regexp
+
+    /[\u0060\\]|\$\{/
+);
+let jslint_rgx_module = (
+    /^[a-zA-Z0-9_$:.@\-\/]+$/
+);
+let jslint_rgx_numeric_separator_illegal = (
+    /__|_$|_n$/m
+);
+let jslint_rgx_slash_star_or_slash = (
+    /\/\*|\/$/
+);
+let jslint_rgx_tab = (
+    /\t/g
+);
+let jslint_rgx_todo = (
+    /\b(?:todo|TO\s?DO|HACK)\b/
+);
+let jslint_rgx_token = new RegExp(
+    "^("
+    + "(\\s+)"
+    + "|([a-zA-Z_$][a-zA-Z0-9_$]*)"
+    + "|[(){}\\[\\],:;'\"~\\`]"
+    + "|\\?[?.]?"
+    + "|=(?:==?|>)?"
+    + "|\\.+"
+    + "|\\*[*\\/=]?"
+    + "|\\/[*\\/]?"
+    + "|\\+[=+]?"
+    + "|-[=\\-]?"
+    + "|[\\^%]=?"
+    + "|&[&=]?"
+    + "|\\"
+    + "|[|=]?"
+    + "|>{1,3}=?"
+    + "|<<?=?"
+    + "|!(?:!|==?)?"
+
+// PR-351 - Add BigInt support.
+// PR-390 - Add numeric-separator support.
+
+    + "|((?:0_?|[1-9][0-9_]*)n?)"
+    + ")"
+    + "(.*)$"
+);
+let jslint_rgx_weird_property = (
+    /^_|\$|Sync$|_$/m
+);
 let jstestCountFailed = 0;
 let jstestCountTotal = 0;
 let jstestItCount = 0;
@@ -202,13 +289,16 @@ function assertJsonEqual(aa, bb, message) {
 
 // This function will assert JSON.stringify(<aa>) === JSON.stringify(<bb>).
 
-    aa = JSON.stringify(objectDeepCopyWithKeysSorted(aa));
-    bb = JSON.stringify(objectDeepCopyWithKeysSorted(bb));
+    aa = JSON.stringify(objectDeepCopyWithKeysSorted(aa), undefined, 1);
+    bb = JSON.stringify(objectDeepCopyWithKeysSorted(bb), undefined, 1);
     if (aa !== bb) {
         throw new Error(
-            JSON.stringify(aa) + " !== " + JSON.stringify(bb) + (
-                message
+            "\n" + aa + "\n!==\n" + bb
+            + (
+                typeof message === "string"
                 ? " - " + message
+                : message
+                ? " - " + JSON.stringify(message)
                 : ""
             )
         );
@@ -302,10 +392,7 @@ function jslint(
     let import_list = [];       // The array collecting all import-from strings.
     let line_list = String(     // The array containing source lines.
         "\n" + source
-    ).split(
-        // rx_crlf
-        /\n|\r\n?/
-    ).map(function (line_source) {
+    ).split(jslint_rgx_crlf).map(function (line_source) {
         return {
             line_source
         };
@@ -676,8 +763,17 @@ function jslint(
                 `Expected 'Object.freeze('. All export values should be frozen.`
             );
             break;
-        case "function_in_loop":
-            mm = `Don't create functions within a loop.`;
+
+// PR-378 - Relax warning "function_in_loop".
+//
+//         case "function_in_loop":
+//             mm = `Don't create functions within a loop.`;
+//             break;
+
+// PR-390 - Add numeric-separator check.
+
+        case "illegal_num_separator":
+            mm = `Illegal numeric separator '_' at column ${column}.`;
             break;
         case "infix_in":
             mm = (
@@ -831,6 +927,15 @@ function jslint(
         case "use_double":
             mm = `Use double quotes, not single quotes.`;
             break;
+
+// PR-386 - Fix issue #382 - Make fart-related warnings more readable.
+
+        case "use_function_not_fart":
+            mm = (
+                `Use 'function (...)', not '(...) =>' when arrow functions`
+                + ` become too complex.`
+            );
+            break;
         case "use_open":
             mm = (
                 `Wrap a ternary expression in parens,`
@@ -864,6 +969,12 @@ function jslint(
         case "wrap_condition":
             mm = `Wrap the condition in parens.`;
             break;
+
+// PR-386 - Fix issue #382 - Make fart-related warnings more readable.
+
+        case "wrap_fart_parameter":
+            mm = `Wrap the parameter before '=>' in parens.`;
+            break;
         case "wrap_immediate":
             mm = (
                 `Wrap an immediate function invocation in parentheses to assist`
@@ -871,9 +982,6 @@ function jslint(
                 + ` result of a function, and not the function itself.`
             );
             break;
-        case "wrap_parameter":
-            mm = `Wrap the parameter in parens.`;
-            break;
         case "wrap_regexp":
             mm = `Wrap this regexp in parens to avoid confusion.`;
             break;
@@ -1449,8 +1557,8 @@ async function jslint_cli({
     let command;
     let data;
     let exit_code = 0;
-    let mode_plugin_vim;
     let mode_report;
+    let mode_wrapper_vim;
     let result;
 
     function jslint_from_file({
@@ -1523,7 +1631,7 @@ async function jslint_cli({
         if (result_from_file.warnings.length > 0) {
             exit_code = 1;
             console_error(
-                mode_plugin_vim
+                mode_wrapper_vim
 
 // PR-349 - Print warnings in format readable by vim.
 
@@ -1656,17 +1764,17 @@ async function jslint_cli({
         }));
         return;
 
-// COMMIT-b26d6df2 - Add command jslint_plugin_vim.
+// PR-363 - Add command jslint_report.
 
-    case "jslint_plugin_vim":
-        mode_plugin_vim = true;
+    case "jslint_report":
+        mode_report = command[1];
         process_argv = process_argv.slice(1);
         break;
 
-// PR-363 - Add command jslint_report.
+// COMMIT-b26d6df2 - Add command jslint_wrapper_vim.
 
-    case "jslint_report":
-        mode_report = command[1];
+    case "jslint_wrapper_vim":
+        mode_wrapper_vim = true;
         process_argv = process_argv.slice(1);
         break;
 
@@ -1674,6 +1782,7 @@ async function jslint_cli({
 
     case "v8_coverage_report":
         await v8CoverageReportCreate({
+            consoleError: console_error,
             coverageDir: command[1],
             processArgv: process_argv.slice(3)
         });
@@ -1682,9 +1791,9 @@ async function jslint_cli({
 
 // PR-349 - Detect cli-option --mode-vim-plugin.
 
-    mode_plugin_vim = (
+    mode_wrapper_vim = (
         process_argv.slice(2).indexOf("--mode-vim-plugin") >= 0
-        || mode_plugin_vim
+        || mode_wrapper_vim
     );
 
 // Normalize file relative to process.cwd().
@@ -1820,37 +1929,16 @@ function jslint_phase2_lex(state) {
     let line_mega;              // The starting line of megastring.
     let line_source = "";       // The remaining line source string.
     let line_whole = "";        // The whole line source string.
+    let mode_digits_empty_string = 1;
+    let mode_digits_numeric_separator = 2;
     let mode_directive = true;  // true if directives are still allowed.
     let mode_mega = false;      // true if currently parsing a megastring
                                 // ... literal.
     let mode_regexp;            // true if regular expression literal seen on
                                 // ... this line.
-    let rx_token = new RegExp(
-        "^("
-        + "(\\s+)"
-        + "|([a-zA-Z_$][a-zA-Z0-9_$]*)"
-        + "|[(){}\\[\\],:;'\"~\\`]"
-        + "|\\?[?.]?"
-        + "|=(?:==?|>)?"
-        + "|\\.+"
-        + "|\\*[*\\/=]?"
-        + "|\\/[*\\/]?"
-        + "|\\+[=+]?"
-        + "|-[=\\-]?"
-        + "|[\\^%]=?"
-        + "|&[&=]?"
-        + "|\\"
-        + "|[|=]?"
-        + "|>{1,3}=?"
-        + "|<<?=?"
-        + "|!(?:!|==?)?"
-
-// PR-351 - Add BigInt support.
-
-        + "|(0n?|[1-9][0-9]*n?)"
-        + ")"
-        + "(.*)$"
-    );
+    let paren_backtrack_list = [];      // List of most recent "(" tokens at any
+                                        // ... paren-depth.
+    let paren_depth = 0;        // Keeps track of current paren-depth.
     let snippet = "";           // A piece of string.
     let token_1;                // The first token.
     let token_prv = token_global;       // The previous token including
@@ -1932,7 +2020,7 @@ function jslint_phase2_lex(state) {
 
                     warn_at("unexpected_a", line, column, char);
                 }
-                if (read_digits("x") > 5) {
+                if (read_digits("x", undefined) > 5) {
 
 // test_cause:
 // ["\"\\u{123456}\"", "char_after_escape", "too_many_digits", "", 11]
@@ -1949,7 +2037,7 @@ function jslint_phase2_lex(state) {
                 return char_after();
             }
             char_before();
-            if (read_digits("x", true) < 4) {
+            if (read_digits("x", mode_digits_empty_string) < 4) {
 
 // test_cause:
 // ["\"\\u0\"", "char_after_escape", "expected_four_digits", "", 5]
@@ -1984,6 +2072,26 @@ function jslint_phase2_lex(state) {
         return char;
     }
 
+    function check_numeric_separator(digits, column) {
+
+// This function will check for illegal numeric-separator in <digits>.
+
+        digits.replace((
+            jslint_rgx_numeric_separator_illegal
+        ), function (ignore, ii) {
+
+// test_cause:
+// ["0x0_0_;", "check_numeric_separator", "illegal_num_separator", "", 6]
+// ["0x0_0__0;", "check_numeric_separator", "illegal_num_separator", "", 6]
+// ["aa=1_2_;", "check_numeric_separator", "illegal_num_separator", "", 7]
+// ["aa=1_2__3;", "check_numeric_separator", "illegal_num_separator", "", 7]
+// ["aa=1_2_n;", "check_numeric_separator", "illegal_num_separator", "", 7]
+
+            warn_at("illegal_num_separator", line, column + ii + 1);
+            return "";
+        });
+    }
+
     function lex_comment() {
         let body;
         let ii = 0;
@@ -2022,12 +2130,12 @@ function jslint_phase2_lex(state) {
 // Lex/loop through each line until "*/".
 
             while (true) {
-                // rx_star_slash
+                // jslint_rgx_star_slash
                 ii = line_source.indexOf("*/");
                 if (ii >= 0) {
                     break;
                 }
-                // rx_slash_star
+                // jslint_rgx_slash_star
                 ii = line_source.indexOf("/*");
                 if (ii >= 0) {
 
@@ -2047,8 +2155,7 @@ function jslint_phase2_lex(state) {
                 }
             }
             jj = line_source.slice(0, ii).search(
-                // rx_slash_star_or_slash
-                /\/\*|\/$/
+                jslint_rgx_slash_star_or_slash
             );
             if (jj >= 0) {
 
@@ -2066,13 +2173,7 @@ function jslint_phase2_lex(state) {
 
 // Uncompleted work comment.
 
-        if (
-            !option_dict.devel
-            && (
-                // rx_todo
-                /\b(?:todo|TO\s?DO|HACK)\b/
-            ).test(snippet)
-        ) {
+        if (!option_dict.devel && jslint_rgx_todo.test(snippet)) {
 
 // test_cause:
 // ["//todo", "lex_comment", "todo_comment", "(comment)", 1] //jslint-quiet
@@ -2084,10 +2185,7 @@ function jslint_phase2_lex(state) {
 
         [
             the_comment.directive, body
-        ] = Array.from(snippet.match(
-            // rx_directive
-            /^(jslint|property|global)\s+(.*)$/
-        ) || []).slice(1);
+        ] = Array.from(snippet.match(jslint_rgx_directive) || []).slice(1);
         if (the_comment.directive === undefined) {
             return the_comment;
         }
@@ -2109,10 +2207,12 @@ function jslint_phase2_lex(state) {
 // Lex/loop through each directive in /*...*/
 
         ii = 0;
-        body.replace((
-            // rx_directive_part
-            /([a-zA-Z$_][a-zA-Z0-9$_]*)(?::\s*(true|false))?,?\s*|$/g
-        ), function (match0, key, val, jj) {
+        body.replace(jslint_rgx_directive_part, function (
+            match0,
+            key,
+            val,
+            jj
+        ) {
             if (ii !== jj) {
 
 // test_cause:
@@ -2186,13 +2286,7 @@ function jslint_phase2_lex(state) {
 // string.
 
         while (true) {
-            match = line_source.match(
-
-// Vim-hack - vim-editor has trouble parsing '`' in regexp
-
-                // rx_mega
-                /[\u0060\\]|\$\{/
-            ) || {
+            match = line_source.match(jslint_rgx_mega) || {
                 "0": "",
                 index: 0
             };
@@ -2269,12 +2363,16 @@ function jslint_phase2_lex(state) {
 
     function lex_number() {
         let prefix = snippet;
+
+// PR-390 - Add numeric-separator check.
+
+        check_numeric_separator(prefix, column - prefix.length);
         char_after();
         switch (prefix === "0" && char) {
         case "b":
         case "o":
         case "x":
-            read_digits(char);
+            read_digits(char, mode_digits_numeric_separator);
 
 // PR-351 - Ignore BigInt suffix 'n'.
 
@@ -2284,14 +2382,14 @@ function jslint_phase2_lex(state) {
             break;
         default:
             if (char === ".") {
-                read_digits("d");
+                read_digits("d", mode_digits_numeric_separator);
             }
             if (char === "E" || char === "e") {
                 char_after(char);
                 if (char !== "+" && char !== "-") {
                     char_before();
                 }
-                read_digits("d");
+                read_digits("d", mode_digits_numeric_separator);
             }
         }
 
@@ -2561,7 +2659,7 @@ function jslint_phase2_lex(state) {
                     }
                     break;
                 case "{":
-                    if (read_digits("d", true) === 0) {
+                    if (read_digits("d", mode_digits_empty_string) === 0) {
 
 // test_cause:
 // ["aa=/aa{/", "lex_regexp_group", "expected_a_before_b", ",", 8]
@@ -2574,7 +2672,7 @@ function jslint_phase2_lex(state) {
 // ["aa=/.{,/", "lex_regexp_group", "comma", "", 0]
 
                         test_cause("comma");
-                        read_digits("d", true);
+                        read_digits("d", mode_digits_empty_string);
                     }
                     if (char_after("}") === "?") {
 
@@ -2867,7 +2965,7 @@ function jslint_phase2_lex(state) {
                 }
             }
             from = column;
-            match = line_source.match(rx_token);
+            match = line_source.match(jslint_rgx_token);
 
 // match[1] token
 // match[2] whitespace
@@ -3028,6 +3126,7 @@ console.log(JSON.stringify(Object.keys(window).sort(), undefined, 4));
 
 // Window.
 
+                "Blob",
                 // "CharacterData",
                 // "DocumentType",
                 // "Element",
@@ -3186,7 +3285,7 @@ node --input-type=module --eval '
 // /\*jslint beta, node*\/
 import moduleHttps from "https";
 (async function () {
-    let dict = {};
+    let dict = Object.create(null);
     let result = "";
     await new Promise(function (resolve) {
         moduleHttps.get((
@@ -3249,41 +3348,48 @@ import moduleHttps from "https";
         return true;
     }
 
-    function read_digits(base, quiet) {
+    function read_digits(base, mode) {
         let digits = line_source.match(
             base === "b"
-            ? (
-                // rx_bits
-                /^[01]*/
-            )
+            ? jslint_rgx_digits_bits
             : base === "o"
-            ? (
-                // rx_octals
-                /^[0-7]*/
-            )
+            ? jslint_rgx_digits_octals
             : base === "x"
-            ? (
-                // rx_hexs
-                /^[0-9A-F]*/i
-            )
-            : (
-                // rx_digits
-                /^[0-9]*/
-            )
+            ? jslint_rgx_digits_hexs
+            : jslint_rgx_digits_decimals
         )[0];
-        let length = digits.length;
-        if (!quiet && length === 0) {
+        if (
+            (mode !== mode_digits_empty_string && digits.length === 0)
+            || digits[0] === "_"
+        ) {
 
 // test_cause:
 // ["0x", "read_digits", "expected_digits_after_a", "0x", 2]
+// ["0x_", "read_digits", "expected_digits_after_a", "0x", 2]
 
             warn_at("expected_digits_after_a", line, column, snippet);
         }
-        column += length;
-        line_source = line_source.slice(length);
+
+// PR-390 - Add numeric-separator check.
+
+        if (mode === mode_digits_numeric_separator) {
+            check_numeric_separator(digits, column);
+        } else if (digits.indexOf("_") >= 0) {
+
+// test_cause:
+// ["\"\\u{1_2}\"", "read_digits", "illegal_num_separator", "", 6]
+
+            warn_at(
+                "illegal_num_separator",
+                line,
+                column + digits.indexOf("_") + 1
+            );
+        }
+        column += digits.length;
+        line_source = line_source.slice(digits.length);
         snippet += digits;
         char_after();
-        return length;
+        return digits.length;
     }
 
     function read_line() {
@@ -3354,6 +3460,7 @@ import moduleHttps from "https";
             test_cause("line_disable");
             line_source = "";
         }
+        // jslint_rgx_tab
         if (line_source.indexOf("\t") >= 0) {
             if (!option_dict.white) {
 
@@ -3362,10 +3469,7 @@ import moduleHttps from "https";
 
                 warn_at("use_spaces", line, line_source.indexOf("\t") + 1);
             }
-            line_source = line_source.replace((
-                // rx_tab
-                /\t/g
-            ), " ");
+            line_source = line_source.replace(jslint_rgx_tab, " ");
         }
         if (!option_dict.white && line_source.endsWith(" ")) {
 
@@ -3385,9 +3489,11 @@ import moduleHttps from "https";
             from,
             id,
             identifier: Boolean(identifier),
+            is_fart: false,
             line,
             nr: token_list.length,
-            thru: column
+            thru: column,
+            value
         };
         token_list.push(the_token);
 
@@ -3397,12 +3503,6 @@ import moduleHttps from "https";
             mode_directive = false;
         }
 
-// If the token is to have a value, give it one.
-
-        if (value !== undefined) {
-            the_token.value = value;
-        }
-
 // If this token is an identifier that touches a preceding number, or
 // a "/", comment, or regular expression literal that touches a preceding
 // comment or regular expression literal, then give a missing space warning.
@@ -3436,6 +3536,29 @@ import moduleHttps from "https";
             the_token.dot = true;
         }
 
+// PR-385 - Bugfix - Fixes issue #382 - failure to detect destructured fart.
+// Farts are now detected by keeping a list of most recent "(" tokens at any
+// given depth. When a "=>" token is encountered, the most recent "(" token at
+// current depth is marked as a fart.
+
+        switch (id) {
+        case "(":
+            paren_backtrack_list[paren_depth] = the_token;
+            paren_depth += 1;
+            break;
+        case ")":
+            paren_depth -= 1;
+            break;
+        case "=>":
+            if (
+                token_prv_expr.id === ")"
+                && paren_backtrack_list[paren_depth]
+            ) {
+                paren_backtrack_list[paren_depth].is_fart = true;
+            }
+            break;
+        }
+
 // The previous token is used to detect adjacency problems.
 
         token_prv = the_token;
@@ -3520,9 +3643,6 @@ function jslint_phase3_parse(state) {
     let catchage = catch_stack[0];      // The current catch-block.
     let functionage = token_global;     // The current function.
     let mode_var;               // "var" if using var; "let" if using let.
-    let rx_identifier = (
-        /^([a-zA-Z_$][a-zA-Z0-9_$]*)$/
-    );
     let token_ii = 0;           // The number of the next token.
     let token_now = token_global;       // The current token being examined in
                                         // ... the parse.
@@ -3539,7 +3659,7 @@ function jslint_phase3_parse(state) {
             anon = token_now.id;
         } else if (
             token_now.id === "(string)"
-            && rx_identifier.test(token_now.value)
+            && jslint_rgx_identifier.test(token_now.value)
         ) {
             anon = token_now.value;
         }
@@ -3551,7 +3671,7 @@ function jslint_phase3_parse(state) {
                 match === undefined
 
 // test_cause:
-// ["()", "advance", "expected_a_b", "(end)", 1]
+// ["{0:0}", "advance", "expected_a_b", "0", 2]
 
                 ? stop("expected_a_b", token_nxt, id, artifact())
 
@@ -3600,7 +3720,7 @@ function jslint_phase3_parse(state) {
 // other assignment operators can modify, but they cannot initialize.
 
         const the_symbol = symbol(id, 20);
-        the_symbol.led = function (left) {
+        the_symbol.led_infix = function (left) {
             const the_token = token_now;
             let right;
             the_token.arity = "assignment";
@@ -3912,7 +4032,7 @@ function jslint_phase3_parse(state) {
 
         const the_symbol = symbol(id);
         the_symbol.constant = true;
-        the_symbol.nud = (
+        the_symbol.nud_prefix = (
             typeof value === "function"
             ? value
             : function () {
@@ -4106,7 +4226,7 @@ function jslint_phase3_parse(state) {
 // Create an infix operator.
 
         const the_symbol = symbol(id, bp);
-        the_symbol.led = function (left) {
+        the_symbol.led_infix = function (left) {
             const the_token = token_now;
             the_token.arity = "binary";
             if (f !== undefined) {
@@ -4164,12 +4284,12 @@ function jslint_phase3_parse(state) {
         return the_token;
     }
 
-    function infix_fart_unwrapped(left) {
+    function infix_fart_unwrapped() {
 
 // test_cause:
-// ["aa=>0", "infix_fart_unwrapped", "wrap_parameter", "aa", 1]
+// ["aa=>0", "infix_fart_unwrapped", "wrap_fart_parameter", "=>", 3]
 
-        return stop("wrap_parameter", left);
+        return stop("wrap_fart_parameter", token_now);
     }
 
     function infix_grave(left) {
@@ -4189,7 +4309,7 @@ function jslint_phase3_parse(state) {
         let the_subscript = parse_expression(0);
         if (the_subscript.id === "(string)" || the_subscript.id === "`") {
             name = survey(the_subscript);
-            if (rx_identifier.test(name)) {
+            if (jslint_rgx_identifier.test(name)) {
 
 // test_cause:
 // ["aa[`aa`]", "infix_lbracket", "subscript_a", "aa", 4]
@@ -4332,13 +4452,13 @@ function jslint_phase3_parse(state) {
 // Create a right associative infix operator.
 
         const the_symbol = symbol(id, bp);
-        the_symbol.led = function parse_infixr_led(left) {
+        the_symbol.led_infix = function parse_infixr_led(left) {
             const the_token = token_now;
 
 // test_cause:
-// ["0**0", "parse_infixr_led", "led", "", 0]
+// ["0**0", "parse_infixr_led", "led_infix", "", 0]
 
-            test_cause("led");
+            test_cause("led_infix");
             the_token.arity = "binary";
             the_token.expression = [left, parse_expression(bp - 1)];
             return the_token;
@@ -4346,39 +4466,70 @@ function jslint_phase3_parse(state) {
         return the_symbol;
     }
 
-    function lookahead() {
-
-// Look ahead one token without advancing, skipping comments.
-
-        let cadet;
-        let ii = token_ii;
-        while (true) {
-            cadet = token_list[ii];
-            if (cadet.id !== "(comment)") {
-                return cadet;
-            }
-            ii += 1;
-        }
-    }
-
     function parse_expression(rbp, initial) {
 
 // This is the heart of JSLINT, the Pratt parser. In addition to parsing, it
-// is looking for ad hoc lint patterns. We add .fud to Pratt's model, which is
-// like .nud except that it is only used on the first token of a statement.
-// Having .fud makes it much easier to define statement-oriented languages like
-// JavaScript. I retained Pratt's nomenclature.
+// is looking for ad hoc lint patterns. We add .fud_stmt to Pratt's model, which
+// is like .nud_prefix except that it is only used on the first token of a
+// statement. Having .fud_stmt makes it much easier to define statement-oriented
+// languages like JavaScript. I retained Pratt's nomenclature.
 // They are elements of the parsing method called Top Down Operator Precedence.
 
-// .nud     Null denotation
-// .fud     First null denotation
-// .led     Left denotation
-//  lbp     Left binding power
-//  rbp     Right binding power
-
-// It processes a nud (variable, constant, prefix operator). It will then
-// process leds (infix operators) until the bind powers cause it to stop. It
-// returns the expression's parse tree.
+// .nud_prefix  Null denotation. The prefix handler.
+// .fud_stmt    First null denotation. The statement handler.
+// .led_infix   Left denotation. The infix/postfix handler.
+//  lbp         Left binding power of infix operator. It tells us how strongly
+//              the operator binds to the argument at its left.
+//  rbp         Right binding power.
+
+// It processes a nud_prefix (variable, constant, prefix operator). It will then
+// process leds (infix operators) until the bind powers cause it to stop (it
+// consumes tokens until it meets a token whose lbp <= rbp). Specifically, it
+// means that it collects all tokens that bind together before returning to the
+// operator that called it. It returns the expression's parse tree.
+
+// For example, "3 + 1 * 2 * 4 + 5"
+// parses into
+// {
+//     "id": "+",
+//     "expression": [
+//         {
+//             "id": "+",
+//             "expression": [
+//                 {
+//                     "id": "(number)",
+//                     "value": "3"
+//                 },
+//                 {
+//                     "id": "*",
+//                     "expression": [
+//                         {
+//                             "id": "*",
+//                             "expression": [
+//                                 {
+//                                     "id": "(number)",
+//                                     "value": "1"
+//                                 },
+//                                 {
+//                                     "id": "(number)",
+//                                     "value": "2"
+//                                 }
+//                             ]
+//                         },
+//                         {
+//                             "id": "(number)",
+//                             "value": "4"
+//                         }
+//                     ]
+//                 }
+//             ]
+//         },
+//         {
+//             "id": "(number)",
+//             "value": "5"
+//         }
+//     ]
+// }
 
         let left;
         let the_symbol;
@@ -4390,13 +4541,13 @@ function jslint_phase3_parse(state) {
             advance();
         }
         the_symbol = syntax_dict[token_now.id];
-        if (the_symbol !== undefined && the_symbol.nud !== undefined) {
+        if (the_symbol !== undefined && the_symbol.nud_prefix !== undefined) {
 
 // test_cause:
 // ["0", "parse_expression", "symbol", "", 0]
 
             test_cause("symbol");
-            left = the_symbol.nud();
+            left = the_symbol.nud_prefix();
         } else if (token_now.identifier) {
 
 // test_cause:
@@ -4421,32 +4572,38 @@ function jslint_phase3_parse(state) {
             the_symbol = syntax_dict[token_nxt.id];
             if (
                 the_symbol === undefined
-                || the_symbol.led === undefined
+                || the_symbol.led_infix === undefined
                 || the_symbol.lbp <= rbp
             ) {
                 break;
             }
             advance();
-            left = the_symbol.led(left);
+            left = the_symbol.led_infix(left);
         }
         return left;
     }
 
-    function parse_fart(pl) {
+    function parse_fart() {
+        let parameters;
+        let signature;
         let the_fart;
+        [parameters, signature] = prefix_function_arg();
         advance("=>");
         the_fart = token_now;
         the_fart.arity = "binary";
         the_fart.name = "=>";
         the_fart.level = functionage.level + 1;
         function_list.push(the_fart);
-        if (functionage.loop > 0) {
 
-// test_cause:
-// ["while(0){aa.map(()=>0);}", "parse_fart", "function_in_loop", "=>", 19]
+// PR-384 - Relax warning "function_in_loop".
+//
+//         if (functionage.loop > 0) {
 
-            warn("function_in_loop", the_fart);
-        }
+// // test_cause:
+// // ["while(0){aa.map(()=>0);}", "parse_fart", "function_in_loop", "=>", 19]
+//
+//             warn("function_in_loop", the_fart);
+//         }
 
 // Give the function properties storing its names and for observing the depth
 // of loops and switches.
@@ -4454,6 +4611,8 @@ function jslint_phase3_parse(state) {
         the_fart.context = empty();
         the_fart.finally = 0;
         the_fart.loop = 0;
+        the_fart.parameters = parameters;
+        the_fart.signature = signature;
         the_fart.switch = 0;
         the_fart.try = 0;
 
@@ -4461,23 +4620,42 @@ function jslint_phase3_parse(state) {
 
         function_stack.push(functionage);
         functionage = the_fart;
-        the_fart.parameters = pl[0];
-        the_fart.signature = pl[1];
-        the_fart.parameters.forEach(function (name) {
+        the_fart.parameters.forEach(function enroll_parameter(name) {
+            if (name.identifier) {
+                enroll(name, "parameter", true);
+            } else {
+
+// PR-385 - Bugfix - Fixes issue #382 - fix warnings against destructured fart.
 
 // test_cause:
-// ["(aa)=>{}", "parse_fart", "parameter", "", 0]
+// ["([aa])=>0", "enroll_parameter", "use_function_not_fart", "=>", 7]
+// ["({aa})=>0", "enroll_parameter", "use_function_not_fart", "=>", 7]
+
+                warn("use_function_not_fart", the_fart);
+
+// Recurse enroll_parameter().
 
-            test_cause("parameter");
-            enroll(name, "parameter", true);
+                name.names.forEach(enroll_parameter);
+            }
         });
         if (token_nxt.id === "{") {
 
 // test_cause:
-// ["()=>{}", "parse_fart", "expected_a_b", "=>", 3]
+// ["()=>{}", "parse_fart", "use_function_not_fart", "=>", 3]
 
-            warn("expected_a_b", the_fart, "function", "=>");
+            warn("use_function_not_fart", the_fart);
             the_fart.block = block("body");
+        } else if (
+            syntax_dict[token_nxt.id] !== undefined
+            && syntax_dict[token_nxt.id].fud_stmt !== undefined
+        ) {
+
+// PR-384 - Bugfix - Fixes issue #379 - warn against naked-statement in fart.
+
+// test_cause:
+// ["()=>delete aa", "parse_fart", "unexpected_a_after_b", "=>", 5]
+
+            stop("unexpected_a_after_b", token_nxt, token_nxt.id, "=>");
         } else {
             the_fart.expression = parse_expression(0);
         }
@@ -4492,13 +4670,7 @@ function jslint_phase3_parse(state) {
         let negative;
         switch (token_nxt.id) {
         case "(number)":
-            if (!(
-
-// https://datatracker.ietf.org/doc/html/rfc7159#section-6
-// number = [ minus ] int [ frac ] [ exp ]
-
-                /^-?(?:0|[1-9]\d*?)(?:\.\d*?)?(?:[eE][+\-]?\d+?)?$/
-            ).test(token_nxt.value)) {
+            if (!jslint_rgx_json_number.test(token_nxt.value)) {
 
 // test_cause:
 // ["[-.0]", "parse_json", "unexpected_a", ".", 3]
@@ -4700,7 +4872,7 @@ function jslint_phase3_parse(state) {
         the_symbol = syntax_dict[first.id];
         if (
             the_symbol !== undefined
-            && the_symbol.fud !== undefined
+            && the_symbol.fud_stmt !== undefined
 
 // PR-318 - Bugfix - Fixes issues #316, #317 - dynamic-import().
 
@@ -4709,7 +4881,7 @@ function jslint_phase3_parse(state) {
             the_symbol.disrupt = false;
             the_symbol.statement = true;
             token_now.arity = "statement";
-            the_statement = the_symbol.fud();
+            the_statement = the_symbol.fud_stmt();
             functionage.statement_prv = the_statement;
         } else {
 
@@ -4779,7 +4951,7 @@ function jslint_phase3_parse(state) {
 // Create one of the postassign operators.
 
         const the_symbol = symbol(id, 150);
-        the_symbol.led = function (left) {
+        the_symbol.led_infix = function (left) {
             token_now.expression = left;
             token_now.arity = "postassign";
             check_mutation(token_now.expression);
@@ -4793,7 +4965,7 @@ function jslint_phase3_parse(state) {
 // Create one of the preassign operators.
 
         const the_symbol = symbol(id);
-        the_symbol.nud = function () {
+        the_symbol.nud_prefix = function () {
             const the_token = token_now;
             the_token.arity = "preassign";
             the_token.expression = parse_expression(150);
@@ -4808,7 +4980,7 @@ function jslint_phase3_parse(state) {
 // Create a prefix operator.
 
         const the_symbol = symbol(id);
-        the_symbol.nud = function () {
+        the_symbol.nud_prefix = function () {
             const the_token = token_now;
             the_token.arity = "unary";
             if (typeof f === "function") {
@@ -4929,18 +5101,20 @@ function jslint_phase3_parse(state) {
 //  }
 //  jslint_assert(!mode_mega, `Expected !mode_mega.`);
 
-// Don't create functions in loops. It is inefficient, and it can lead to
-// scoping errors.
-
-        if (functionage.loop > 0) {
-
-// test_cause:
-// ["
-// while(0){aa.map(function(){});}
-// ", "prefix_function", "function_in_loop", "function", 17]
-
-            warn("function_in_loop", the_function);
-        }
+// PR-378 - Relax warning "function_in_loop".
+//
+// // Don't create functions in loops. It is inefficient, and it can lead to
+// // scoping errors.
+//
+//         if (functionage.loop > 0) {
+//
+// // test_cause:
+// // ["
+// // while(0){aa.map(function(){});}
+// // ", "prefix_function", "function_in_loop", "function", 17]
+//
+//             warn("function_in_loop", the_function);
+//         }
 
 // Give the function properties for storing its names and for observing the
 // depth of loops and switches.
@@ -4988,6 +5162,9 @@ function jslint_phase3_parse(state) {
             if (name.identifier) {
                 enroll(name, "parameter", false);
             } else {
+
+// Recurse enroll_parameter().
+
                 name.names.forEach(enroll_parameter);
             }
         });
@@ -5448,25 +5625,14 @@ function jslint_phase3_parse(state) {
     }
 
     function prefix_lparen() {
-        const cadet = lookahead().id;
-        const the_paren = token_now;
+        let the_paren = token_now;
         let the_value;
 
-// We can distinguish between a parameter list for => and a wrapped expression
-// with one token of lookahead.
-
-        if (
-            token_nxt.id === ")"
-            || token_nxt.id === "..."
-            || (token_nxt.identifier && (cadet === "," || cadet === "="))
-        ) {
-
-// test_cause:
-// ["()=>0", "prefix_lparen", "fart", "", 0]
+// PR-385 - Bugfix - Fixes issue #382 - failure to detect destructured fart.
 
-            test_cause("fart");
+        if (token_now.is_fart) {
             the_paren.free = false;
-            return parse_fart(prefix_function_arg());
+            return parse_fart();
         }
 
 // test_cause:
@@ -5484,31 +5650,6 @@ function jslint_phase3_parse(state) {
         }
         the_value.wrapped = true;
         advance(")", the_paren);
-        if (token_nxt.id === "=>") {
-            if (the_value.arity !== "variable") {
-                if (the_value.id === "{" || the_value.id === "[") {
-
-// test_cause:
-// ["([])=>0", "prefix_lparen", "expected_a_before_b", "(", 1]
-// ["({})=>0", "prefix_lparen", "expected_a_before_b", "(", 1]
-
-                    warn("expected_a_before_b", the_paren, "function", "(");
-
-// test_cause:
-// ["([])=>0", "prefix_lparen", "expected_a_b", "=>", 5]
-// ["({})=>0", "prefix_lparen", "expected_a_b", "=>", 5]
-
-                    return stop("expected_a_b", token_nxt, "{", "=>");
-                }
-
-// test_cause:
-// ["(0)=>0", "prefix_lparen", "expected_identifier_a", "0", 2]
-
-                return stop("expected_identifier_a", the_value);
-            }
-            the_paren.expression = [the_value];
-            return parse_fart([the_paren.expression, "(" + the_value.id + ")"]);
-        }
         return the_value;
     }
 
@@ -5589,12 +5730,12 @@ function jslint_phase3_parse(state) {
         anon = "anonymous";
     }
 
-    function stmt(id, fud) {
+    function stmt(id, fud_stmt) {
 
 // Create a statement.
 
         const the_symbol = symbol(id);
-        the_symbol.fud = fud;
+        the_symbol.fud_stmt = fud_stmt;
         return the_symbol;
     }
 
@@ -6029,10 +6170,7 @@ function jslint_phase3_parse(state) {
         advance("from");
         advance("(string)");
         the_import.import = token_now;
-        if (!(
-            // rx_module
-            /^[a-zA-Z0-9_$:.@\-\/]+$/
-        ).test(token_now.value)) {
+        if (!jslint_rgx_module.test(token_now.value)) {
 
 // test_cause:
 // ["import aa from \"!aa\"", "stmt_import", "bad_module_name_a", "!aa", 16]
@@ -6643,13 +6781,13 @@ function jslint_phase3_parse(state) {
 
         if (id === "(string)") {
             id = name.value;
-            if (!rx_identifier.test(id)) {
+            if (!jslint_rgx_identifier.test(id)) {
                 return id;
             }
         } else if (id === "`") {
             if (name.value.length === 1) {
                 id = name.value[0].value;
-                if (!rx_identifier.test(id)) {
+                if (!jslint_rgx_identifier.test(id)) {
                     return id;
                 }
             }
@@ -6682,10 +6820,7 @@ function jslint_phase3_parse(state) {
             } else if (
                 !option_dict.name
                 && name.identifier
-                && (
-                    // rx_weird_property
-                    /^_|\$|Sync$|_$/m
-                ).test(id)
+                && jslint_rgx_weird_property.test(id)
             ) {
 
 // test_cause:
@@ -6723,7 +6858,7 @@ function jslint_phase3_parse(state) {
 // Create a ternary operator.
 
         const the_symbol = symbol(id1, 30);
-        the_symbol.led = function parse_ternary_led(left) {
+        the_symbol.led_infix = function parse_ternary_led(left) {
             const the_token = token_now;
             let second;
             second = parse_expression(20);
@@ -7439,10 +7574,7 @@ function jslint_phase4_walk(state) {
                 test_cause("cack");
                 cack = !cack;
             }
-            if ((
-                // rx_cap
-                /^[A-Z]/
-            ).test(left.name.id) !== cack) {
+            if (jslint_rgx_cap.test(left.name.id) !== cack) {
                 if (the_new !== undefined) {
 
 // test_cause:
@@ -9311,6 +9443,9 @@ body {
         : "global"
     );
     if (global.length + froms.length + exports.length > 0) {
+        if (functions.length === 0) {
+            html += "<br>\n";
+        }
         html += "<div class=\"level level0\">\n";
         html += detail(module, global);
         html += detail("import from", froms);
@@ -9560,7 +9695,7 @@ function objectDeepCopyWithKeysSorted(obj) {
 
 // Recursively deep-copy obj with keys sorted.
 
-    sorted = {};
+    sorted = Object.create(null);
     Object.keys(obj).sort().forEach(function (key) {
         sorted[key] = objectDeepCopyWithKeysSorted(obj[key]);
     });
@@ -10834,7 +10969,7 @@ function sentinel() {}
 
 // 3. Create html-coverage-reports in <coverageDir>.
 
-    fileDict = {};
+    fileDict = Object.create(null);
     await Promise.all(v8CoverageObj.result.map(async function ({
         functions,
         url: pathname