[PATCH] D46758: [clang-format] Move #include related style to libToolingCore

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 14 06:59:59 PDT 2018


ilya-biryukov added a comment.

LG. Just a quick comment about possibly unrelated change.



================
Comment at: lib/Format/Format.cpp:2277
     auto Replace =
-        Includes.insert(trimInclude(IncludeName), IncludeName.startswith("<"));
+        Includes.insert(IncludeName.trim("\"<>"), IncludeName.startswith("<"));
     if (Replace) {
----------------
This change seems unrelated. Is this patch a proper place for it?


Repository:
  rC Clang

https://reviews.llvm.org/D46758





More information about the cfe-commits mailing list