[clang-tools-extra] 7a328cf - [include-cleaner] Bail out in the standalone tool for invalid ignore-headers

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 14 01:06:35 PDT 2023


Author: Haojian Wu
Date: 2023-07-14T10:05:22+02:00
New Revision: 7a328cf539b38484218a0eb902611c69cfba7981

URL: https://github.com/llvm/llvm-project/commit/7a328cf539b38484218a0eb902611c69cfba7981
DIFF: https://github.com/llvm/llvm-project/commit/7a328cf539b38484218a0eb902611c69cfba7981.diff

LOG: [include-cleaner] Bail out in the standalone tool for invalid ignore-headers
flag

Added: 
    

Modified: 
    clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index c6cd7995d2e829..ae4d1e97414415 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -271,6 +271,8 @@ int main(int argc, const char **argv) {
     }
   }
   auto HeaderFilter = headerFilter();
+  if (!HeaderFilter)
+    return 1; // error already reported.
   ActionFactory Factory(HeaderFilter);
   return clang::tooling::ClangTool(OptionsParser->getCompilations(),
                                    OptionsParser->getSourcePathList())


        


More information about the cfe-commits mailing list