On Oct 24, 2011, at 10:59 PM, David Blaikie wrote:

> Here's a fairly non-intrusive fix for PR11216: "Can't derive from a
> decltype-specifier" including a couple of test cases & some updated
> naming/comments.
> 
> Let me know if this looks OK & I'll check it in.

It looks pretty good, but I have two requests:

1) The "else" that handles class-names is really long. I think the flow would 
be better if decltype() was handled completely in its own "if" early on, since 
it shares so little code with the class-name case.

2) Please add some tests that have a dependent decltype expression, to make 
sure template instantiation works. I'm sure it will, but...

        - Doug

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to