Merge #801: Fix: make StaticDebugAndDisplay Send and Sync

ea5b955 doc: fix ParseError::RelativeLockTime doc-comment (Chris Ricketts)
cd4ace2 fix: make StaticDebugAndDisplay Send & Sync (Chris Ricketts)

Pull request description:

  Fixes #800

ACKs for top commit:
  apoelstra:
    ACK ea5b955; successfully ran local tests

Tree-SHA512: 435dac54470272919ac22d79b4c3acdadb7ff0f830f7082b657ae8a9dbfe8ffc22e1334555a478ebb3b5bc6d4bf8e9a9e023af6947f6582d7c7fadf1cc18b1ae

Author: apoelstra

src/blanket_traits.rs

@@ -17,10 +17,11 @@ use core::{fmt, hash};
 
 use crate::MiniscriptKey;
 
-/// Auxiliary trait indicating that a type implements both `Debug` and `Display`.
-pub trait StaticDebugAndDisplay: fmt::Debug + fmt::Display + 'static {}
+/// Auxiliary trait indicating that a type implements both `Debug`, `Display`, `Send` and `Sync`.
+// NOTE: `Send` / `Sync` is required to maintain compatibility with downstream error handling libraries.
+pub trait StaticDebugAndDisplay: fmt::Debug + fmt::Display + Send + Sync + 'static {}
 
-impl<T: fmt::Debug + fmt::Display + 'static> StaticDebugAndDisplay for T {}
+impl<T: fmt::Debug + fmt::Display + Send + Sync + 'static> StaticDebugAndDisplay for T {}
 
 /// Blanket trait describing a key where all associated types implement `FromStr`,
 /// and all `FromStr` errors can be displayed.

src/error.rs

@@ -18,7 +18,7 @@ use crate::Box;
 pub enum ParseError {
     /// Invalid absolute locktime
     AbsoluteLockTime(AbsLockTimeError),
-    /// Invalid absolute locktime
+    /// Invalid relative locktime
     RelativeLockTime(RelLockTimeError),
     /// Failed to parse a public key or hash.
     ///