This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] add an alias hicpp-braces-around-statements
ClosedPublic

Authored by JonasToth on Aug 10 2017, 6:24 AM.

Details

Summary

This is an alias into the readability-module and enforces rule 6.1.1 of the High Integrity C++ Standard.

Event Timeline

JonasToth created this revision.Aug 10 2017, 6:24 AM
JonasToth retitled this revision from [clang-tidy] add an alias hicpp-braces-around-statements This is an alias into the readability-module and enforces rule `6.1.1 of the High Integrity C++ Standard <http://www.codingstandard.com/rule/6-1-1-enclose-the-body-of-a-selection-or-an... to [clang-tidy] add an alias hicpp-braces-around-statements.Aug 10 2017, 6:25 AM
JonasToth edited the summary of this revision. (Show Details)
JonasToth edited the summary of this revision. (Show Details)
JonasToth added a project: Restricted Project.
alexfh accepted this revision.Aug 10 2017, 8:33 AM
This revision is now accepted and ready to land.Aug 10 2017, 8:33 AM
JonasToth added a comment.EditedAug 10 2017, 8:35 AM

could you commit it please, i dont have rights?

Could you rebase the patch? It doesn't apply cleanly.

hokein added inline comments.Aug 10 2017, 9:31 AM
docs/clang-tidy/checks/hicpp-braces-around-statements.rst
13

nit: remove this extra empty line.

JonasToth updated this revision to Diff 110601.Aug 10 2017, 9:50 AM
  • [Misc] fix nit
JonasToth marked an inline comment as done.Aug 10 2017, 9:50 AM
aaron.ballman closed this revision.Aug 11 2017, 5:13 AM
aaron.ballman edited edge metadata.

I've commit as r310707.