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

[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::Replacements RunResult;
     deriveLocalStyle(AnnotatedLines);
----------------
Call this "RequoteChanges".

================
Comment at: lib/Format/Format.cpp:831
@@ -830,1 +830,3 @@
+    RunResult = RunResult.merge(Whitespaces.generateReplacements());
+    return RunResult;
   }
----------------
Just

  return RequoteChanges.merge(Whitespaces.generateReplacements());


https://reviews.llvm.org/D24155