configure.ac: add -Werror to default compiler options

Alexandre Julliard julliard at winehq.org
Fri Sep 19 09:35:45 CDT 2008


"Dan Kegel" <dank at kegel.com> writes:

> Agreed, to an extent.   A user who is trying to compile with a really
> whacky toolchain (say, a C compiler on an Amiga, a mainframe, or a wristwatch)
> should expect some errors, and we should not try to avoid those if they reflect
> real problems that need to be solved before wine can run properly.

You don't need a wacky toolchain. All you need is a slightly old gcc, or
a platform that not everybody has access to, like Mac OS. I periodically
build on Mac OS and check for warnings, and there are always a few that
creep in, like size_t printf formats. Do you really think Wine would be
better off if it failed to build on Mac 90% of the time?  Is printing a
size_t with %d really so dangerous that it needs to break the build?

> And I also feel pretty strongly that compiler warnings have
> some value, and we should pay attention to them.  Right
> now we're plugging our ears and going "la la la la la I can't hear you",
> and that seems a bit careless.   As Wine aims for higher and
> higher quality levels, eventually we will have to change our ways here.

That's absolutely not true. We are trying very hard to avoid warnings,
I don't commit patches that add warnings, and many people are sending
fixes when they find warnings on their platform. That doesn't mean they
should break the build.

> If we do it in the right way, it could end up increasing
> our code quality quite a bit without inconveniencing any users.

It wouldn't make any difference to code quality, since we are already
warnings-free on most standard builds.

> The right way is to slowly fix all the warnings -- like Andrew Talbot
> is doing --
> and slowly encourage all wine developers to crank up their warning
> levels to the max.  Once we've voluntarily cleared out nearly all the
> warnings, we can then have a flag day to clean out all the rest,
> and switch -Werror on.  That will keep errors from creeping back in.

Everybody is free to build their tree with -Werror, but there's no way
that it will become the default.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list