avoid cloning sef.data before the replace

gkorland · gkorland · commit cc69f10eddf6 · 2019-07-03T11:04:49.000+03:00
diff --git a/src/redisjson.rs b/src/redisjson.rs
@@ -63,7 +63,8 @@ impl RedisJSON {
     }
 
     pub fn delete_path(&mut self, path: &str) -> Result<usize, Error> {
-        self.data = jsonpath_lib::delete(self.data.clone(), path)?;
+        let current_value = mem::replace(&mut self.data, Value::Null);
+        self.data = jsonpath_lib::delete(current_value, path)?;
 
         let res : usize = match self.data {
             Value::Null => 0,

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,8 @@ impl RedisJSON {`
`63`	`63`	`}`
`64`	`64`
`65`	`65`	`pub fn delete_path(&mut self, path: &str) -> Result<usize, Error> {`
`66`		`- self.data = jsonpath_lib::delete(self.data.clone(), path)?;`
	`66`	`+ let current_value = mem::replace(&mut self.data, Value::Null);`
	`67`	`+ self.data = jsonpath_lib::delete(current_value, path)?;`
`67`	`68`
`68`	`69`	`let res : usize = match self.data {`
`69`	`70`	`Value::Null => 0,`