This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] add more aliases for the hicpp module
ClosedPublic

Authored by JonasToth on Sep 9 2017, 6:18 AM.

Details

Summary

This patch will introduce even more aliases for the hicpp-module to already existing
checks and is a follow up for D30383 finishing the other sections.
It fixes a forgotten highlight in hicpp-braces-around-statements.rst, too.

Event Timeline

JonasToth created this revision.Sep 9 2017, 6:18 AM
JonasToth updated this revision to Diff 114485.Sep 9 2017, 6:22 AM
  • fix minor doc errors
alexfh accepted this revision.Sep 10 2017, 5:54 AM

Looks reasonable. One comment inline.

docs/ReleaseNotes.rst
145

I'd probably list the aliases here.

This revision is now accepted and ready to land.Sep 10 2017, 5:54 AM
JonasToth updated this revision to Diff 114547.Sep 11 2017, 2:05 AM
  • [Doc] add list of aliased checks in release notes
JonasToth marked an inline comment as done.Sep 11 2017, 2:05 AM
JonasToth closed this revision.Sep 11 2017, 2:10 AM