[PATCH] D68969: [clang-format] Remove the dependency on frontend

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 21 23:34:48 PDT 2019


MyDeveloperDay marked 2 inline comments as done.
MyDeveloperDay added inline comments.


================
Comment at: clang/tools/clang-format/ClangFormat.cpp:356
+
+      StringRef Line(StartBuf, (EndBuf - StartBuf) - 1);
+
----------------
klimek wrote:
> - 1 is to exclude the \n I'd assume?
correct, without the -1 I end up with an extra blank line in the output


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68969/new/

https://reviews.llvm.org/D68969





More information about the cfe-commits mailing list