Hiralo added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp:329-330 + + if (!ConfigFile.empty()) { + if (!Config.empty()) { + llvm::errs() << "Error: --config-file and --config are " ---------------- njames93 wrote: > nit: Should we be using `Config(File)?.getNumOccurances() > 0` here and below > If someone puts `--config=""` They are specifying a config(file)?, even if > its empty. > nit: Should we be using `Config(File)?.getNumOccurances() > 0` here and below > If someone puts `--config=""` They are specifying a config(file)?, even if > its empty. Sounds good. What is expected o/p with -config="" ? Currently it works as below... $ ./myinstall/bin/clang-tidy -config="" --dump-config -- config_occurances =1 config_is_empty = yes ``` Checks: 'clang-diagnostic-*,clang-analyzer-*' WarningsAsErrors: '' HeaderFilterRegex: '' AnalyzeTemporaryDtors: false FormatStyle: none CheckOptions: - key: llvm-else-after-return.WarnOnConditionVariables ... ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89936/new/ https://reviews.llvm.org/D89936 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits