On 4/28/12 6:28 PM, WALTER LITWINCZYK (RIT Student) wrote:
[...] I will note -Werror does make it difficult to do code->compile->test->repeat, but it's definitely better to have on than off
Sean and I have mixed feelings about this as well. There's been plenty of "YES, I know that variable's unused, I just haven't gotten around to it!" On the other hand, things such as passing a 'struct foo*' to a function that expects a 'struct bar*' is only considered worth a warning, while I think it definitely deserves an error.
For the specific case of unused variables (whose warnings are generated by -Wextra) we could make a way to disable -Wextra or -Werror with options to make (e.g. make WERROR=false WEXTRA=false) with the default being to enable them. Do you think that this would help?
-David