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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to