[PATCH] D26853: Make llvm::Error generated from replacement interfaces more specific.

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 18 06:59:22 PST 2016


ioeric created this revision.
ioeric added reviewers: klimek, djasper, bkramer.
ioeric added a subscriber: cfe-commits.

The new error information contains the type of error (e.g. overlap or bad file path)
and the replacement(s) that is causing the error. This enables us to resolve some errors.
For example, for insertion at the same location conflict, we need to know the
existing replacement which conflicts with the new replacement in order to calculate
the new position to be insert before/after the existing replacement (for merging).


https://reviews.llvm.org/D26853

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26853.78519.patch
Type: text/x-patch
Size: 12608 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161118/8d2880e2/attachment.bin>


More information about the cfe-commits mailing list