[PATCH] D30931: [clang-tidy] modified identifier naming case to use CT_AnyCase for ignoring case style

2017-03-22 Thread Juliusz Toczydłowski via Phabricator via cfe-commits
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

[PATCH] D30931: [clang-tidy] modified identifier naming case to use CT_AnyCase for ignoring case style

2017-03-17 Thread Juliusz Toczydłowski via Phabricator via cfe-commits
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

[PATCH] D30931: [clang-tidy] modified identifier naming case to use CT_AnyCase for ignoring case style

2017-03-17 Thread Juliusz Toczydłowski via Phabricator via cfe-commits
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

[PATCH] D30931: [clang-tidy] added new identifier naming case type for ignoring case style

2017-03-16 Thread Juliusz Toczydłowski via Phabricator via cfe-commits
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

[PATCH] D30932: [clang-format] disabled adding extra space after MSVC '__super' keyword

2017-03-15 Thread Juliusz Toczydłowski via Phabricator via cfe-commits
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