optional param to prevent function overwriting

src/utility.js

@@ -92,7 +92,22 @@ function sum(x) {
   return x.reduce((a, b) => a + b, 0);
 }
 
-function mergeInto(obj, other) {
+global.LibraryParam = Object.freeze({
+  DisallowOverride: 1,
+});
+
+// if allowOverride is false, it shows error in case of symbol redefinition
+function mergeInto(obj, other, libraryParams = 0) {
+  // check for unintended symbol redefinition
+  if (libraryParams === LibraryParam.DisallowOverride) {
+    for (const key of Object.keys(other)) {
+      if (obj.hasOwnProperty(key)) {
+        error('Symbol re-definition in JavaScript library: ' + key + '. Do not use DisallowOverride if this is intended');
+        return;
+      }
+    }
+  }
+
   return Object.assign(obj, other);
 }
 

tests/test_other.py

@@ -3423,6 +3423,39 @@ def test_compilation_database(self):
     self.run_process([EMCC, 'a.c', '-MJ', 'hello.json', '-c', '-o', 'test.o'])
     self.assertContained('"file": "a.c", "output": "test.o"', read_file('hello.json'))
 
+  def test_duplicate_js_functions(self):
+    create_file('duplicated_func.cpp', '''
+      #include <stdio.h>
+      extern "C" {
+        extern int duplicatedFunc();
+      }
+      int main() {
+        int res = duplicatedFunc();
+        printf("*%d*\\n", res);
+        return 0;
+      }
+    ''')
+    create_file('duplicated_func_1.js', '''
+      mergeInto(LibraryManager.library, {
+        duplicatedFunc : function() {
+            return 1;
+          }
+        }, LibraryParam.DisallowOverride
+      );
+    ''')
+    create_file('duplicated_func_2.js', '''
+      mergeInto(LibraryManager.library, {
+        duplicatedFunc : function() {
+            return 2;
+          }
+        }, LibraryParam.DisallowOverride
+      );
+    ''')
+
+    self.emcc_args += ['--js-library', 'duplicated_func_1.js', '--js-library', 'duplicated_func_2.js']
+    err = self.expect_fail([EMCC, 'duplicated_func.cpp'] + self.get_emcc_args())
+    self.assertContained('error: Symbol re-definition in JavaScript library: duplicatedFunc. Do not use DisallowOverride if this is intended', err)
+
   def test_js_lib_quoted_key(self):
     create_file('lib.js', r'''
 mergeInto(LibraryManager.library, {