Avoids copy on the transformation operation

Author: zimmerle

headers/modsecurity/actions/action.h

@@ -57,7 +57,7 @@ class Action {
 
     virtual ~Action() { }
 
-    virtual std::string evaluate(std::string exp,
+    virtual std::string evaluate(const std::string &exp,
         Transaction *transaction);
     virtual bool evaluate(Rule *rule, Transaction *transaction);
     virtual bool evaluate(Rule *rule, Transaction *transaction,

src/actions/action.cc

@@ -45,7 +45,7 @@ namespace modsecurity {
 namespace actions {
 
 
-std::string Action::evaluate(std::string value,
+std::string Action::evaluate(const std::string &value,
     Transaction *transaction) {
     return value;
 }

src/actions/transformations/base64_decode.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string Base64Decode::evaluate(std::string value,
+std::string Base64Decode::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret = Utils::Base64::decode(value);
 

src/actions/transformations/base64_decode.h

@@ -32,7 +32,7 @@ class Base64Decode : public Transformation {
  public:
     explicit Base64Decode(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/base64_decode_ext.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string Base64DecodeExt::evaluate(std::string value,
+std::string Base64DecodeExt::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret = Utils::Base64::decode_forgiven(value);
 

src/actions/transformations/base64_decode_ext.h

@@ -32,7 +32,7 @@ class Base64DecodeExt : public Transformation {
  public:
     explicit Base64DecodeExt(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/base64_encode.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string Base64Encode::evaluate(std::string value,
+std::string Base64Encode::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret = Utils::Base64::encode(value);
 

src/actions/transformations/base64_encode.h

@@ -32,7 +32,7 @@ class Base64Encode : public Transformation {
  public:
     explicit Base64Encode(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/cmd_line.cc

@@ -31,7 +31,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string CmdLine::evaluate(std::string value,
+std::string CmdLine::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     int space = 0;

src/actions/transformations/cmd_line.h

@@ -33,7 +33,7 @@ class CmdLine : public Transformation {
     explicit CmdLine(const std::string &action) 
         : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };