Re: [PATCH] D13641: clang-format: [JS] handle character classes in regexes.

2015-10-11 Thread Daniel Jasper via cfe-commits
djasper closed this revision. djasper added a comment. Submitted as r250009. http://reviews.llvm.org/D13641 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D13641: clang-format: [JS] handle character classes in regexes.

2015-10-11 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. looks good Comment at: lib/Format/Format.cpp:879 @@ -878,2 +878,3 @@ unsigned TokenCount = 0; +auto InCharacterClass = false; for (auto I = Tokens.rbegin() + 1

[PATCH] D13641: clang-format: [JS] handle character classes in regexes.

2015-10-11 Thread Martin Probst via cfe-commits
mprobst created this revision. mprobst added a reviewer: djasper. mprobst added subscribers: cfe-commits, klimek. Slashes in regular expressions do not need to be escaped and do not terminate the regular expression even without a preceding backslash. http://reviews.llvm.org/D13641 Files: lib/F