From: Vsevolod Stakhov <vsevolod@rspamd.com>
Date: Fri, 24 Jun 2022 20:52:17 +0000 (+0100)
Subject: [Minor] Simplify code
X-Git-Tag: 3.3~177
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=8bb63981133b479dcaf24b9d34d43993c1b732c8;p=thirdparty%2Frspamd.git

[Minor] Simplify code
---

diff --git a/src/libserver/css/css.cxx b/src/libserver/css/css.cxx
index 9e26eb42f8..8958531329 100644
--- a/src/libserver/css/css.cxx
+++ b/src/libserver/css/css.cxx
@@ -78,13 +78,10 @@ css_style_sheet::add_selector_rule(std::unique_ptr<css_selector> &&selector,
 	}
 
 	if (target_hash) {
-		auto found_it = target_hash->find(selector);
 
-		if (found_it == target_hash->end()) {
-			/* Easy case, new element */
-			target_hash->insert({std::move(selector), decls});
-		}
-		else {
+		auto [found_it, found] = target_hash->try_emplace(std::move(selector), decls);
+
+		if (found) {
 			/* The problem with merging is actually in how to handle selectors chains
 			 * For example, we have 2 selectors:
 			 * 1. class id tag -> meaning that we first match class, then we ensure that