Issue 5131 - [emscripten] Clean separation of bindings code and runtime code

compile

 #!/usr/bin/env python
 
 # This file is part of Adblock Plus <https://adblockplus.org/>,
 # Copyright (C) 2006-2017 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
@@ skipping 23 matching lines @@
     'ALLOW_MEMORY_GROWTH': 1,
     'NO_EXIT_RUNTIME': 1,
     'NO_DYNAMIC_EXECUTION': 1,
     'NO_FILESYSTEM': 1,
     'INVOKE_RUN': 0,
     'TEXTDECODER': 0,
     'EXPORTED_RUNTIME_METHODS': ['cwrap', 'ccall', 'stringToAscii'],
 }
 DEFINES = []
 ADDITIONAL_PARAMS = ['-O3', '-m32', '-std=gnu++14', '--memory-init-file', '0',
-                     '--emit-symbol-map', '-Wall', '-Werror', '-fno-exceptions']
+                     '--emit-symbol-map', '-Wall', '-Werror', '-fno-rtti']
 
 
 def get_source_files(phase):
     for (path, dirs, files) in os.walk(SOURCE_DIR):
         for f in files:
             if os.path.splitext(f)[1] != '.cpp':
                 continue
             if (
                 phase != 'bindings' and
                 os.path.basename(path) == 'bindings' and
                 not f.startswith('runtime_')

[sergei, 2017/04/18 10:46:09]

I expected bindings/runtime_utils.cpp and files for generator to be in different
directories.

[Wladimir Palant, 2017/04/18 11:01:37]

I know. However, IMHO code locality is more important here - this is all the
code related to JS bindings, including the runtime parts that the generator
expects. If anything, maybe it should be bindings/generation/{main|generator}
and bindings/runtime/utils.

             ):
                 continue
             yield os.path.join(path, f)
 
 def getenv(emscripten_config):
     scope = {}
     execfile(emscripten_config, scope, scope)
     env = os.environ.copy()
     env.update({
         'EM_CONFIG': emscripten_config,
@@ skipping 54 matching lines @@
     parser.add_argument(
         '-t', '--tracing',
         action='store_true',
         help='Enable memory tracing'
     )
     args = parser.parse_args()
 
     env = getenv(args.emscripten_config)
     generate_bindings(env)
     run_compiler(env, debug=args.debug, tracing=args.tracing)