Re: r313521 - Another attempt to fix warning discovered by r313487. [-Wunused-lambda-capture]

2017-09-18 Thread Faisal Vali via cfe-commits 
thanks!
Faisal Vali



On Mon, Sep 18, 2017 at 4:10 AM, NAKAMURA Takumi  wrote:
> Seems fixed! Thanks!
>
> On Mon, Sep 18, 2017 at 5:27 PM Vitaly Buka via cfe-commits
>  wrote:
>>
>> Author: vitalybuka
>> Date: Mon Sep 18 01:26:01 2017
>> New Revision: 313521
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=313521&view=rev
>> Log:
>> Another attempt to fix warning discovered by r313487.
>> [-Wunused-lambda-capture]
>>
>> Modified:
>> cfe/trunk/lib/Lex/PPLexerChange.cpp
>>
>> Modified: cfe/trunk/lib/Lex/PPLexerChange.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/PPLexerChange.cpp?rev=313521&r1=313520&r2=313521&view=diff
>>
>> ==
>> --- cfe/trunk/lib/Lex/PPLexerChange.cpp (original)
>> +++ cfe/trunk/lib/Lex/PPLexerChange.cpp Mon Sep 18 01:26:01 2017
>> @@ -42,7 +42,7 @@ bool Preprocessor::isInPrimaryFile() con
>>   "Top level include stack isn't our primary lexer?");
>>return std::none_of(
>>IncludeMacroStack.begin() + 1, IncludeMacroStack.end(),
>> -  [this](const IncludeStackInfo &ISI) -> bool { return
>> IsFileLexer(ISI); });
>> +  [&](const IncludeStackInfo &ISI) -> bool { return IsFileLexer(ISI);
>> });
>>  }
>>
>>  /// getCurrentLexer - Return the current file lexer being lexed from.
>> Note
>>
>>
>> ___
>> cfe-commits mailing list
>> cfe-commits@lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r313521 - Another attempt to fix warning discovered by r313487. [-Wunused-lambda-capture]

2017-09-18 Thread NAKAMURA Takumi via cfe-commits 
Seems fixed! Thanks!

On Mon, Sep 18, 2017 at 5:27 PM Vitaly Buka via cfe-commits <
cfe-commits@lists.llvm.org> wrote:

> Author: vitalybuka
> Date: Mon Sep 18 01:26:01 2017
> New Revision: 313521
>
> URL: http://llvm.org/viewvc/llvm-project?rev=313521&view=rev
> Log:
> Another attempt to fix warning discovered by r313487.
> [-Wunused-lambda-capture]
>
> Modified:
> cfe/trunk/lib/Lex/PPLexerChange.cpp
>
> Modified: cfe/trunk/lib/Lex/PPLexerChange.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/PPLexerChange.cpp?rev=313521&r1=313520&r2=313521&view=diff
>
> ==
> --- cfe/trunk/lib/Lex/PPLexerChange.cpp (original)
> +++ cfe/trunk/lib/Lex/PPLexerChange.cpp Mon Sep 18 01:26:01 2017
> @@ -42,7 +42,7 @@ bool Preprocessor::isInPrimaryFile() con
>   "Top level include stack isn't our primary lexer?");
>return std::none_of(
>IncludeMacroStack.begin() + 1, IncludeMacroStack.end(),
> -  [this](const IncludeStackInfo &ISI) -> bool { return
> IsFileLexer(ISI); });
> +  [&](const IncludeStackInfo &ISI) -> bool { return IsFileLexer(ISI);
> });
>  }
>
>  /// getCurrentLexer - Return the current file lexer being lexed from.
> Note
>
>
> ___
> cfe-commits mailing list
> cfe-commits@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r313521 - Another attempt to fix warning discovered by r313487. [-Wunused-lambda-capture]

2017-09-18 Thread Vitaly Buka via cfe-commits 
Author: vitalybuka
Date: Mon Sep 18 01:26:01 2017
New Revision: 313521

URL: http://llvm.org/viewvc/llvm-project?rev=313521&view=rev
Log:
Another attempt to fix warning discovered by r313487. [-Wunused-lambda-capture]

Modified:
cfe/trunk/lib/Lex/PPLexerChange.cpp

Modified: cfe/trunk/lib/Lex/PPLexerChange.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/PPLexerChange.cpp?rev=313521&r1=313520&r2=313521&view=diff
==
--- cfe/trunk/lib/Lex/PPLexerChange.cpp (original)
+++ cfe/trunk/lib/Lex/PPLexerChange.cpp Mon Sep 18 01:26:01 2017
@@ -42,7 +42,7 @@ bool Preprocessor::isInPrimaryFile() con
  "Top level include stack isn't our primary lexer?");
   return std::none_of(
   IncludeMacroStack.begin() + 1, IncludeMacroStack.end(),
-  [this](const IncludeStackInfo &ISI) -> bool { return IsFileLexer(ISI); 
});
+  [&](const IncludeStackInfo &ISI) -> bool { return IsFileLexer(ISI); });
 }
 
 /// getCurrentLexer - Return the current file lexer being lexed from.  Note


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits