clean release build output of benign warnings: -Wempty-body, -Wmaybe-unitialized, -Wunused-variable, -Wstrict-overflow
S4 - Minor
I get these same sets of warnings on my x86-64 and ARMv7 arch linux machines when building RELEASE:
- The two -Wempty-body deal with a Q_ASSERT statement inside of an else that doesn't have brackets. Release build removes that assert, and hence nothing is inside the else.
- For the -Wmaybe-uninitialized I need your help to figure out reasonable initial defaults.
- The -Wunused-variable refers to Ms::updatePeriods in preferences.cpp, which I'm going to comment out of the code, so incase someone later wants to return to using those strings, then it will still be in the code, just commented out.
- The -Wstrict-overflow I need to figure out the best way to tell the compiler that it is ok to not be concerned about the overflow. Either suppress the warning in makefile or change something in the code...
After I remove those warnings, then I should have a clean release build.