[PATCH] D63975: Warn when ScopeDepthOrObjCQuals overflows

[John McCall via Phabricator via cfe-commits]

rjmccall added inline comments.


================
Comment at: clang/lib/Parse/ParseDecl.cpp:6587
+        return;
+      }
+
----------------
Mordante wrote:
> rjmccall wrote:
> > Comment indentation.
> > 
> > Should we do this when starting to parse a function prototype instead of when parsing a parameter?
> Thanks I noticed I forgot to change the tabs to spaces.
> 
> I looked at your suggestion to move the code, but I think this is the proper place. Now it also validates whether lambdas exceed the limit, else we need to check at two places.
> I'll also add a unit test to test for the lambda.
I don't understand.  I'm just saying to put the check at the top of the function instead of inside the loop.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63975/new/

https://reviews.llvm.org/D63975