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

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 14 07:02:22 PDT 2018


ioeric added inline comments.


================
Comment at: lib/Format/Format.cpp:2277
     auto Replace =
-        Includes.insert(trimInclude(IncludeName), IncludeName.startswith("<"));
+        Includes.insert(IncludeName.trim("\"<>"), IncludeName.startswith("<"));
     if (Replace) {
----------------
ilya-biryukov wrote:
> This change seems unrelated. Is this patch a proper place for it?
Indeed this was unrelated. I forgot to revert this when splitting from a previous patch.


Repository:
  rC Clang

https://reviews.llvm.org/D46758





More information about the cfe-commits mailing list