Merge #358: lib: implement PartialEq for miniscript::Error

1356f03 lib: implement PartialEq for miniscript::Error (Antoine Poinsot)

Pull request description:

  This helps a lot downstream, and we can do that now that rust-bitcoin derives `PartialEq` everywhere.

ACKs for top commit:
  tcharding:
    ACK 1356f03
  sanket1729:
    ACK 1356f03

Tree-SHA512: 112e56bf58402f65e41263d4cb51e7826d58d0130594ff51b794c77c7852e67801056155d6fbb78d7adec425db45fb25f768a60ce80eb6b9536aaff72e322f2b

Author: sanket1729

src/lib.rs

@@ -497,7 +497,7 @@ pub trait ForEachKey<Pk: MiniscriptKey> {
 
 /// Miniscript
 
-#[derive(Debug)]
+#[derive(Debug, PartialEq)]
 pub enum Error {
     /// Opcode appeared which is not part of the script subset
     InvalidOpcode(opcodes::All),

src/miniscript/analyzable.rs

@@ -31,7 +31,7 @@ use {Miniscript, MiniscriptKey, ScriptContext};
 /// 3. The script is malleable and thereby some of satisfaction weight
 ///    guarantees are not satisfied.
 /// 4. It has repeated publickeys
-#[derive(Debug)]
+#[derive(Debug, PartialEq)]
 pub enum AnalysisError {
     /// Top level is not safe.
     SiglessBranch,