fix: don't unnecessarily call value_parser twice

This commit is contained in:
Marcus Holland-Moritz 2024-04-27 08:59:10 +02:00
parent 1ff2878e23
commit 04119a91f8

View File

@ -92,7 +92,7 @@ bool parse_metadata_requirements_set(T& container, folly::dynamic& req,
v.asString(), name, e.what()));
}
if (auto maybe_value = value_parser(v)) {
if (maybe_value) {
if (!container.emplace(*maybe_value).second) {
throw std::runtime_error(fmt::format(
"duplicate value '{}' for requirement '{}'", v.asString(), name));