This is an archive of the discontinued LLVM Phabricator instance.

add support for constants with appropriate prefix. Start with 'k' is not the only option. This is according to http://google.github.io/styleguide/objcguide.html#constants
AbandonedPublic

Authored by Wizard on Feb 22 2018, 11:10 AM.

Details

Summary

Supporting static constant name with appropriate prefix like ABCMyConstantVariable.

Event Timeline

Wizard created this revision.Feb 22 2018, 11:10 AM
Wizard edited the summary of this revision. (Show Details)Feb 22 2018, 11:12 AM
Wizard added reviewers: hokein, benhamilton.
Wizard updated this revision to Diff 135485.Feb 22 2018, 11:13 AM

fix format

I have this change out for review as well:
https://reviews.llvm.org/D43581

I have this change out for review as well:
https://reviews.llvm.org/D43581

Ah cool. Your change looks good. Technically the same as mine. I will discard this diff.

Wizard abandoned this revision.Feb 22 2018, 11:34 AM