================
Comment at: lib/Parse/ParseDecl.cpp:4882-4888
@@ -4881,9 +4913,1 @@
- // as part of the parameter-declaration-clause.
- if (Tok.is(tok::ellipsis) && D.getCXXScopeSpec().isEmpty() &&
- !((D.getContext() == Declarator::PrototypeContext ||
- D.getContext() == Declarator::LambdaExprParameterContext ||
- D.getContext() == Declarator::BlockLiteralContext) &&
- NextToken().is(tok::r_paren) &&
- !D.hasGroupingParens() &&
- !Actions.containsUnexpandedParameterPacks(D))) {
SourceLocation EllipsisLoc = ConsumeToken();
----------------
Please keep this existing code structure and just add the check for 'auto' to
the end, or factor out a meaningful part of this into a separate function.
Adding a page full of semi-unrelated code into the middle of a function like
this makes it harder to read, and the code communicates its intent well enough
without the comments.
http://reviews.llvm.org/D6520
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
