This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [C++] PATCH c++/17542
- From: Gabriel Dos Reis <gdr at cs dot tamu dot edu>
- To: Matt Austern <austern at apple dot com>
- Cc: "gcc-patches at gcc dot gnu dot org Patches" <gcc-patches at gcc dot gnu dot org>
- Date: 29 Oct 2004 10:33:30 -0500
- Subject: Re: [C++] PATCH c++/17542
- Organization: Texas A&M University, Department of Computer Science
- References: <9A5B94DA-29BD-11D9-84FF-000A95BCF344@apple.com>
Matt Austern <austern@apple.com> writes:
| This patch issues a warning when users put an attribute in a place
| that looks reasonable but is in fact wrong. (The documentation
| already says so.) And if you're wondering why we don't just accept
| attributes in this position... Well, in a sense the problem is that
| we already do. An attribute before the class head is perfectly legal,
| it just means something different. It applies to the declarator,
| which in this case is empty. This is legal code that doesn't do what
| the user probably thought it did, so a warning is what's called for.
|
| OK to commit to mainline?
>From diagnostic perspective, yes it is OK.
-- Gaby