On 2012-03-08 19:16, Brandon Falk wrote:
> Is there anyone actively working on cleaning up the codebase to not throw
> warnings when built with clang. I realize these warnings do not matter, but I
> generally prefer to get rid of warnings if possible. I can't imagine it would
> take too long, although I could see an issue with developers not familiar with
> the clang warning system forgetting to check clang builds for warnings while
> they add new code.

Take care saying "these warnings do not matter".  Unfortunately, it is
not as simple as that.

In some cases, the warnings are indeed superfluous, and it is rather
annoying to fix them (a.k.a "shut up the compiler").  In other cases,
however, the warnings point out real problems.

Obviously, those problems can be fixed, but sometimes that is either
very hard, or very disruptive, or both.  And there are even cases where
the problem may be very subtle, and cannot be fixed without extremely
careful inspection (and retrospection).

Last but not least, applying fixes may break code in unexpected ways.
Especially if the code has been sitting untouched for years, it is prone
to be sensitive to even the smallest changes...

One big issue is that there are no tinderbox builds that use clang, and
therefore people don't notice that commits introduce new warnings (or
errors, even) when building with clang.  This is something that should
be fixed sooner rather than later.

On the other hand, if -CURRENT would be switched to use clang by
default, the problems would hopefully be taken care of sooner. :)
_______________________________________________
freebsd-hackers@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org"

Reply via email to