https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53431

--- Comment #16 from Jeffrey Walton <noloader at gmail dot com> ---
(In reply to Jonathan Wakely from comment #15)
> (In reply to Jeffrey Walton from comment #13)
> > This issued caused Crypto++ to remove -Wall (and above) under GCC.
> 
> That seems to be throwing the baby out with the bathwater.

Yeah, for us its a calculated risk. We had other compilers available that
worked as expected, so we could lower the bar for GCC. The residual risk is the
set of issues GCC will catch minus the set of issues the other compilers will
catch. Hopefully, the set is the empty set.

> Why not simply use
> 
>  -Wall -Wno-unused-value -Wno-unused-variable
> 
We don't want to pollute the command line.

All in all, the best solution for us from a usability and engineering
standpoint is to have the GCC diagnostic block work as advertised. Then, we
manage our warnings in our headers and source code through the GCC block, and
they don't cross pollinate into the user's code.

And the users of the library do not need to worry about disabling warnings on
the command line. Without GCC diagnostic blocks, users of the library must
guess at what needs to be on the command line when compiling and linking
against the library. And then they wonder why we did not take the time to clear
or silence the warning.

Reply via email to