Page MenuHomePhabricator

[clang-tidy] Move google-readability-function check to readability-named-parameter.
ClosedPublic

Authored by alexfh on Mar 11 2015, 11:10 AM.

Details

Summary

The relevant style rule is going to be removed, thus the check is no longer needed in the Google module. Leaving the check in readability/ in case someone needs it.

Diff Detail

Event Timeline

alexfh updated this revision to Diff 21747.Mar 11 2015, 11:10 AM
alexfh retitled this revision from to [clang-tidy] Remove google-readability-function check..
alexfh updated this object.
alexfh edited the test plan for this revision. (Show Details)
alexfh added a reviewer: djasper.
alexfh added a subscriber: Unknown Object (MLST).
djasper accepted this revision.Mar 16 2015, 7:54 AM
djasper edited edge metadata.

Looks good. I guess we can move it to misc if somebody would want to have this check.

This revision is now accepted and ready to land.Mar 16 2015, 7:54 AM

Looks good. I guess we can move it to misc if somebody would want to have this check.

It's better suitable for readability/. I'll move it there just in case.

alexfh updated this revision to Diff 22059.Mar 16 2015, 3:28 PM
alexfh edited edge metadata.

Move google-readability-function to readability/ and rename it to readability-named-parameter.

alexfh retitled this revision from [clang-tidy] Remove google-readability-function check. to [clang-tidy] Move google-readability-function check to readability-named-parameter..Mar 16 2015, 3:33 PM
alexfh updated this object.
alexfh closed this revision.Mar 16 2015, 3:33 PM