r203800 - Remove an unnecessary check for FormatTok not being null.
Alexander Kornienko
alexfh at google.com
Thu Mar 13 06:59:48 PDT 2014
Author: alexfh
Date: Thu Mar 13 08:59:48 2014
New Revision: 203800
URL: http://llvm.org/viewvc/llvm-project?rev=203800&view=rev
Log:
Remove an unnecessary check for FormatTok not being null.
Summary:
This check hints clang analyzer, that FormatTok may be null, and it
reports a null pointer dereference error:
http://buildd-clang.debian.net/scan-build/report-827c64.html#Path28
Reviewers: djasper
Reviewed By: djasper
CC: cfe-commits, klimek
Differential Revision: http://llvm-reviews.chandlerc.com/D3040
Modified:
cfe/trunk/lib/Format/UnwrappedLineParser.cpp
Modified: cfe/trunk/lib/Format/UnwrappedLineParser.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/UnwrappedLineParser.cpp?rev=203800&r1=203799&r2=203800&view=diff
==============================================================================
--- cfe/trunk/lib/Format/UnwrappedLineParser.cpp (original)
+++ cfe/trunk/lib/Format/UnwrappedLineParser.cpp Thu Mar 13 08:59:48 2014
@@ -770,7 +770,7 @@ bool UnwrappedLineParser::tryToParseLamb
if (!tryToParseLambdaIntroducer())
return false;
- while (FormatTok && FormatTok->isNot(tok::l_brace)) {
+ while (FormatTok->isNot(tok::l_brace)) {
if (FormatTok->isSimpleTypeSpecifier()) {
nextToken();
continue;
@@ -1369,6 +1369,7 @@ void UnwrappedLineParser::readToken() {
bool CommentsInCurrentLine = true;
do {
FormatTok = Tokens->getNextToken();
+ assert(FormatTok);
while (!Line->InPPDirective && FormatTok->Tok.is(tok::hash) &&
(FormatTok->HasUnescapedNewline || FormatTok->IsFirst)) {
// If there is an unfinished unwrapped line, we flush the preprocessor
More information about the cfe-commits
mailing list