parity-common icon indicating copy to clipboard operation
parity-common copied to clipboard

Published 20 hours ago • verified publisher icon paritytech

From<i64> impl is invalid

Open benluelo opened this issue 1 year ago • 2 comments

This implementation is invalid: https://github.com/paritytech/parity-common/blob/a5ef7308d6986e62431e35d3156fed0a7a585d39/uint/src/uint.rs#L1349-L1356

As per the docs on From:

Note: This trait must not fail. The From trait is intended for perfect conversions. If the conversion can fail or is not perfect, use TryFrom.

benluelo avatar Jul 07 '23 19:07 benluelo