[PATCH] D27125: Consider nested namespaces in the canonical namespace as canonical as well.

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 25 01:08:50 PST 2016


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

For example, this case was missed when looking for different but canonical
namespaces. UseContext in this case should be considered as in the canonical
namespace.

  namespace a { namespace b { <FromContext> } }
  namespace a { namespace b { namespace c { <UseContext> } } }

Added some commenting.


https://reviews.llvm.org/D27125

Files:
  lib/Tooling/Core/Lookup.cpp
  unittests/Tooling/LookupTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27125.79282.patch
Type: text/x-patch
Size: 5390 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161125/22faca6e/attachment.bin>


More information about the cfe-commits mailing list