Bug 1010634, Part 6: Enable -Wall with a few exceptions for certverifier, r=cviecco

--HG--
extra : rebase_source : 611f0d65e7edb74345a4a599a6606de37e3da75e

security/certverifier/moz.build

@@ -26,6 +26,39 @@ DIRS += [
     '../pkix',
 ]
 
+CXXFLAGS += ['-Wall']
+if CONFIG['_MSC_VER']:
+  # -Wall with Visual C++ enables too many problematic warnings
+  CXXFLAGS += [
+    '-wd4510', # default constructor could not be generated
+    '-wd4512', # assignment operator could not be generated
+    '-wd4514', # 'function': unreferenced inline function has been removed
+    '-wd4610', # struct 'symbol' can never be instantiated - user defined
+               # constructor required
+    '-wd4619', # pragma warning: there is no warning 'warning'
+    '-wd4625', # copy constructor could not be generated because a base
+               # class copy constructor is inaccessible or deleted
+    '-wd4626', # assignment operator could not be generated because a base
+               # class assignment operator is inaccessible or deleted
+    '-wd4710', # 'function': function not inlined
+    '-wd4711', # function 'function' selected for inline expansion
+    '-wd4820', # 'bytes' bytes padding added after construct 'member_name'
+  ]
+
+  # MSVC 2010's headers trigger these
+  CXXFLAGS += [
+      '-wd4548', # expression before comma has no effect; ...
+      '-wd4668', # 'symbol' is not defined as a preprocessor macro...
+      '-wd4987', # nonstandard extension used
+  ]
+
+  # Gecko headers aren't warning-free enough for us to enable these warnings
+  CXXFLAGS += [
+    '-wd4100', # 'symbol' : unreferenced formal parameter
+    '-wd4127', # conditional expression is constant
+    '-wd4946', # reinterpret_cast used between related types
+  ]
+
 FAIL_ON_WARNINGS = True
 
 LIBRARY_NAME = 'certverifier'