simplify gcc warning flags
Currently we have the following compiler warning flags for gcc:
-Wall -Wno-error -Wno-reorder -Wno-unused-local-typedefs -Werror=unused-variable -std=c++11
I would like to remove the following flags if no one objects:
-Wno-error: already default so no need to declare it.
-Wno-reorder: suppresses a warning that is easy to fix and can (admittedly rarely) lead to non-intuitive code execution.
-Wno-unused-local-typedefs: suppresses a warning that is easy to fix and usually points to uncleaned code or wrong logic.
Furthermore, there are very few code changes needed to fix the code (which I will do).