[PATCH] D68937: [clangd] Add parameter renaming to define-inline code action
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 14 03:04:22 PDT 2019
kadircet created this revision.
kadircet added a reviewer: ilya-biryukov.
Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay.
Herald added a project: clang.
When moving a function definition to declaration location we also need
to handle renaming of the both function and template parameters.
This patch achives that by making sure every parameter name and dependent type
in destination is renamed to their respective name in the source.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D68937
Files:
clang-tools-extra/clangd/refactor/tweaks/DefineInline.cpp
clang-tools-extra/clangd/unittests/TweakTests.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68937.224826.patch
Type: text/x-patch
Size: 9733 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191014/e23828a1/attachment.bin>
More information about the cfe-commits
mailing list