Hi! I implemented a warning in parser as suggested here http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20121217/159766.html, e.g. warn in the following case if () { } if () { // probably should have been 'else if' } else { }
Please review! This diagnostic found two places in chromium and another one in firefox source base that are suspicious to have missed else. By the way, there is a name clash between Parser::ParenParseOption::CompoundStmt and CompoundStmt from AST. Maybe the former is worth renaming?
missing-else.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
