jutocz updated this revision to Diff 92619.
jutocz added a comment.
I like the idea of using `llvm::None` plus `operator bool` and `operator *`.
I've updated the diff to use `llvm::Optional` this way .
https://reviews.llvm.org/D30931
Files:
clang-tidy/readability/IdentifierNamingCheck.cpp
c
jutocz updated this revision to Diff 92205.
jutocz added a comment.
Updated diff to show full context
https://reviews.llvm.org/D30931
Files:
clang-tidy/readability/IdentifierNamingCheck.cpp
clang-tidy/readability/IdentifierNamingCheck.h
Index: clang-tidy/readability/IdentifierNamingCheck.h
jutocz updated this revision to Diff 92108.
jutocz retitled this revision from "[clang-tidy] added new identifier naming
case type for ignoring case style" to "[clang-tidy] modified identifier naming
case to use CT_AnyCase for ignoring case style".
jutocz edited the summary of this revision.
juto
jutocz added a comment.
You are right, it does look misleading. I'll try to modify it the way you
suggest (though I'm new to LLVM, so be ready to give me more comments;).
https://reviews.llvm.org/D30931
___
cfe-commits mailing list
cfe-commits@list
jutocz added a comment.
As this is my first patch, can someone please help me in commiting it?
https://reviews.llvm.org/D30932
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits