Issue 4125 - [emscripten] Convert filter classes to C++

compiled/debug.h

Index: compiled/debug.h
===================================================================
new file mode 100644
--- /dev/null
+++ b/compiled/debug.h
@@ -0,0 +1,66 @@
+#pragma once
+
+#include <emscripten.h>
+#include <emscripten/trace.h>
+
+#if defined(assert)
+#undef assert
+#endif
+
+class String;
+
+struct console_type
+{
+  static void log(const String& str)
+  {
+    EM_ASM_ARGS(console.log(readString($0)), &str);
+  }
+
+  static void log(int i)
+  {
+    EM_ASM_ARGS(console.log($0), i);
+  }
+
+  static void log(void* ptr)
+  {
+    EM_ASM_ARGS(console.log($0), ptr);
+  }
+
+  static void error(const String& str)
+  {
+    EM_ASM_ARGS(console.error(new Error(readString($0)).stack), &str);
+  }
+};
+
+static console_type console;

[sergei, 2017/01/10 15:58:12]

Why do we need a global static variable of a class all methods of which are
static?

[Wladimir Palant, 2017/03/13 17:42:32]

Just so that one can call `console.log()` the way you would do in JavaScript.

[sergei, 2017/03/20 17:06:58]

`console.log()` calls a method of `console` object, it's not required to have
these methods to be static. Though, for me it's fine to have the class with
static methods but I think that we don't need an instance of it (per each
translation unit), it should work if we simply rename console_type to console.

+
+#if defined(DEBUG)
+inline void assert(bool condition, const String& str)
+{
+  if (!condition)
+    console.error(str);
+}
+#else
+#define assert(condition, str)
+#endif
+
+inline void annotate_address(void* address, const char* name)
+{
+#if defined(__EMSCRIPTEN_TRACING__)
+  emscripten_trace_annotate_address_type(address, name);
+#endif
+}
+
+inline void enter_context(const char* context)
+{
+#if defined(__EMSCRIPTEN_TRACING__)
+  emscripten_trace_enter_context(context);
+#endif
+}
+
+inline void exit_context()
+{
+#if defined(__EMSCRIPTEN_TRACING__)
+  emscripten_trace_exit_context();
+#endif
+}