Re: [PATCH] D24155: clang-format: [JS] merge requoting replacements.

2016-09-02 Thread Martin Probst via cfe-commits
This revision was automatically updated to reflect the committed changes. mprobst marked an inline comment as done. Closed by commit rL280487: clang-format: [JS] merge requoting replacements. (authored by mprobst). Changed prior to commit: https://reviews.llvm.org/D24155?vs=70066&id=70164#toc

Re: [PATCH] D24155: clang-format: [JS] merge requoting replacements.

2016-09-02 Thread Martin Probst via cfe-commits
mprobst marked an inline comment as done. Comment at: lib/Format/Format.cpp:806 @@ -805,2 +805,3 @@ FormatTokenLexer &Tokens, tooling::Replacements &Result) override { +tooling::Replacements RunResult; deriveLocalStyle(AnnotatedLines); djasper

Re: [PATCH] D24155: clang-format: [JS] merge requoting replacements.

2016-09-02 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Basically looks good. Comment at: lib/Format/Format.cpp:806 @@ -805,2 +805,3 @@ FormatTokenLexer &Tokens, tooling::Replacements &Result) override { +tooling::