Re: [PATCH] Add testcase for c++/68116

[Jason Merrill <jason at redhat dot com>]

On 12/08/2015 06:54 AM, Bernd Schmidt wrote:
> On 12/07/2015 06:49 PM, Marek Polacek wrote:
>
> > diff --git gcc/testsuite/g++.dg/cpp0x/pr68116.C
> > gcc/testsuite/g++.dg/cpp0x/pr68116.C
> > index e69de29..04ed901 100644
> > --- gcc/testsuite/g++.dg/cpp0x/pr68116.C
> > +++ gcc/testsuite/g++.dg/cpp0x/pr68116.C
> > @@ -0,0 +1,12 @@
> > +// PR c++/68116
> > +// { dg-do compile { target c++11 } }
> > +
> > +class C {
> > +  void foo ();
> > +  typedef void (C::*T) (int);
> > +  static T b[];
> > +};
> > +C::T C::b[]
> > +{
> > +  T (&C::foo)
> > +};
>
> The problem I have with approving C++ testcases is that I have no idea
> whether this is valid or not or what it expresses. You should Cc Jason
> (which I've now done).

That's odd code--I don't approve of the cast in the initializer--but it 
is well-formed.  OK.

Jason