[PATCH] D26288: Deduplicate replacements by FileEntry instead of file names.

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 3 12:38:53 PDT 2016


ioeric created this revision.
ioeric added a reviewer: djasper.
ioeric added a subscriber: cfe-commits.
Herald added a subscriber: klimek.

The current version does not deduplicate equivalent file paths correctly.
For example, a relative path and an absolute path are considered inequivalent.
Comparing FileEnry addresses these issues.


https://reviews.llvm.org/D26288

Files:
  include/clang/Tooling/Core/Replacement.h
  lib/Tooling/Core/Replacement.cpp
  lib/Tooling/Refactoring.cpp
  unittests/Tooling/RefactoringTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26288.76881.patch
Type: text/x-patch
Size: 5565 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161103/61b447dc/attachment.bin>


More information about the cfe-commits mailing list