This change will support cases like:
@property(assign, nonatomic) int ID;
Differential D46374
Add support for ObjC property name to be a single acronym. Wizard on May 2 2018, 6:44 PM. Authored by
Details This change will support cases like: @property(assign, nonatomic) int ID;
Diff Detail
Event Timeline
|
s is a regular expression here, so you need to match it using llvm::Regex, not ==.
Why not just update validPropertyNameRegex() to handle this case?