Merge "Fix header-abi-diff skips diff when config.json is not found"

This commit is contained in:
Mu-Le Lee
2022-08-22 15:09:09 +00:00
committed by Gerrit Code Review
2 changed files with 5 additions and 5 deletions

View File

@@ -213,7 +213,10 @@ bool ShouldEmitWarningMessage(CompatibilityStatusIR status) {
int main(int argc, const char **argv) {
llvm::cl::ParseCommandLineOptions(argc, argv, "header-checker");
ReadConfigFile(GetConfigFilePath(old_dump));
const std::string config_file_path = GetConfigFilePath(old_dump);
if (llvm::sys::fs::exists(config_file_path)) {
ReadConfigFile(config_file_path);
}
std::set<std::string> ignored_symbols;
if (llvm::sys::fs::exists(ignore_symbol_list)) {

View File

@@ -72,10 +72,7 @@ bool ConfigFile::Load(std::istream &istream) {
bool ConfigFile::Load(const std::string &path) {
std::ifstream stream(path);
if (stream.is_open()) {
return Load(stream);
}
return false;
return stream && Load(stream);
}
} // namespace utils