[PATCH] D30493: [change-namespace] avoid adding leading '::' when possible.
Eric Liu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 1 03:47:30 PST 2017
ioeric created this revision.
When changing namespaces, the tool adds leading "::" to references that need to
be fully-qualified, which would affect readability.
We avoid adding "::" when the symbol name does not conflict with the new
namespace name. For example, a symbol name "na::nb::X" conflicts with "ns::na"
since it would be resolved to "ns::na::nb::X" in the new namespace.
https://reviews.llvm.org/D30493
Files:
change-namespace/ChangeNamespace.cpp
unittests/change-namespace/ChangeNamespaceTests.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30493.90154.patch
Type: text/x-patch
Size: 17560 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170301/c03edbcb/attachment.bin>
More information about the cfe-commits
mailing list