NFC: TreeTransform.h: make the function TransformNestedNameSpecifierLoc more beautiful

[zhouyizhou]

Summary

Hi,
I am new to LLVM, and I really want to get involved in LLVM community.

I guess if the following switch case of function
TransformNestedNameSpecifierLoc could possibly made more beautiful with
'break' stmt moved inside of the right brace.

I think move of 'break' stmt will not change the invoking destructor of
IdInfo.

Thanks for your effort.

I have done make check-all on x86_64-linux
Signed-off-by: Zhouyi Zhou <[e-mail]>

[zhouyizhou]

I use git diff -U99999 instead

[zhouyizhou]

I do some "grep" in clang/lib directory:
llvm-project/clang/lib#find . -name "*.*"|xargs grep -n -B 2 "break;"|grep -A 2 "}"

my patch also seems to match the coding style of other parts of clang.

Thanks again

[pengfei]

I think you can add a NFC in the title.

clang/lib/Sema/TreeTransform.h

3947–3949

There're some format issues in this function. It's better if you can fix them too. You can use clang-format to help with you.

[zhouyizhou]

use clang-format to fix the function TransformNestedNameSpecifierLoc

[pengfei]

LGTM. Thanks for improving it.

clang/lib/Sema/TreeTransform.h

       if (SemaRef.BuildCXXNestedNameSpecifier(/*Scope=*/nullptr, IdInfo, false,
                                               SS, FirstQualifierInScope, false))
         return NestedNameSpecifierLoc();
-    }
       break;
+    }
 
     case NestedNameSpecifier::Namespace: {
       NamespaceDecl *NS