[PATCH] D53489: [change-namespace] Enhance detection of conflicting namespaces.

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 22 03:28:22 PDT 2018


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

For example:

  namespace util { class Base; }
  
  namespace new {
  namespace util { class Internal; }
  }
  
  namespace old {
  util::Base b1;
  }

When changing `old::` to `new::`, `util::` in namespace "new::" will conflict
with "new::util::" unless a leading "::" is added.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53489

Files:
  change-namespace/ChangeNamespace.cpp
  unittests/change-namespace/ChangeNamespaceTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53489.170382.patch
Type: text/x-patch
Size: 5539 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181022/f1841820/attachment.bin>


More information about the cfe-commits mailing list