Linux-kernel mailing list archive 2000-26,: Re: -Werror in test3-pre2

# Re: -Werror in test3-pre2

Giacomo Catenazzi (cate@neper.ethz.ch)
Fri, 30 Jun 2000 14:14:59 +0200 (CEST)

- Messages sorted by: [ date ][ thread ][ subject ][ author ]
- Next message: Tim Walberg: "weird kernel/ps interaction"
- Previous message: Ulrich Windl: "Re: 2.2.16: VM: do_try_to_free_pages failed"

On 29 Jun 2000, Linus Torvalds wrote:

> In article <395C29BF.30FD1778@isn.net>, Garst R. Reese <reese@isn.net> wrote:
> >I'm just curious to know if any kernel has *ever* compiled with this
> >flag?
> >I don't recall seeing any without at least one "defined but not used".
>
> My kernel certainly compiles.
>
> I checked that every single filesystem can be compiled with the flag.
>
> But yes, the fact that _I_ am anal about not having warnings does not
> mean that everybody else is. I added -Werror to see if people would fix
> some of the things it shows, and I've already gotten some patches. More
> are certainly needed.
>
> I'll probably end up dropping -Werror at some point before the real
> 2.4.0, if for no other reason than the fact that there are lots of gcc
> versions out there that complain about bogus things (the signed/unsigned
> comparison warnings in some gcc versions are basically not possible to
> fix, which is why no "real" gcc version does this warning by default to
> my knowledge).
>
> And anybody who isn't interested in fixing them might as well remove the
> -Werror from their makefile too.
>
> But maybe it will cause some cleanup.
>

Another usefull compiler flag to add is: '-Winline'
It warns if a inline function cannot be inlined. With this option and with
-Werror we can eliminate all 'static inline' and 'extern inline'. And
developers are sure that inline are really inlined.

giacomo

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/

- Next message: Tim Walberg: "weird kernel/ps interaction"
- Previous message: Ulrich Windl: "Re: 2.2.16: VM: do_try_to_free_pages failed"