common: config_cacher: use set::contains() instead of count()

ronen-fr · ronen-fr · commit 5c098880829a · 2025-01-14T03:23:16.000-06:00
moved out of the main commit to
facilitate backporting it to pre-C++20.

Signed-off-by: Ronen Friedman &lt;rfriedma@redhat.com&gt;
diff --git a/src/common/config_cacher.h b/src/common/config_cacher.h
@@ -40,7 +40,7 @@ class md_config_cacher_t : public md_config_obs_t {
 
   void handle_conf_change(const ConfigProxy& conf,
                           const std::set<std::string>& changed) override {
-    if (changed.count(keys[0])) {
+    if (changed.contains(keys[0])) {
       value_cache.store(conf.get_val<ValueT>(keys[0]));
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ class md_config_cacher_t : public md_config_obs_t {`
`40`	`40`
`41`	`41`	`void handle_conf_change(const ConfigProxy& conf,`
`42`	`42`	`const std::set<std::string>& changed) override {`
`43`		`- if (changed.count(keys[0])) {`
	`43`	`+ if (changed.contains(keys[0])) {`
`44`	`44`	`value_cache.store(conf.get_val<ValueT>(keys[0]));`
`45`	`45`	`}`
`46`	`46`	`}`