payjoin / rust-payjoin / 17521365048

86%

master: 83%

push

github

Replace SystemTime with bitcoin::absolute::Time

Introduce a newtype wrapping bitcoin::absolute::Time (which is what BIP
77 specifies) instead of using std::time::SystemTime.

A fallible constructor `Time::from_now(Duration)` is used to construct
these values from a `Duration`, so `with_expiry` no longer takes a
`Duration` directly. No `IntoTime` trait analogous to `IntoUrl` is
included in this commit.

Introducing a newtype lets us add additional conversion and arithmetic
functionality later without breaking the API or relying on extension
traits on the bitcoin Time type.

Co-authored-by: Yuval Kogman <nothingmuch@woobling.org>

81 of 87 new or added lines in 7 files covered. (93.1%)

8205 of 9549 relevant lines covered (85.93%)

489.72 hits per line