[cfe-commits] r147837 - in /cfe/trunk: include/clang/AST/DeclCXX.h lib/Sema/Sema.cpp

[John McCall]

On Jan 9, 2012, at 8:44 PM, Eli Friedman wrote:
> On Mon, Jan 9, 2012 at 8:24 PM, John McCall <rjmccall at apple.com> wrote:
>> On Jan 9, 2012, at 7:38 PM, Eli Friedman wrote:
>>> Author: efriedma
>>> Date: Mon Jan  9 21:38:46 2012
>>> New Revision: 147837
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=147837&view=rev
>>> Log:
>>> Per John's comment, it makes sense to ask isLambda on any CXXRecordDecl; make sure that's safe.  Get rid of a check which is now unnecessary in Sema::getFunctionLevelDeclContext().
>> 
>> You could also just start the definition as soon as you start processing the lambda expression.
> 
> I do; the extra check is necessary to handle non-lambda CXXRecordDecls
> without a definition.

Oh, of course.  Sorry for the confusion.

John.