Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Disable -Wsuggest-final-xxx in the headers for now

Browse Source 
We don't use "final" in our code, as very few classes in wx code have
virtual functions but are not meant to be derived from in the user code.

Ideal would be to check the existing warnings and maybe apply "final" if
it's relevant and disable it otherwise, as these warnings can be useful
to build the application code with, but for now just disable them in the
test suite.
This commit is contained in:
Vadim Zeitlin
2020-08-30 23:52:32 +02:00
parent 3fc5d134a3
commit 6f9390306d
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tests/allheaders.cpp
View File

@@ -368,6 +368,13 @@
GCC_TURN_OFF(zero-as-null-pointer-constant) GCC_TURN_OFF(zero-as-null-pointer-constant)
#endif #endif
// These ones could be useful to explore, but for now we don't use "final"
// at all anywhere.
#if CHECK_GCC_VERSION(5,1)
GCC_TURN_OFF(suggest-final-methods)
GCC_TURN_OFF(suggest-final-types)
#endif // 5.1
// wxWARNING_SUPPRESS_MISSING_OVERRIDE() inside wxRTTI macros is just // wxWARNING_SUPPRESS_MISSING_OVERRIDE() inside wxRTTI macros is just
// ignored by gcc up to 9.x for some reason, so we have no choice but to // ignored by gcc up to 9.x for some reason, so we have no choice but to
// disable it. // disable it.