This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] add camelBackOrCase casing style to readability-identifier-naming to support change to variable naming policy (if adopted)
AbandonedPublic

Authored by MyDeveloperDay on Feb 8 2019, 11:31 AM.

Details

Summary

Introduction of a new variable policy (as outlined in D57896: Variable names rule) could lead to clang tidy complaining about 1000's of variables

Consider add a new casing style to readability-identifier-naming to support allowing either camelBack and camelCase styles which could be applied to the VariableCase option in the LLVM .clang-tidy files

Users wishing to convert their files to new policy would simply change this setting to camelBack and run with -fix

Depends on D57896

Diff Detail

Event Timeline

MyDeveloperDay created this revision.Feb 8 2019, 11:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 8 2019, 11:31 AM
Herald added a subscriber: xazax.hun. · View Herald Transcript
Eugene.Zelenko added inline comments.
docs/clang-tidy/checks/readability-identifier-naming.rst
30

Please fix double space.

JonasToth added inline comments.Feb 14 2019, 6:22 AM
test/clang-tidy/readability-identifier-naming-camelback-or-case.cpp
97

why no test for fixing?

MyDeveloperDay abandoned this revision.Jun 20 2019, 11:17 AM