ref: cargo clippy

Author: raphael-goetz

taurus/src/context/context.rs

@@ -19,16 +19,16 @@ pub struct Context {
 
 impl Context {
     pub fn new(flow_input: Value) -> Self {
-        return Context {
+        Context {
             results: HashMap::new(),
             input_types: HashMap::new(),
             flow_input,
             current_node_id: 0,
-        };
+        }
     }
 
     pub fn get_current_node_id(&mut self) -> i64 {
-        return self.current_node_id;
+        self.current_node_id
     }
 
     pub fn set_current_node_id(&mut self, node_id: i64) {
@@ -108,7 +108,7 @@ impl Context {
     }
 
     fn get_flow_input(&mut self) -> ContextResult {
-        return ContextResult::Success(self.flow_input.clone());
+        ContextResult::Success(self.flow_input.clone())
     }
 
     fn get_input_type(&mut self, input_type: InputType) -> ContextResult {

taurus/src/context/executor.rs

@@ -193,7 +193,7 @@ impl<'a> Executor<'a> {
                             ArgTrace {
                                 index: i,
                                 kind: ArgKind::Reference {
-                                    reference: reference,
+                                    reference,
                                     hit: true,
                                 },
                                 preview: format!("ctx.get({:?}) -> {:?}", r, v),
@@ -247,8 +247,8 @@ impl<'a> Executor<'a> {
                 .copied()
                 .unwrap_or(ParameterNode::Eager);
 
-            if matches!(mode, ParameterNode::Eager) {
-                if let Argument::Thunk(id) = *arg {
+            if matches!(mode, ParameterNode::Eager)
+                && let Argument::Thunk(id) = *arg {
                     let (child_sig, child_root) = self.execute_call(id, ctx, tracer);
 
                     tracer.link_child(
@@ -271,7 +271,6 @@ impl<'a> Executor<'a> {
                         }
                     }
                 }
-            }
         }
 
         Ok(())

taurus/src/debug/tracer.rs

@@ -15,6 +15,12 @@ pub struct Tracer {
     stack: Vec<u64>,
 }
 
+impl Default for Tracer {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 impl Tracer {
     pub fn new() -> Self {
         Self {

taurus/src/implementation/array.rs

@@ -162,7 +162,7 @@ fn filter(
     let mut out: Vec<Value> = Vec::new();
     let node_id = ctx.get_current_node_id();
     let input_type = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 0,
     };
@@ -210,7 +210,7 @@ fn find(
     };
     let node_id = ctx.get_current_node_id();
     let input_type = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 0,
     };
@@ -266,7 +266,7 @@ fn find_last(
     array.values.reverse();
     let node_id = ctx.get_current_node_id();
     let input_type = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 0,
     };
@@ -321,7 +321,7 @@ fn find_index(
     };
     let node_id = ctx.get_current_node_id();
     let input_type = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 0,
     };
@@ -410,7 +410,7 @@ fn for_each(
     };
     let node_id = ctx.get_current_node_id();
     let input_type = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 0,
     };
@@ -457,7 +457,7 @@ fn map(
     let mut out: Vec<Value> = Vec::with_capacity(array.values.len());
     let node_id = ctx.get_current_node_id();
     let input_type = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 0,
     };
@@ -649,13 +649,13 @@ fn sort(
     let mut out: Vec<f64> = Vec::new();
     let node_id = ctx.get_current_node_id();
     let input_type = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 0,
     };
 
     let input_type_next = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 1,
     };
@@ -666,7 +666,7 @@ fn sort(
         ctx.insert_input_type(input_type_next, b.clone());
         let sig = run(*transform_node, ctx);
         signals.push(sig);
-        return Ordering::Equal;
+        Ordering::Equal
     });
 
     for sig in signals {
@@ -735,13 +735,13 @@ fn sort_reverse(
     let mut out: Vec<f64> = Vec::new();
     let node_id = ctx.get_current_node_id();
     let input_type = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 0,
     };
 
     let input_type_next = InputType {
-        node_id: node_id,
+        node_id,
         parameter_index: 1,
         input_index: 1,
     };
@@ -752,7 +752,7 @@ fn sort_reverse(
         ctx.insert_input_type(input_type_next, b.clone());
         let sig = run(*transform_node, ctx);
         signals.push(sig);
-        return Ordering::Equal;
+        Ordering::Equal
     });
 
     for sig in signals {