berenm marked 2 inline comments as done.

================
Comment at: clang-tidy/readability/IdentifierNamingCheck.cpp:166
@@ +165,3 @@
+  static llvm::Regex Splitter(
+      "(([a-z0-9A-Z]*)(_+)|([A-Z]?[a-z0-9]+)([A-Z]|$)|([A-Z]+)([A-Z]|$))");
+
----------------
alexfh wrote:
> Why do you need the outermost parentheses?
For some unknown reason, I always thought parentheses were required around 
regex alternation... Well, I now know they are not.


http://reviews.llvm.org/D10933



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to