Revert "Adjust handling of -Wsuggest-override"

This reverts commit e7e4af869f7144fa9a265ccf1e15c7b2f182ac2c.

I misunderstood what was going on, and we were just missing some
DCPOMATIC_{DIS,EN}ABLE_WARNINGS calls.

diff --git a/src/lib/warnings.h b/src/lib/warnings.h
index c6c2f7e0c89e4e83c2e480b08e06c6fb73ab19e1..ca8a9ab5e5ee209c87aa4aa53e4f91b36c7fbfec 100644 (file)
--- a/src/lib/warnings.h
+++ b/src/lib/warnings.h
@@ -35,7 +35,8 @@
   _Pragma("GCC diagnostic ignored \"-Wmissing-field-initializers\"") \
   _Pragma("GCC diagnostic ignored \"-Wunused-parameter\"") \
   _Pragma("GCC diagnostic ignored \"-Waddress\"") \
-  _Pragma("GCC diagnostic ignored \"-Wparentheses\"")
+  _Pragma("GCC diagnostic ignored \"-Wparentheses\"") \
+  _Pragma("GCC diagnostic ignored \"-Wsuggest-override\"")
 #endif
 
 #define DCPOMATIC_ENABLE_WARNINGS \

diff --git a/wscript b/wscript
index 994f1b6f3e7aee7c90de95c21dba1272849f3cff..4ccadf6fd3dfdfef13b91d3d6389d2f5e169aae4 100644 (file)
--- a/wscript
+++ b/wscript
@@ -109,6 +109,7 @@ def configure(conf):
                                        '-Wall',
                                        '-Wextra',
                                        '-Wwrite-strings',
+                                       '-Wsuggest-override',
                                        '-Wno-error=deprecated',
                                        # I tried and failed to ignore these with _Pragma
                                        '-Wno-ignored-qualifiers',
@@ -126,10 +127,6 @@ def configure(conf):
         if int(gcc[0]) >= 8:
             # I tried and failed to ignore these with _Pragma
             conf.env.append_value('CXXFLAGS', ['-Wno-cast-function-type'])
-        if int(gcc[0]) >= 9:
-            # I can't get #pragma GCC diagnostic ignored "-Wsuggest-override" to work on older GCCs,
-            # so only enable this if we're on a newer one
-            conf.env.append_value('CXXFLAGS', '-Wsuggest-override')
         have_c11 = int(gcc[0]) >= 4 and int(gcc[1]) >= 8 and int(gcc[2]) >= 1
         # Most gccs still give these warnings from boost::optional
         conf.env.append_value('CXXFLAGS', ['-Wno-maybe-uninitialized'])