[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2015-04-23 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 Marek Polacek changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED CC|

[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2014-06-17 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 --- Comment #8 from Manuel López-Ibáñez --- (In reply to Manuel López-Ibáñez from comment #7) > In fact, the main show-stopper for adding -Wlogical-op to -Wextra (or -Wall) > is PR40172, which was the reason it was moved out of -Wextra in the fir

[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2014-06-10 Thread P at draigBrady dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 Bug 53131 depends on bug 43772, which changed state. Bug 43772 Summary: Errant -Wlogical-op warning when testing limits https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43772 What|Removed |Added

[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2012-05-07 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 --- Comment #7 from Manuel López-Ibáñez 2012-05-07 13:42:10 UTC --- In fact, the main show-stopper for adding -Wlogical-op to -Wextra (or -Wall) is PR40172, which was the reason it was moved out of -Wextra in the first place. Someone would need

[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2012-04-28 Thread marc.glisse at normalesup dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 --- Comment #6 from Marc Glisse 2012-04-28 12:45:19 UTC --- (In reply to comment #5) > It seems a pretty small warning, but I guess #1 and #2 could > be split up, if that helps get #2 in. I think it is the opposite actually, #2 is more controver

[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2012-04-28 Thread dcb314 at hotmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 --- Comment #5 from dcb 2012-04-28 12:17:57 UTC --- (In reply to comment #4) > Could you give a sample? -Wlogical-op merges 2 unrelated warnings: > *) x && 2 (you would expect a boolean, not 2, so maybe x&2 was meant) > *) x<0 && x>0 (not so likel

[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2012-04-28 Thread marc.glisse at normalesup dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 Marc Glisse changed: What|Removed |Added CC||marc.glisse at normalesup

[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2012-04-28 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 --- Comment #3 from Manuel López-Ibáñez 2012-04-28 10:37:04 UTC --- (In reply to comment #2) > > > > Do the warnings indicate bugs or not? > > Yes. I checked the first ten. And what do the kernel people say? Do they want Wlogical-op enabled?

[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2012-04-28 Thread dcb314 at hotmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 --- Comment #2 from dcb 2012-04-28 07:29:17 UTC --- (In reply to comment #1) > (In reply to comment #0) > > I am somewhat surprised that -Wlogical-op isn't part of either -Wall > > or possibly -Wextra. > > It could be in -Wextra for GCC 4.8. Rig

[Bug c/53131] -Wlogical-op: ready for prime time in -Wall ?

2012-04-26 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131 Manuel López-Ibáñez changed: What|Removed |Added Keywords||diagnostic CC|