[clang] [lldb] fixing issue #64441 (PR #74814)

Felipe de Azevedo Piovezan via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 8 06:14:49 PST 2023


================
@@ -56,10 +56,10 @@ namespace Foo = A::B;   // namespace alias
 
 using Foo::myfunc;      // using declaration
 
-using namespace Foo;    // using directive
+//removing namespace foo; for quality naming 
----------------
felipepiovezan wrote:

Hi @jeevanghimire, please note that this is a test, so we must ensure that LLDB does the right thing regardless of the input it receives, even if said input is not considered best practices. The test was likely created to either ensure LLDB works in the case, or because it didn't use to work and  someone created a test at the same time the fix was done.

 As such, I don't think we should change this.


https://github.com/llvm/llvm-project/pull/74814


More information about the cfe-commits mailing list