rust-bitcoin / rust-bitcoin / 13843194225

83%

push

github

Merge rust-bitcoin/rust-bitcoin#4235: Use impl_op_for_references macro in fee module

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/<a class="double-link" href="https://git"><a class=hub.com/rust-bitcoin/rust-bitcoin/commit/93c6c8cef59ceed56932d62daeb212c2b40fc4a1">93c6c8cef Use impl_op_for_references macro in fee module (Erick Cestari)

Pull request description:

  This pr replaces the individual operator implementations in the fee module with the impl_op_for_references macro to handle reference operations. This removes the need to manually implement reference combinations for operands, simplifying the code and improving consistency.

  ### Changes:
  - Replaces direct implementations of operators with macro usage
  - Adds tests to verify that reference operations work correctly
  - Maintains the same semantics as the original implementation

  Closes #4173

ACKs for top commit:
  tcharding:
    ACK 93c6c8cef59ceed56932d62daeb212c2b40fc4a1
  apoelstra:
    ACK 93c6c8cef59ceed56932d62daeb212c2b40fc4a1; successfully ran local tests; yeah, I think the docs loss is fine -- the docs are hard to find and say exactly what users expect of the / operator

Tree-SHA512: 51d7643c216e16518dc9463f5290528bca1

9 of 9 new or added lines in 1 file covered. (100.0%)

21788 of 26196 relevant lines covered (83.17%)

16835.27 hits per line