> On 27 Oct 2016, at 13:25, Erik Verbruggen <erik.verbrug...@me.com> wrote: > > Eh? Preprocessor::IsFileLexer(const IncludeStackInfo &) is static and doesn't > need 'this'... Apparently MSVC mistakingly confuses that method with the > non-static Preprocessor::IsFileLexer() method?
You’re not kidding, its incredibly annoying. I can add a comment explaining why its there if you wish? Simon >> On 27 Oct 2016, at 12:51, Simon Pilgrim via cfe-commits >> <cfe-commits@lists.llvm.org> wrote: >> >> Author: rksimon >> Date: Thu Oct 27 05:51:29 2016 >> New Revision: 285281 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=285281&view=rev >> Log: >> Fix MSVC warning about missing 'this' from lambda default capture mode >> >> 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=285281&r1=285280&r2=285281&view=diff >> ============================================================================== >> --- cfe/trunk/lib/Lex/PPLexerChange.cpp (original) >> +++ cfe/trunk/lib/Lex/PPLexerChange.cpp Thu Oct 27 05:51:29 2016 >> @@ -41,7 +41,7 @@ bool Preprocessor::isInPrimaryFile() con >> assert(IsFileLexer(IncludeMacroStack[0]) && >> "Top level include stack isn't our primary lexer?"); >> return std::none_of(IncludeMacroStack.begin() + 1, IncludeMacroStack.end(), >> - [](const IncludeStackInfo &ISI) -> bool { >> + [this](const IncludeStackInfo &ISI) -> bool { >> return IsFileLexer(ISI); >> }); >> } >> >> >> _______________________________________________ >> 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