[PATCH] D42639: [clang-move] Clever on handling header file which includes itself.

[Haojian Wu via Phabricator via cfe-commits]

hokein marked an inline comment as done.
hokein added inline comments.


================
Comment at: clang-move/ClangMove.cpp:708
+    // Find old.h includes "old.h".
+    if (AbsoluteOldHeader == AbsoluteOldHeader) {
+      OldHeaderIncludeRangeInHeader = IncludeFilenameRange;
----------------
ioeric wrote:
> This check is always true?
Oops, good catch, should be AbsoluteIncludeHeader.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D42639



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