Remove assert(x) hack in favor of NDEBUG

basis_transcoder.js 140999
basis_transcoder.wasm 832985

transcoder/basisu.h

@@ -50,13 +50,6 @@
 #include <assert.h>
 #include <random>
 
-#if defined(__EMSCRIPTEN__) && !defined(_DEBUG) && !defined(DEBUG)
-// HUGE HACK: I've been unable to disable assertions using emcc -O2 -s ASSERTIONS=0, no idea why. 
-// We definitely don't want them enabled in release.
-#undef assert
-#define assert(x) ((void)0)
-#endif
-
 #ifdef max
 #undef max
 #endif

webgl/transcoder/CMakeLists.txt

@@ -15,5 +15,5 @@ if (EMSCRIPTEN)
   set_target_properties(basis_transcoder.js PROPERTIES
       OUTPUT_NAME "basis_transcoder"
       SUFFIX ".js"
-      LINK_FLAGS "--bind -s ALLOW_MEMORY_GROWTH=1 -O3 -s ASSERTIONS=0")
+      LINK_FLAGS "--bind -s ALLOW_MEMORY_GROWTH=1 -O3 -s ASSERTIONS=0 -DNDEBUG")
 endif()