diff options
| author | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2023-12-04 18:56:21 +0100 |
|---|---|---|
| committer | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2023-12-07 09:59:04 +0100 |
| commit | 3f74b0d4217b10e0c84f6f3ad32d813c19083976 (patch) | |
| tree | 31e0304f00b96c6d55d86e0d203bf589501907d9 /src/other_tools/just_mr/fetch.cpp | |
| parent | c9149af8acfe9f016aff783f453e930809a16b95 (diff) | |
| download | justbuild-3f74b0d4217b10e0c84f6f3ad32d813c19083976.tar.gz | |
content_cas_map: Remove unnecessary return type
As this map is supposed to work as an all-or-nothing guarantee of
content being in CAS, we can replace the bool return type with the
null type instead.
Diffstat (limited to 'src/other_tools/just_mr/fetch.cpp')
| -rw-r--r-- | src/other_tools/just_mr/fetch.cpp | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/src/other_tools/just_mr/fetch.cpp b/src/other_tools/just_mr/fetch.cpp index c4450c2a..1f4a04d1 100644 --- a/src/other_tools/just_mr/fetch.cpp +++ b/src/other_tools/just_mr/fetch.cpp @@ -309,15 +309,7 @@ auto MultiRepoFetch(std::shared_ptr<Configuration> const& config, repo_fetch_map.ConsumeAfterKeysReady( &ts, repos_to_fetch, - [&failed](auto const& values) { - // report any fetch fails - for (auto const& val : values) { - if (not *val) { - failed = true; - break; - } - } - }, + []([[maybe_unused]] auto const& values) {}, [&failed](auto const& msg, bool fatal) { Logger::Log(fatal ? LogLevel::Error : LogLevel::Warning, "While performing just-mr fetch:\n{}", |