diff options
author | Alberto Sartori <alberto.sartori@huawei.com> | 2022-06-28 18:10:30 +0200 |
---|---|---|
committer | Alberto Sartori <alberto.sartori@huawei.com> | 2022-06-28 19:05:59 +0200 |
commit | 41e647a3f54afbb527e9457c0a41b8c9a5ed70f8 (patch) | |
tree | 787a8ffefa97cc965e37b2ca70426d398205bdc6 /src | |
parent | ea7c6c8dab09c65d54a6f10309e7d973e8a764e3 (diff) | |
download | justbuild-41e647a3f54afbb527e9457c0a41b8c9a5ed70f8.tar.gz |
add parenthesis to emphasize the order of evaluation
Diffstat (limited to 'src')
-rw-r--r-- | src/buildtool/execution_api/local/local_action.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/buildtool/execution_api/local/local_action.cpp b/src/buildtool/execution_api/local/local_action.cpp index 93a00c0d..74f90258 100644 --- a/src/buildtool/execution_api/local/local_action.cpp +++ b/src/buildtool/execution_api/local/local_action.cpp @@ -209,7 +209,7 @@ auto LocalAction::CollectOutputFile(std::filesystem::path const& exec_path, const noexcept -> std::optional<bazel_re::OutputFile> { auto file_path = exec_path / local_path; auto type = FileSystemManager::Type(file_path); - if (not type or not IsFileObject(*type)) { + if ((not type) or (not IsFileObject(*type))) { Logger::Log(LogLevel::Error, "expected file at {}", local_path); return std::nullopt; } @@ -232,7 +232,7 @@ auto LocalAction::CollectOutputDir(std::filesystem::path const& exec_path, -> std::optional<bazel_re::OutputDirectory> { auto dir_path = exec_path / local_path; auto type = FileSystemManager::Type(dir_path); - if (not type or not IsTreeObject(*type)) { + if ((not type) or (not IsTreeObject(*type))) { Logger::Log(LogLevel::Error, "expected directory at {}", local_path); return std::nullopt; } |