Merge pull request #9542 from Icinga/context-evaluating-apply-rules-f…

…or-host Construct string once, not unnecessarily N times
Icinga · Oct 17, 2022 · f2563cc · f2563cc
2 parents dd7009d + ce1a122
commit f2563cc
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/icinga/service-apply.cpp b/lib/icinga/service-apply.cpp
@@ -121,9 +121,9 @@ bool Service::EvaluateApplyRule(const Host::Ptr& host, const ApplyRule& rule)
 
 void Service::EvaluateApplyRules(const Host::Ptr& host)
 {
-	for (ApplyRule& rule : ApplyRule::GetRules("Service")) {
-		CONTEXT("Evaluating 'apply' rules for host '" + host->GetName() + "'");
+	CONTEXT("Evaluating 'apply' rules for host '" + host->GetName() + "'");
 
+	for (ApplyRule& rule : ApplyRule::GetRules("Service")) {
 		if (EvaluateApplyRule(host, rule))
 			rule.AddMatch();
 	}