From 3f74b0d4217b10e0c84f6f3ad32d813c19083976 Mon Sep 17 00:00:00 2001 From: Paul Cristian Sarbu Date: Mon, 4 Dec 2023 18:56:21 +0100 Subject: 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. --- src/other_tools/just_mr/fetch.cpp | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) (limited to 'src/other_tools/just_mr/fetch.cpp') 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 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{}", -- cgit v1.2.3