Skip to content

Commit c3bdb49

Browse files
authored
[eslint] Enable all recommended rules. NFC (#23431)
1 parent 0ec72a8 commit c3bdb49

File tree

2 files changed

+7
-12
lines changed

2 files changed

+7
-12
lines changed

eslint.config.mjs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,8 @@ export default [{
5252
'src/Fetch.js',
5353
'src/settings.js',
5454
'src/settings_internal.js',
55+
'src/growableHeap.js',
56+
'src/emrun_prejs.js',
5557
'src/arrayUtils.js',
5658
'src/deterministic.js',
5759
'src/base64Decode.js',
@@ -61,7 +63,7 @@ export default [{
6163
'src/URIUtils.js',
6264
'tools/experimental',
6365
],
64-
}, ...compat.extends('prettier'), {
66+
}, ...compat.extends('prettier'), js.configs.recommended, {
6567
languageOptions: {
6668
globals: {
6769
...globals.browser,
@@ -79,6 +81,7 @@ export default [{
7981
'require-jsdoc': 'off',
8082
'arrow-body-style': ['error', 'as-needed'],
8183
'space-infix-ops': 'error',
84+
'no-prototype-builtins': 'off',
8285

8386
quotes: ['error', 'single', {
8487
avoidEscape: true,

tools/acorn-optimizer.mjs

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -457,7 +457,7 @@ function JSDCE(ast, aggressive) {
457457
cleanUp(ast, names);
458458
return removed;
459459
}
460-
while (iteration() && aggressive) {}
460+
while (iteration() && aggressive) {} // eslint-disable-line no-empty
461461
}
462462

463463
// Aggressive JSDCE - multiple iterations
@@ -1275,8 +1275,6 @@ function growableHeap(ast) {
12751275
makeCallExpression(node, 'GROWABLE_HEAP_F64', []);
12761276
break;
12771277
}
1278-
default: {
1279-
}
12801278
}
12811279
}
12821280
},
@@ -1419,8 +1417,6 @@ function asanify(ast) {
14191417
makeCallExpression(node, '_asan_js_store_d', [ptr, value]);
14201418
break;
14211419
}
1422-
default: {
1423-
}
14241420
}
14251421
} else {
14261422
c(target);
@@ -1466,8 +1462,6 @@ function asanify(ast) {
14661462
makeCallExpression(node, '_asan_js_load_d', [ptr]);
14671463
break;
14681464
}
1469-
default: {
1470-
}
14711465
}
14721466
}
14731467
},
@@ -1615,8 +1609,6 @@ function safeHeap(ast) {
16151609
]);
16161610
break;
16171611
}
1618-
default: {
1619-
}
16201612
}
16211613
}
16221614
},
@@ -1660,7 +1652,7 @@ function ensureMinifiedNames(n) {
16601652
if (!RESERVED.has(name)) minifiedNames.push(name);
16611653
// increment the state
16621654
let i = 0;
1663-
while (1) {
1655+
while (true) {
16641656
minifiedState[i]++;
16651657
if (minifiedState[i] < (i === 0 ? VALID_MIN_INITS : VALID_MIN_LATERS).length) break;
16661658
// overflow
@@ -1739,7 +1731,7 @@ function minifyLocals(ast) {
17391731
let nextMinifiedName = 0;
17401732

17411733
function getNextMinifiedName() {
1742-
while (1) {
1734+
while (true) {
17431735
ensureMinifiedNames(nextMinifiedName);
17441736
const minified = minifiedNames[nextMinifiedName++];
17451737
// TODO: we can probably remove !isLocalName here

0 commit comments

Comments
 (0)