[PATCH] D26592: [change-namespace] consider typedef/using alias decls in the moved namespace.
    Eric Liu via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Mon Nov 14 07:36:45 PST 2016
    
    
  
ioeric marked an inline comment as done.
ioeric added inline comments.
================
Comment at: unittests/change-namespace/ChangeNamespaceTests.cpp:829
                      "void f() {\n"
-                     "  using na::CA;\n"
-                     "  CA ca;\n"
+                     "  using ::na::C_A;\n"
+                     "  C_A ca;\n"
----------------
hokein wrote:
> Any reason to change the existing test case?
This was a typo that I found when manually running unit tests (instead of using `check clang-tools-extra`). In the old code, `using na::CA;` does not compile, so I figure I might just fix here as well.
https://reviews.llvm.org/D26592
    
    
More information about the cfe-commits
mailing list