Catch method calls too

Author: RReverser

src/cpuprofiler.js

@@ -366,7 +366,7 @@ var emscriptenCpuProfiler = {
       fpsOverlay.style = 'position: fixed; font-weight: bold; padding: 3px; -webkit-touch-callout: none; -webkit-user-select: none; -khtml-user-select: none; -moz-user-select: none; -ms-user-select: none; user-select: none; cursor: pointer;';
       fpsOverlay.onclick = () => {
         var view = document.getElementById('cpuprofiler_canvas');
-        if (view) view.scrollIntoView();
+        view?.scrollIntoView();
       };
       fpsOverlay.oncontextmenu = (e) => e.preventDefault();
       document.body.appendChild(fpsOverlay);

src/headless.js

@@ -98,9 +98,7 @@ var window = {
   },
   callEventListeners(id) {
     var listeners = this.eventListeners[id];
-    if (listeners) {
-      listeners.forEach((listener) => listener());
-    }
+    listeners?.forEach((listener) => listener());
   },
   URL: {
     createObjectURL(x) {

src/library_glemu.js

@@ -182,7 +182,7 @@ var LibraryGLEmulation = {
       _glEnable = _emscripten_glEnable = (cap) => {
         // Clean up the renderer on any change to the rendering state. The optimization of
         // skipping renderer setup is aimed at the case of multiple glDraw* right after each other
-        if (GLImmediate.lastRenderer) GLImmediate.lastRenderer.cleanup();
+        GLImmediate.lastRenderer?.cleanup();
         if (cap == 0xB60 /* GL_FOG */) {
           if (GLEmulation.fogEnabled != true) {
             GLImmediate.currentRenderer = null; // Fog parameter is part of the FFP shader state, we must re-lookup the renderer to use.
@@ -231,7 +231,7 @@ var LibraryGLEmulation = {
 
       var glDisable = _glDisable;
       _glDisable = _emscripten_glDisable = (cap) => {
-        if (GLImmediate.lastRenderer) GLImmediate.lastRenderer.cleanup();
+        GLImmediate.lastRenderer?.cleanup();
         if (cap == 0xB60 /* GL_FOG */) {
           if (GLEmulation.fogEnabled != false) {
             GLImmediate.currentRenderer = null; // Fog parameter is part of the FFP shader state, we must re-lookup the renderer to use.
@@ -3537,7 +3537,7 @@ var LibraryGLEmulation = {
   $emulGlBindVertexArray: (vao) => {
     // undo vao-related things, wipe the slate clean, both for vao of 0 or an actual vao
     GLEmulation.currentVao = null; // make sure the commands we run here are not recorded
-    if (GLImmediate.lastRenderer) GLImmediate.lastRenderer.cleanup();
+    GLImmediate.lastRenderer?.cleanup();
     _glBindBuffer(GLctx.ARRAY_BUFFER, 0); // XXX if one was there before we were bound?
     _glBindBuffer(GLctx.ELEMENT_ARRAY_BUFFER, 0);
     for (var vaa in GLEmulation.enabledVertexAttribArrays) {

src/library_trace.js

@@ -63,9 +63,7 @@ var LibraryTracing = {
       req.addEventListener('load', function() {
         var blob = new Blob([this.responseText], { type: 'text/javascript' });
         var worker = new Worker(window.URL.createObjectURL(blob));
-        if (ready) {
-          ready.call(scope, worker);
-        }
+        ready?.call(scope, worker);
       }, req);
       req.open("get", url, false);
       req.send();

src/library_wget.js

@@ -232,9 +232,7 @@ var LibraryWget = {
   emscripten_async_wget2_abort__proxy: 'sync',
   emscripten_async_wget2_abort: (handle) => {
     var http = wget.wgetRequests[handle];
-    if (http) {
-      http.abort();
-    }
+    http?.abort();
   },
 };
 

src/webGLWorker.js

@@ -738,7 +738,7 @@ function WebGLWorker() {
           }
           if (!obj[name]) {
             obj[name] = { what: type, name: fullname, size, location: -1, type: getTypeId(m[1]) };
-            if (vec) vec.push(name);
+            vec?.push(name);
           }
         });
       });