Replace unnecessary Sync<uint64_t> with std::atomic<uint64_t>

This commit is contained in:
Patrick Jackson 2023-03-31 17:44:13 -07:00
parent cf76b38e27
commit 2b905d1d35

View file

@ -10,7 +10,6 @@
#include "archive.hh"
#include "callback.hh"
#include "remote-store.hh"
#include "sync.hh"
#include <nlohmann/json.hpp>
#include <regex>
@ -1103,7 +1102,7 @@ std::map<StorePath, StorePath> copyPaths(
};
// total is accessed by each copy, which are each handled in separate threads
Sync<uint64_t> _total = 0;
std::atomic<uint64_t> total = 0;
for (auto & missingPath : sortedMissing) {
auto info = srcStore.queryPathInfo(missingPath);
@ -1126,9 +1125,8 @@ std::map<StorePath, StorePath> copyPaths(
PushActivity pact(act.id);
LambdaSink progressSink([&](std::string_view data) {
auto total(_total.lock());
*total += data.size();
act.progress(*total, info->narSize);
total += data.size();
act.progress(total, info->narSize);
});
TeeSink tee { sink, progressSink };