[clang] 8f41d28 - [c-index-test] Fix warnings

Kazu Hirata via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 18 23:18:14 PST 2025


Author: Kazu Hirata
Date: 2025-02-18T23:18:08-08:00
New Revision: 8f41d28d89ee287d0f5a6518116ab316be2657b8

URL: https://github.com/llvm/llvm-project/commit/8f41d28d89ee287d0f5a6518116ab316be2657b8
DIFF: https://github.com/llvm/llvm-project/commit/8f41d28d89ee287d0f5a6518116ab316be2657b8.diff

LOG: [c-index-test] Fix warnings

This patch fixes:

  clang/tools/c-index-test/c-index-test.c:1240:15: error: mixing
  declarations and code is a C99 extension
  [-Werror,-Wdeclaration-after-statement]

  clang/tools/c-index-test/c-index-test.c:1367:14: error: mixing
  declarations and code is a C99 extension
  [-Werror,-Wdeclaration-after-statement]

  clang/tools/c-index-test/c-index-test.c:1468:14: error: mixing
  declarations and code is a C99 extension
  [-Werror,-Wdeclaration-after-statement]

Added: 
    

Modified: 
    clang/tools/c-index-test/c-index-test.c

Removed: 
    


################################################################################
diff  --git a/clang/tools/c-index-test/c-index-test.c b/clang/tools/c-index-test/c-index-test.c
index fed6fe0736904..0e7de8b98ea07 100644
--- a/clang/tools/c-index-test/c-index-test.c
+++ b/clang/tools/c-index-test/c-index-test.c
@@ -1231,14 +1231,16 @@ static CXString GetCursorSource(CXCursor Cursor) {
   CXSourceLocation Loc = clang_getCursorLocation(Cursor);
   CXString source;
   CXFile file;
+  const char *b;
+  CXString result;
   clang_getExpansionLocation(Loc, &file, 0, 0, 0);
   source = clang_getFileName(file);
   if (!clang_getCString(source)) {
     clang_disposeString(source);
     return createCXString("<invalid loc>");
   }
-  const char *b = basename(clang_getCString(source));
-  CXString result = duplicateCXString(b);
+  b = basename(clang_getCString(source));
+  result = duplicateCXString(b);
   clang_disposeString(source);
   return result;
 }
@@ -1363,8 +1365,9 @@ enum CXChildVisitResult FilteredPrintingVisitor(CXCursor Cursor,
   if (!Data->Filter || (Cursor.kind == *(enum CXCursorKind *)Data->Filter)) {
     CXSourceLocation Loc = clang_getCursorLocation(Cursor);
     unsigned line, column;
+    CXString source;
     clang_getFileLocation(Loc, 0, &line, &column, 0);
-    CXString source = GetCursorSource(Cursor);
+    source = GetCursorSource(Cursor);
     printf("// %s: %s:%d:%d: ", FileCheckPrefix, clang_getCString(source), line,
            column);
     clang_disposeString(source);
@@ -1461,11 +1464,12 @@ enum CXChildVisitResult USRVisitor(CXCursor C, CXCursor parent,
   if (!Data->Filter || (C.kind == *(enum CXCursorKind *)Data->Filter)) {
     CXString USR = clang_getCursorUSR(C);
     const char *cstr = clang_getCString(USR);
+    CXString CursorSource;
     if (!cstr || cstr[0] == '\0') {
       clang_disposeString(USR);
       return CXChildVisit_Recurse;
     }
-    CXString CursorSource = GetCursorSource(C);
+    CursorSource = GetCursorSource(C);
     printf("// %s: %s %s", FileCheckPrefix, clang_getCString(CursorSource),
            cstr);
     clang_disposeString(CursorSource);


        


More information about the cfe-commits mailing list