commit 8bb6398: [Minor] Simplify code
Vsevolod Stakhov
vsevolod at rspamd.com
Fri Jun 24 20:56:03 UTC 2022
Author: Vsevolod Stakhov
Date: 2022-06-24 21:52:17 +0100
URL: https://github.com/rspamd/rspamd/commit/8bb63981133b479dcaf24b9d34d43993c1b732c8 (HEAD -> master)
[Minor] Simplify code
---
src/libserver/css/css.cxx | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/libserver/css/css.cxx b/src/libserver/css/css.cxx
index 9e26eb42f..895853132 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
More information about the Commits
mailing list