This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy objc-property-declaration] Expand list of ObjC acronyms
ClosedPublic

Authored by benhamilton on Jan 18 2018, 11:10 AM.

Details

Summary

We were missing some pretty common acronyms in the camelCase
property name check objc-property-declaration.

This expands the list and sorts it lexicographically, so we can
avoid duplicates.

Test Plan: make -j12 check-clang-tools

Diff Detail

Event Timeline

benhamilton created this revision.Jan 18 2018, 11:10 AM
Wizard accepted this revision.Jan 18 2018, 11:35 AM
This revision is now accepted and ready to land.Jan 18 2018, 11:35 AM

Can you update the doc btw since the acronyms are not only for prefix anymore?

  • Added comment about prefixes and suffixes.
  • Updated list of acronyms.
This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.