This is bizarre. I'm glad you found it and fixed the other cases where it appeared throughout the codebase.
-- Sean Silva On Wed, Dec 19, 2012 at 10:17 AM, Dmitri Gribenko <[email protected]> wrote: > Author: gribozavr > Date: Wed Dec 19 11:17:09 2012 > New Revision: 170563 > > URL: http://llvm.org/viewvc/llvm-project?rev=170563&view=rev > Log: > Comment parsing: add a missing 'else'. Found by inspection. > > No testcase because we were just building an extra AST node and eventually > throwing it away, so it did not affect correctness. > > Modified: > cfe/trunk/lib/AST/CommentParser.cpp > > Modified: cfe/trunk/lib/AST/CommentParser.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/CommentParser.cpp?rev=170563&r1=170562&r2=170563&view=diff > ============================================================================== > --- cfe/trunk/lib/AST/CommentParser.cpp (original) > +++ cfe/trunk/lib/AST/CommentParser.cpp Wed Dec 19 11:17:09 2012 > @@ -318,7 +318,7 @@ > PC = S.actOnParamCommandStart(Tok.getLocation(), > Tok.getEndLocation(), > Tok.getCommandID()); > - } if (Info->IsTParamCommand) { > + } else if (Info->IsTParamCommand) { > IsTParam = true; > TPC = S.actOnTParamCommandStart(Tok.getLocation(), > Tok.getEndLocation(), > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
