[Bug c/49975] warn on malformed __attribute((...))__

2011-08-04 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49975 --- Comment #6 from Jonathan Wakely 2011-08-04 13:56:46 UTC --- Google codesearch shows __attribute is used widely in the android kernel and many other places. Whitespace is not significant in C, it would be strange for "))foo" to be different t

[Bug c/49975] warn on malformed __attribute((...))__

2011-08-04 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49975 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|

[Bug c/49975] warn on malformed __attribute((...))__

2011-08-04 Thread ext at sidvind dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49975 --- Comment #4 from David Sveningsson 2011-08-04 13:48:55 UTC --- (In reply to comment #3) > It doesn't make sense to require whitespace after a paren, that will likely > break a lot of programs. In this quite specific case? "__attribute((..))fo

[Bug c/49975] warn on malformed __attribute((...))__

2011-08-04 Thread sch...@linux-m68k.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49975 --- Comment #3 from Andreas Schwab 2011-08-04 13:45:13 UTC --- It doesn't make sense to require whitespace after a paren, that will likely break a lot of programs.

[Bug c/49975] warn on malformed __attribute((...))__

2011-08-04 Thread ext at sidvind dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49975 --- Comment #2 from David Sveningsson 2011-08-04 13:33:21 UTC --- Might be valid, but confusing. If whitespace were required after "))" it would make more sense as it is clear that a global is created. As the documentation states that __attribut

[Bug c/49975] warn on malformed __attribute((...))__

2011-08-04 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49975 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #1 f