Derive Default

We can derive these implementations. Found by clippy.

Author: tcharding

src/interpreter/stack.rs

@@ -83,7 +83,7 @@ impl<'txin> Element<'txin> {
 
 /// Stack Data structure representing the stack input to Miniscript. This Stack
 /// is created from the combination of ScriptSig and Witness stack.
-#[derive(Clone, PartialEq, Eq, PartialOrd, Ord, Debug, Hash)]
+#[derive(Clone, PartialEq, Eq, PartialOrd, Ord, Debug, Default, Hash)]
 pub struct Stack<'txin>(Vec<Element<'txin>>);
 
 impl<'txin> From<Vec<Element<'txin>>> for Stack<'txin> {
@@ -92,12 +92,6 @@ impl<'txin> From<Vec<Element<'txin>>> for Stack<'txin> {
     }
 }
 
-impl<'txin> Default for Stack<'txin> {
-    fn default() -> Self {
-        Stack(vec![])
-    }
-}
-
 impl<'txin> Stack<'txin> {
     /// Whether the stack is empty
     pub fn is_empty(&self) -> bool {

src/miniscript/types/extra_props.rs

@@ -13,7 +13,7 @@ use std::cmp;
 use std::iter::once;
 
 /// Helper struct Whether any satisfaction of this fragment contains any timelocks
-#[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Debug, Hash)]
+#[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Debug, Default, Hash)]
 pub struct TimeLockInfo {
     /// csv with heights
     pub csv_with_height: bool,
@@ -27,18 +27,6 @@ pub struct TimeLockInfo {
     pub contains_combination: bool,
 }
 
-impl Default for TimeLockInfo {
-    fn default() -> Self {
-        Self {
-            csv_with_height: false,
-            csv_with_time: false,
-            cltv_with_height: false,
-            cltv_with_time: false,
-            contains_combination: false,
-        }
-    }
-}
-
 /// Helper struct to store information about op code limits. Note that this only
 /// counts the non-push opcodes. This is not relevant for TapScript context
 #[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Debug, Hash)]