[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