[bikeshed] -Werror pains

  • From: David Larsen <dcl9934@xxxxxxx>
  • To: "WALTER LITWINCZYK (RIT Student)" <wtl4112@xxxxxxx>, Bikeshed Development <bikeshed@xxxxxxxxxxxxx>
  • Date: Sat, 28 Apr 2012 22:27:09 -0400

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

Other related posts:

  • » [bikeshed] -Werror pains - David Larsen