warn on missing symbols by default; fixes #1794

1 files changed, 8 insertions, 5 deletions

diff --git a/src/settings.js b/src/settings.js
index 0d8d3da5..bc665973 100644
--- a/src/settings.js
+++ b/src/settings.js
@@ -390,13 +390,16 @@ var FAKE_X86_FP80 = 1; // Replaces x86_fp80 with double. This loses precision. I
 
 var GC_SUPPORT = 1; // Enables GC, see gc.h (this does not add overhead, so it is on by default)
 
-var WARN_ON_UNDEFINED_SYMBOLS = 0; // If set to 1, we will warn on any undefined symbols that
-                                   // are not resolved by the library_*.js files. We by default
-                                   // do not warn because (1) it is normal in large projects to
+var WARN_ON_UNDEFINED_SYMBOLS = 1; // If set to 1, we will warn on any undefined symbols that
+                                   // are not resolved by the library_*.js files. Note that
+                                   // it is common in large projects to
                                    // not implement everything, when you know what is not
                                    // going to actually be called (and don't want to mess with
-                                   // the existing buildsystem), and (2) functions might be
-                                   // implemented later on, say in --pre-js
+                                   // the existing buildsystem), and functions might be
+                                   // implemented later on, say in --pre-js, so you may
+                                   // want to build with -s WARN_ON_UNDEFINED_SYMBOLS=0 to
+                                   // disable the warnings if they annoy you.
+                                   // See also ERROR_ON_UNDEFINED_SYMBOLS
 
 var ERROR_ON_UNDEFINED_SYMBOLS = 0; // If set to 1, we will give a compile-time error on any
                                     // undefined symbols (see WARN_ON_UNDEFINED_SYMBOLS).