show signature in dyncall errors, and simplify approach

2 files changed, 2 insertions, 9 deletions

diff --git a/src/jsifier.js b/src/jsifier.js
index ff43c8c6..0e82a547 100644
--- a/src/jsifier.js
+++ b/src/jsifier.js
@@ -1402,7 +1402,8 @@ function JSify(data, functionsOnly, givenFunctions) {
         assert(returnType.search(/\("'\[,/) == -1); // XXX need isFunctionType(type, out)
         callIdent = '(' + callIdent + ')&{{{ FTM_' + sig + ' }}}'; // the function table mask is set in emscripten.py
       } else if (SAFE_DYNCALLS) {
-        callIdent = '(tempInt=' + callIdent + ',tempInt < 0 || tempInt >= FUNCTION_TABLE.length-1 ? abort("dyncall error") : tempInt)';
+        assert(!ASM_JS, 'cannot emit safe dyncalls in asm');
+        callIdent = '(tempInt=' + callIdent + ',tempInt < 0 || tempInt >= FUNCTION_TABLE.length-1 || !FUNCTION_TABLE[tempInt] ? abort("dyncall error: ' + sig + '") : tempInt)';
       }
       callIdent = Functions.getTable(sig) + '[' + callIdent + ']';
     }
diff --git a/src/modules.js b/src/modules.js
index 7f8a959b..501ff22e 100644
--- a/src/modules.js
+++ b/src/modules.js
@@ -330,14 +330,6 @@ var Functions = {
           }
         }
       }
-      if (SAFE_DYNCALLS) {
-        assert(!ASM_JS, 'cannot emit safe dyncalls in asm');
-        for (var j = 0; j < table.length; j++) {
-          if (table[j] == 0) {
-            table[j] = "function() { abort('dyncall error') }";
-          }
-        }
-      }
       if (table.length > 20) {
         // add some newlines in the table, for readability
         var j = 10;