Remove redundant assertion from emscripten_console_* functions. NFC

The UTF8ToString functions that these all call has this assertion
already and with a useful message.

Author: sbc100

src/lib/libcore.js

@@ -1967,42 +1967,16 @@ addToLibrary({
   _emscripten_get_progname__deps: ['$getExecutableName', '$stringToUTF8'],
   _emscripten_get_progname: (str, len) => stringToUTF8(getExecutableName(), str, len),
 
-  emscripten_console_log: (str) => {
-#if ASSERTIONS
-    assert(typeof str == 'number');
-#endif
-    console.log(UTF8ToString(str));
-  },
-
-  emscripten_console_warn: (str) => {
-#if ASSERTIONS
-    assert(typeof str == 'number');
-#endif
-    console.warn(UTF8ToString(str));
-  },
-
-  emscripten_console_error: (str) => {
-#if ASSERTIONS
-    assert(typeof str == 'number');
-#endif
-    console.error(UTF8ToString(str));
-  },
-
-  emscripten_console_trace: (str) => {
-#if ASSERTIONS
-    assert(typeof str == 'number');
-#endif
-    console.trace(UTF8ToString(str));
-  },
+  emscripten_console_log: (str) => console.log(UTF8ToString(str)),
+  emscripten_console_warn: (str) => console.warn(UTF8ToString(str)),
+  emscripten_console_error: (str) => console.error(UTF8ToString(str)),
+  emscripten_console_trace: (str) => console.trace(UTF8ToString(str)),
 
   emscripten_throw_number: (number) => {
     throw number;
   },
 
   emscripten_throw_string: (str) => {
-#if ASSERTIONS
-    assert(typeof str == 'number');
-#endif
     throw UTF8ToString(str);
   },
 
@@ -2185,7 +2159,7 @@ addToLibrary({
 
   $alignMemory: (size, alignment) => {
 #if ASSERTIONS
-    assert(alignment, "alignment argument is required");
+    assert(alignment, 'alignment argument is required');
 #endif
     return Math.ceil(size / alignment) * alignment;
   },