diff options
| author | Carl Hetherington <cth@carlh.net> | 2025-03-04 22:33:31 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2025-03-05 23:52:14 +0100 |
| commit | 2719964f89431a0598ef617326158309afe5ba43 (patch) | |
| tree | 56bdbfaf7263bae3b724737457d56b5de2511e59 | |
| parent | 4168c7508d96df13794e49dfe7186a508a387186 (diff) | |
all_of() is clearer than find_if() here.
| -rw-r--r-- | src/lib/find_missing.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/lib/find_missing.cc b/src/lib/find_missing.cc index af91b3682..585a6e713 100644 --- a/src/lib/find_missing.cc +++ b/src/lib/find_missing.cc @@ -109,7 +109,7 @@ dcpomatic::find_missing (vector<shared_ptr<Content>> content_to_fix, boost::file Replacements digest_replacement_paths; for (auto content: content_to_fix) { auto const& repl = name_replacement_paths[content]; - bool const replacements_exist = std::find_if(repl.begin(), repl.end(), [](path p) { return !exists(p); }) == repl.end(); + bool const replacements_exist = std::all_of(repl.begin(), repl.end(), [](path p) { return exists(p); }); if (replacements_exist && simple_digest(name_replacement_paths[content]) == content->digest()) { content->set_paths (repl); } else { @@ -128,7 +128,7 @@ dcpomatic::find_missing (vector<shared_ptr<Content>> content_to_fix, boost::file auto iter = digest_replacement_paths.find(content); if (iter != digest_replacement_paths.end()) { auto const& repl = iter->second; - bool const replacements_exist = std::find_if(repl.begin(), repl.end(), [](path p) { return !exists(p); }) == repl.end(); + bool const replacements_exist = std::all_of(repl.begin(), repl.end(), [](path p) { return exists(p); }); if (replacements_exist) { content->set_paths(repl); } |