[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers
Michel Dänzer
michel at daenzer.net
Mon Jan 18 19:42:09 PST 2016
On 19.01.2016 01:05, Emil Velikov wrote:
>
> A few of those are already implicit with either Wall or Wextra. Both
> of which, imho, are a must have for any serious project.
I think -Wextra is generally too noisy for that, but I guess we're now
deeply in arguing about taste territory.
> But seriously - it makes me think that people are rushed to write the
> code and get it out. Or perhaps a too strong "no warnings" policy ?
> After all warnings are to hint that things can be improved/might be
> wrong. If it looks trivial, just ignore it :-)
One problem with that is that leaving trivial/irrelevant/incorrect
warnings makes it easier to miss important warnings. That being said, I
fully agree that one should resist the urge to just get rid of warnings
in whatever way. (I tend to cringe whenever a commit log says something
along the lines of "fix warning" — a change either fixes a problem which
was pointed out by the warning, or it just silences the warning.)
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
More information about the dri-devel
mailing list