Run prettier on compiler-internal JS files. NFC (#21545)

This is in preparation for #21542 which converts them to `.mjs`.  Since
we run prettier during CI on all `.mjs` files this change this change
avoid superfluous changes at the point of rename.

Author: sbc100

src/jsifier.js

@@ -31,7 +31,7 @@ function mangleCSymbolName(f) {
 function splitter(array, filter) {
   const splitOut = array.filter(filter);
   const leftIn = array.filter((x) => !filter(x));
-  return { leftIn, splitOut };
+  return {leftIn, splitOut};
 }
 
 function escapeJSONKey(x) {
@@ -54,7 +54,7 @@ function stringifyWithFunctions(obj) {
     // Handle JS method syntax where the function property starts with its own
     // name. e.g.  foo(a) {},
     if (typeof value === 'function' && str.startsWith(key)) {
-      rtn += str + ',\n'
+      rtn += str + ',\n';
     } else {
       rtn += escapeJSONKey(key) + ':' + str + ',\n';
     }
@@ -98,7 +98,7 @@ function getTransitiveDeps(symbol) {
 }
 
 function shouldPreprocess(fileName) {
-  var content = read(fileName).trim()
+  var content = read(fileName).trim();
   return content.startsWith('#preprocess\n') || content.startsWith('#preprocess\r\n');
 }
 
@@ -157,7 +157,10 @@ function runJSify() {
     return modifyJSFunction(snippet, (args, body, async_, oneliner) => {
       let argLines = args.split('\n');
       argLines = argLines.map((line) => line.split('//')[0]);
-      const argNames = argLines.join(' ').split(',').map((name) => name.trim());
+      const argNames = argLines
+        .join(' ')
+        .split(',')
+        .map((name) => name.trim());
       const newArgs = [];
       let innerArgs = [];
       let argConversions = '';
@@ -198,9 +201,9 @@ function runJSify() {
             return `${async_}(${args}) => {
 ${argConversions}
   return ${makeReturn64(body)};
-}`
+}`;
           }
-          return `${async_}(${args}) => ${makeReturn64(body)};`
+          return `${async_}(${args}) => ${makeReturn64(body)};`;
         }
         return `\
 ${async_}function(${args}) {
@@ -243,22 +246,27 @@ ${argConversions}
 
     // apply LIBRARY_DEBUG if relevant
     if (LIBRARY_DEBUG && !isJsOnlySymbol(symbol)) {
-      snippet = modifyJSFunction(snippet, (args, body, async) => `\
+      snippet = modifyJSFunction(
+        snippet,
+        (args, body, async) => `\
 function(${args}) {
   var ret = (() => { if (runtimeDebug) err("[library call:${mangled}: " + Array.prototype.slice.call(arguments).map(prettyPrint) + "]");
   ${body}
   })();
   if (runtimeDebug && typeof ret != "undefined") err("  [     return:" + prettyPrint(ret));
   return ret;
-}`);
+}`,
+      );
     }
 
     const sig = LibraryManager.library[symbol + '__sig'];
     const i53abi = LibraryManager.library[symbol + '__i53abi'];
-    if (sig &&
-        ((i53abi && sig.includes('j')) || ((MEMORY64 || CAN_ADDRESS_2GB) && sig.includes('p')))) {
+    if (
+      sig &&
+      ((i53abi && sig.includes('j')) || ((MEMORY64 || CAN_ADDRESS_2GB) && sig.includes('p')))
+    ) {
       snippet = handleI64Signatures(symbol, snippet, sig, i53abi);
-      i53ConversionDeps.forEach((d) => deps.push(d))
+      i53ConversionDeps.forEach((d) => deps.push(d));
     }
 
     const proxyingMode = LibraryManager.library[symbol + '__proxy'];
@@ -274,23 +282,27 @@ function(${args}) {
               body = `return ${body}`;
             }
             const rtnType = sig && sig.length ? sig[0] : null;
-            const proxyFunc = (MEMORY64 && rtnType == 'p') ? 'proxyToMainThreadPtr' : 'proxyToMainThread';
+            const proxyFunc =
+              MEMORY64 && rtnType == 'p' ? 'proxyToMainThreadPtr' : 'proxyToMainThread';
             deps.push('$' + proxyFunc);
             return `
 function(${args}) {
 if (ENVIRONMENT_IS_PTHREAD)
   return ${proxyFunc}(${proxiedFunctionTable.length}, 0, ${+sync}${args ? ', ' : ''}${args});
 ${body}
-}\n`
+}\n`;
           });
         } else if (WASM_WORKERS && ASSERTIONS) {
           // In ASSERTIONS builds add runtime checks that proxied functions are not attempted to be called in Wasm Workers
           // (since there is no automatic proxying architecture available)
-          snippet = modifyJSFunction(snippet, (args, body) => `
+          snippet = modifyJSFunction(
+            snippet,
+            (args, body) => `
 function(${args}) {
   assert(!ENVIRONMENT_IS_WASM_WORKER, "Attempted to call proxied function '${mangled}' in a Wasm Worker, but in Wasm Worker enabled builds, proxied function architecture is not available!");
   ${body}
-}\n`);
+}\n`,
+          );
         }
         proxiedFunctionTable.push(mangled);
       }
@@ -328,7 +340,9 @@ function(${args}) {
     // of argument.
     if (LINK_AS_CXX && !WASM_EXCEPTIONS && symbol.startsWith('__cxa_find_matching_catch_')) {
       if (DISABLE_EXCEPTION_THROWING) {
-        error('DISABLE_EXCEPTION_THROWING was set (likely due to -fno-exceptions), which means no C++ exception throwing support code is linked in, but exception catching code appears. Either do not set DISABLE_EXCEPTION_THROWING (if you do want exception throwing) or compile all source files with -fno-except (so that no exceptions support code is required); also make sure DISABLE_EXCEPTION_CATCHING is set to the right value - if you want exceptions, it should be off, and vice versa.');
+        error(
+          'DISABLE_EXCEPTION_THROWING was set (likely due to -fno-exceptions), which means no C++ exception throwing support code is linked in, but exception catching code appears. Either do not set DISABLE_EXCEPTION_THROWING (if you do want exception throwing) or compile all source files with -fno-except (so that no exceptions support code is required); also make sure DISABLE_EXCEPTION_CATCHING is set to the right value - if you want exceptions, it should be off, and vice versa.',
+        );
         return;
       }
       if (!(symbol in LibraryManager.library)) {
@@ -374,9 +388,10 @@ function(${args}) {
       let isAsyncFunction = false;
       if (ASYNCIFY) {
         const original = LibraryManager.library[symbol];
-        if (typeof original == 'function' ) {
-          isAsyncFunction = LibraryManager.library[symbol + '__async'] ||
-                            original.constructor.name == 'AsyncFunction'
+        if (typeof original == 'function') {
+          isAsyncFunction =
+            LibraryManager.library[symbol + '__async'] ||
+            original.constructor.name == 'AsyncFunction';
         }
         if (isAsyncFunction) {
           asyncFuncs.push(symbol);
@@ -388,11 +403,17 @@ function(${args}) {
           var value = LibraryManager.library[symbol];
           var resolvedSymbol = symbol;
           // Resolve aliases before looking up deps
-          if (typeof value == 'string' && value[0] != '=' && LibraryManager.library.hasOwnProperty(value)) {
+          if (
+            typeof value == 'string' &&
+            value[0] != '=' &&
+            LibraryManager.library.hasOwnProperty(value)
+          ) {
             resolvedSymbol = value;
           }
           var transtiveDeps = getTransitiveDeps(resolvedSymbol);
-          symbolDeps[symbol] = transtiveDeps.filter((d) => !isJsOnlySymbol(d) && !(d in LibraryManager.library));
+          symbolDeps[symbol] = transtiveDeps.filter(
+            (d) => !isJsOnlySymbol(d) && !(d in LibraryManager.library),
+          );
         }
         return;
       }
@@ -419,8 +440,13 @@ function(${args}) {
           if (dependent) msg += ` (referenced by ${dependent})`;
           if (ERROR_ON_UNDEFINED_SYMBOLS) {
             error(msg);
-            warnOnce('To disable errors for undefined symbols use `-sERROR_ON_UNDEFINED_SYMBOLS=0`');
-            warnOnce(mangled + ' may need to be added to EXPORTED_FUNCTIONS if it arrives from a system library');
+            warnOnce(
+              'To disable errors for undefined symbols use `-sERROR_ON_UNDEFINED_SYMBOLS=0`',
+            );
+            warnOnce(
+              mangled +
+                ' may need to be added to EXPORTED_FUNCTIONS if it arrives from a system library',
+            );
           } else if (VERBOSE || WARN_ON_UNDEFINED_SYMBOLS) {
             warn(msg);
           }
@@ -508,7 +534,7 @@ function(${args}) {
       }
 
       if (VERBOSE) {
-        printErr(`adding ${symbol} (referenced by ${dependent})`)
+        printErr(`adding ${symbol} (referenced by ${dependent})`);
       }
       const deps_list = deps.join("','");
       const identDependents = symbol + `__deps: ['${deps_list}']`;
@@ -521,15 +547,23 @@ function(${args}) {
         // in library.js and library_pthread.js.  These happen before deps are
         // processed so depending on it via `__deps` doesn't work.
         if (dep === '$noExitRuntime') {
-          error('noExitRuntime cannot be referenced via __deps mechanism.  Use DEFAULT_LIBRARY_FUNCS_TO_INCLUDE or EXPORTED_RUNTIME_METHODS')
+          error(
+            'noExitRuntime cannot be referenced via __deps mechanism.  Use DEFAULT_LIBRARY_FUNCS_TO_INCLUDE or EXPORTED_RUNTIME_METHODS',
+          );
         }
         return addFromLibrary(dep, `${symbol}, referenced by ${dependent}`, dep === aliasTarget);
       }
       let contentText;
       if (isFunction) {
         // Emit the body of a JS library function.
-        if ((USE_ASAN || USE_LSAN || UBSAN_RUNTIME) && LibraryManager.library[symbol + '__noleakcheck']) {
-          contentText = modifyJSFunction(snippet, (args, body) => `(${args}) => withBuiltinMalloc(() => {${body}})`);
+        if (
+          (USE_ASAN || USE_LSAN || UBSAN_RUNTIME) &&
+          LibraryManager.library[symbol + '__noleakcheck']
+        ) {
+          contentText = modifyJSFunction(
+            snippet,
+            (args, body) => `(${args}) => withBuiltinMalloc(() => {${body}})`,
+          );
           deps.push('$withBuiltinMalloc');
         } else {
           contentText = snippet; // Regular JS function that will be executed in the context of the calling thread.
@@ -552,7 +586,8 @@ function(${args}) {
         //  emits
         //   'var foo;[code here verbatim];'
         contentText = 'var ' + mangled + snippet;
-        if (snippet[snippet.length - 1] != ';' && snippet[snippet.length - 1] != '}') contentText += ';';
+        if (snippet[snippet.length - 1] != ';' && snippet[snippet.length - 1] != '}')
+          contentText += ';';
       } else if (typeof snippet == 'undefined') {
         contentText = `var ${mangled};`;
       } else {
@@ -593,13 +628,18 @@ function(${args}) {
 
       let commentText = '';
       if (force) {
-        commentText += '/** @suppress {duplicate } */\n'
+        commentText += '/** @suppress {duplicate } */\n';
       }
       if (LibraryManager.library[symbol + '__docs']) {
         commentText += LibraryManager.library[symbol + '__docs'] + '\n';
       }
 
-      const depsText = (deps ? deps.map(addDependency).filter((x) => x != '').join('\n') + '\n' : '');
+      const depsText = deps
+        ? deps
+            .map(addDependency)
+            .filter((x) => x != '')
+            .join('\n') + '\n'
+        : '';
       return depsText + commentText + contentText;
     }
 
@@ -608,7 +648,7 @@ function(${args}) {
   }
 
   function includeFile(fileName, needsPreprocess = true) {
-    print(getIncludeFile(fileName, needsPreprocess))
+    print(getIncludeFile(fileName, needsPreprocess));
   }
 
   function finalCombiner() {
@@ -672,26 +712,31 @@ var proxiedFunctionTable = [
       includeFile(fileName, shouldPreprocess(fileName));
     }
 
-    print('//FORWARDED_DATA:' + JSON.stringify({
-      librarySymbols,
-      warnings,
-      asyncFuncs,
-      ATINITS: ATINITS.join('\n'),
-      ATMAINS: STRICT ? '' : ATMAINS.join('\n'),
-      ATEXITS: ATEXITS.join('\n'),
-    }));
+    print(
+      '//FORWARDED_DATA:' +
+        JSON.stringify({
+          librarySymbols,
+          warnings,
+          asyncFuncs,
+          ATINITS: ATINITS.join('\n'),
+          ATMAINS: STRICT ? '' : ATMAINS.join('\n'),
+          ATEXITS: ATEXITS.join('\n'),
+        }),
+    );
   }
 
   for (const sym of symbolsNeeded) {
     symbolHandler(sym);
   }
 
   if (symbolsOnly) {
-    print(JSON.stringify({
-      deps: symbolDeps,
-      asyncFuncs,
-      extraLibraryFuncs,
-    }));
+    print(
+      JSON.stringify({
+        deps: symbolDeps,
+        asyncFuncs,
+        extraLibraryFuncs,
+      }),
+    );
   } else {
     finalCombiner();
   }

src/modules.js

@@ -225,13 +225,15 @@ globalThis.LibraryManager = {
       currentFile = filename;
       try {
         processed = processMacros(preprocess(filename), filename);
-        vm.runInThisContext(processed, { filename: filename.replace(/\.\w+$/, '.preprocessed$&') });
+        vm.runInThisContext(processed, {filename: filename.replace(/\.\w+$/, '.preprocessed$&')});
       } catch (e) {
         error(`failure to execute js library "${filename}":`);
         if (VERBOSE) {
           const orig = read(filename);
           if (processed) {
-            error(`preprocessed source (you can run a js engine on this to get a clearer error message sometimes):\n=============\n${processed}\n=============`);
+            error(
+              `preprocessed source (you can run a js engine on this to get a clearer error message sometimes):\n=============\n${processed}\n=============`,
+            );
           } else {
             error(`original source:\n=============\n${orig}\n=============`);
           }
@@ -252,7 +254,7 @@ globalThis.LibraryManager = {
 if (!BOOTSTRAPPING_STRUCT_INFO) {
   let structInfoFile = 'generated_struct_info32.json';
   if (MEMORY64) {
-    structInfoFile = 'generated_struct_info64.json'
+    structInfoFile = 'generated_struct_info64.json';
   }
   // Load struct and define information.
   const temp = JSON.parse(read(structInfoFile));
@@ -268,19 +270,23 @@ if (!BOOTSTRAPPING_STRUCT_INFO) {
 C_STRUCTS = new Proxy(C_STRUCTS, {
   get(target, prop, receiver) {
     if (!(prop in target)) {
-      throw new Error(`Missing C struct ${prop}! If you just added it to struct_info.json, you need to run ./tools/gen_struct_info.py`);
+      throw new Error(
+        `Missing C struct ${prop}! If you just added it to struct_info.json, you need to run ./tools/gen_struct_info.py`,
+      );
     }
-    return target[prop]
-  }
+    return target[prop];
+  },
 });
 
 cDefs = C_DEFINES = new Proxy(C_DEFINES, {
   get(target, prop, receiver) {
     if (!(prop in target)) {
-      throw new Error(`Missing C define ${prop}! If you just added it to struct_info.json, you need to run ./tools/gen_struct_info.py`);
+      throw new Error(
+        `Missing C define ${prop}! If you just added it to struct_info.json, you need to run ./tools/gen_struct_info.py`,
+      );
     }
-    return target[prop]
-  }
+    return target[prop];
+  },
 });
 
 // Legacy function that existed solely to give error message.  These are now
@@ -383,10 +389,14 @@ function exportRuntime() {
     'HEAPF32',
     'HEAPF64',
     'HEAP_DATA_VIEW',
-    'HEAP8',  'HEAPU8',
-    'HEAP16', 'HEAPU16',
-    'HEAP32', 'HEAPU32',
-    'HEAP64', 'HEAPU64',
+    'HEAP8',
+    'HEAPU8',
+    'HEAP16',
+    'HEAPU16',
+    'HEAP32',
+    'HEAPU32',
+    'HEAP64',
+    'HEAPU64',
   ];
 
   // These are actually native wasm functions these days but we allow exporting
@@ -424,11 +434,11 @@ function exportRuntime() {
   }
 
   if (RETAIN_COMPILER_SETTINGS) {
-    runtimeElements.push('getCompilerSetting')
+    runtimeElements.push('getCompilerSetting');
   }
 
   if (RUNTIME_DEBUG) {
-    runtimeElements.push('prettyPrint')
+    runtimeElements.push('prettyPrint');
   }
 
   // dynCall_* methods are not hardcoded here, as they