This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: parenthesess around constructor class name


>>>>> "Gerald" == Gerald Pfeifer <pfeifer@dbai.tuwien.ac.at> writes:

    Gerald> On Tue, 13 Feb 2001, Aldy Hernandez wrote:
    >> yet another silly ansi-ism.  Jason has said it's ok.
    >> 
    >> aldy
    >> 
    >> 2001-02-08 Aldy Hernandez <aldyh@redhat.com>
    >> 
    >> * cp/parse.y (component_constructor_declarator): allow optional
    >> parenthesess around constructor class name.

    Gerald> Should this go to the branch as well?

We should try to follow the guidelines in:

  http://gcc.gnu.org/ml/gcc/2001-02/msg00403.html

as clarified by the various follow-ups.  Does this case meet any of
these bullets?

Note that this list does *not* contain "seems like a simple enough
change and does something good".  It could -- but it doesn't.  I'm not
by any means going to be draconian, so if maintainers want to approve
such patches for the brnach that's OK by me.  But, we shouldn't do so
blithely.  Mucking with the C++ parser, for example, can have subtle
side-effects; in this case, we should at least convince ourselves that
the vagaries of bison will not affect things.

--
Mark Mitchell                   mark@codesourcery.com
CodeSourcery, LLC               http://www.codesourcery.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]