From a112cbefc6d6766a2dec30f736aa6c0540f9b6db Mon Sep 17 00:00:00 2001
From: Harsh Shandilya <me@msfjarvis.dev>
Date: Tue, 15 Feb 2022 02:54:16 +0530
Subject: [PATCH] refactor: don't move value in Act and Match implementations

---
 src/config.rs | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/config.rs b/src/config.rs
index 6cca01a..042923c 100644
--- a/src/config.rs
+++ b/src/config.rs
@@ -49,19 +49,19 @@ pub enum Action<'config> {
 }
 
 pub trait Match {
-    fn check_match(self, string: &str) -> bool;
+    fn check_match(&self, string: &str) -> bool;
 }
 
 pub trait Act {
-    fn apply_action(self, input: &str) -> String;
+    fn apply_action(&self, input: &str) -> String;
 }
 
 impl Match for Matcher<'_> {
-    fn check_match(self, string: &str) -> bool {
+    fn check_match(&self, string: &str) -> bool {
         match self {
-            Matcher::StartsWith { prefix } => string.starts_with(&prefix),
-            Matcher::EndsWith { suffix } => string.ends_with(&suffix),
-            Matcher::Contains { substring } => string.contains(&substring),
+            Matcher::StartsWith { prefix } => string.starts_with(prefix),
+            Matcher::EndsWith { suffix } => string.ends_with(suffix),
+            Matcher::Contains { substring } => string.contains(substring),
             Matcher::Regex { pattern } => {
                 return if let Ok(regex) = Regex::from_str(pattern) {
                     regex.is_match(string)
@@ -69,18 +69,18 @@ impl Match for Matcher<'_> {
                     false
                 }
             }
-            Matcher::Exactly { content } => string == content,
+            Matcher::Exactly { content } => &string == content,
         }
     }
 }
 
 impl Act for Action<'_> {
-    fn apply_action(self, input: &str) -> String {
+    fn apply_action(&self, input: &str) -> String {
         return match self {
             Action::Replace { from, to } => input.replace(from, to),
             Action::Prefix { prefix } => format!("{prefix}{input}"),
             Action::Suffix { suffix } => format!("{input}{suffix}"),
-            Action::Set { content } => content.to_owned(),
+            Action::Set { content } => content.to_owned().to_owned(),
         };
     }
 }