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

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 29 05:24:51 PST 2018


hokein created this revision.
hokein added a reviewer: ioeric.
Herald added a subscriber: klimek.

Previously, we assume only old.cc includes "old.h", which would
introduce incorrect fixes for the cases where old.h also includes `#include "old.h"`

Although it should not be occurred in real projects, clang-move should handle this.

Old.h:

  class Foo {};

after moving to a new old.h:

  class Foo {};


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D42639

Files:
  clang-move/ClangMove.cpp
  clang-move/ClangMove.h
  unittests/clang-move/ClangMoveTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42639.131778.patch
Type: text/x-patch
Size: 5359 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180129/7912a5e0/attachment.bin>


More information about the cfe-commits mailing list