This is an archive of the discontinued LLVM Phabricator instance.

Force braces on the else branch if they are being added to the if branch.
ClosedPublic

Authored by sbenza on Mar 30 2015, 1:33 PM.

Diff Detail

Event Timeline

sbenza updated this revision to Diff 22905.Mar 30 2015, 1:33 PM
sbenza retitled this revision from to Force braces on the else branch if they are being added to the if branch..
sbenza updated this object.
sbenza edited the test plan for this revision. (Show Details)
sbenza added a reviewer: alexfh.
sbenza added a subscriber: Unknown Object (MLST).
alexfh accepted this revision.Mar 31 2015, 2:43 AM
alexfh edited edge metadata.

Awesome! Thanks for the fix!

clang-tidy/readability/BracesAroundStatementsCheck.h
46

Maybe add a comment describing the meaning of the return value?

This revision is now accepted and ready to land.Mar 31 2015, 2:43 AM
sbenza updated this revision to Diff 22952.Mar 31 2015, 6:51 AM
sbenza edited edge metadata.

Added comment

This revision was automatically updated to reflect the committed changes.