On Wed, Sep 25, 2013 at 12:27:14AM +0200, Colomban Wendling wrote:
[...]
Other suggestions:
- enable -Wall -Wextra -Werror in compiler
Yes, kinda advertized in HACKING -- but -Werror is harder to use because you need to check your flags VERY carefully not to produce any false positive. E.g. -Wwrite-strings have a few false positive due to imperfect API. But yeah, it's nice to stop on possible errors. FTR, we
s/imperfect/broken/
It's not acceptable to modify const strings. Ever.
(devs) all (I believe) use this or stricter options (but maybe not with -Werror). [...]