Re: [PATCH] D20208: clang-format: [JS] fix template string width counting.

2016-05-16 Thread Martin Probst via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL269747: clang-format: [JS] fix template string width counting. (authored by mprobst). Changed prior to commit: http://reviews.llvm.org/D20208?vs=57066&id=57434#toc Repository: rL LLVM http://reviews

Re: [PATCH] D20208: clang-format: [JS] fix template string width counting.

2016-05-16 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Nice. http://reviews.llvm.org/D20208 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe

Re: [PATCH] D20208: clang-format: [JS] fix template string width counting.

2016-05-12 Thread Martin Probst via cfe-commits
mprobst updated this revision to Diff 57066. mprobst added a comment. - simplify logic by parsing forward, reduce complexity to O(n) from O(n^2) http://reviews.llvm.org/D20208 Files: lib/Format/Format.cpp unittests/Format/FormatTestJS.cpp Index: unittests/Format/FormatTestJS.cpp ==