Apply suggestions from code review

airween · Copilot · web-flow · commit 1e75649254d5 · 2026-03-10T22:49:52.000+01:00
Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/headers/modsecurity/audit_log.h b/headers/modsecurity/audit_log.h
@@ -172,7 +172,7 @@ class AuditLog {
 
     bool saveIfRelevant(Transaction *transaction);
     bool saveIfRelevant(Transaction *transaction, int parts);
-    bool isRelevant(const int status) const;
+    bool isRelevant(int status) const;
     bool isRelevant(int status);
 
     static int addParts(int parts, std::string_view new_parts);
diff --git a/src/anchored_set_variable.cc b/src/anchored_set_variable.cc
@@ -160,7 +160,7 @@ void AnchoredSetVariable::resolveRegularExpression(Utils::Regex *r,
 void AnchoredSetVariable::resolveRegularExpression(Utils::Regex *r,
     std::vector<const VariableValue *> *l,
     variables::KeyExclusions &ke) {  // cppcheck-suppress constParameterReference
-    static_cast<const AnchoredSetVariable&>(*this).resolveRegularExpression(r, l);
+    static_cast<const AnchoredSetVariable&>(*this).resolveRegularExpression(r, l, ke);
 }
 
 
diff --git a/src/audit_log/audit_log.cc b/src/audit_log/audit_log.cc
@@ -248,7 +248,7 @@ bool AuditLog::init(std::string *error) {
 }
 
 
-bool AuditLog::isRelevant(const int status) const {
+bool AuditLog::isRelevant(int status) const {
     std::string sstatus = std::to_string(status);
 
     if (m_relevant.empty()) {

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,7 @@ void AnchoredSetVariable::resolveRegularExpression(Utils::Regex *r,`
`160`	`160`	`void AnchoredSetVariable::resolveRegularExpression(Utils::Regex *r,`
`161`	`161`	`std::vector<const VariableValue > l,`
`162`	`162`	`variables::KeyExclusions &ke) { // cppcheck-suppress constParameterReference`
`163`		`- static_cast<const AnchoredSetVariable&>(*this).resolveRegularExpression(r, l);`
	`163`	`+ static_cast<const AnchoredSetVariable&>(*this).resolveRegularExpression(r, l, ke);`
`164`	`164`	`}`
`165`	`165`
`166`	`166`
Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,7 @@ bool AuditLog::init(std::string *error) {`
`248`	`248`	`}`
`249`	`249`
`250`	`250`
`251`		`-bool AuditLog::isRelevant(const int status) const {`
	`251`	`+bool AuditLog::isRelevant(int status) const {`
`252`	`252`	`std::string sstatus = std::to_string(status);`
`253`	`253`
`254`	`254`	`if (m_relevant.empty()) {`