milksad
/
rust-bitcoin-unsafe-fast
14
0
Fork 0
Code Issues Pull Requests Activity

Remove non_exhaustive from struct errors with pub inner 

Using `non_exhaustive` as well as a public inner field is incorrect, it
prohibits users from creating or matching on the error and does not
achieve forward comparability.

This was never right, we shouldn't have done it.
This commit is contained in:
Tobin C. Harding 2024-02-05 16:26:31 +11:00
parent a3c4194c3f
commit 8c17ad7fd7
No known key found for this signature in database
GPG Key ID: 40BF9E4C269D6607
2 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
bitcoin/src/blockdata/transaction.rs
View File

@ -1042,7 +1042,6 @@ impl Transaction {
/// Error attempting to do an out of bounds access on the transaction inputs vector. /// Error attempting to do an out of bounds access on the transaction inputs vector.
#[derive(Debug, Clone, PartialEq, Eq)] #[derive(Debug, Clone, PartialEq, Eq)]
#[non_exhaustive]
pub struct InputsIndexError(pub IndexOutOfBoundsError); pub struct InputsIndexError(pub IndexOutOfBoundsError);
impl fmt::Display for InputsIndexError { impl fmt::Display for InputsIndexError {
@ -1062,7 +1061,6 @@ impl From<IndexOutOfBoundsError> for InputsIndexError {
/// Error attempting to do an out of bounds access on the transaction outputs vector. /// Error attempting to do an out of bounds access on the transaction outputs vector.
#[derive(Debug, Clone, PartialEq, Eq)] #[derive(Debug, Clone, PartialEq, Eq)]
#[non_exhaustive]
pub struct OutputsIndexError(pub IndexOutOfBoundsError); pub struct OutputsIndexError(pub IndexOutOfBoundsError);
impl fmt::Display for OutputsIndexError { impl fmt::Display for OutputsIndexError {

2
bitcoin/src/crypto/sighash.rs
View File

@ -504,7 +504,6 @@ impl TapSighashType {
/// Integer is not a consensus valid sighash type. /// Integer is not a consensus valid sighash type.
#[derive(Debug, Clone, PartialEq, Eq)] #[derive(Debug, Clone, PartialEq, Eq)]
#[non_exhaustive]
pub struct InvalidSighashTypeError(pub u32); pub struct InvalidSighashTypeError(pub u32);
impl fmt::Display for InvalidSighashTypeError { impl fmt::Display for InvalidSighashTypeError {
@ -521,7 +520,6 @@ impl std::error::Error for InvalidSighashTypeError {
/// This type is consensus valid but an input including it would prevent the transaction from /// This type is consensus valid but an input including it would prevent the transaction from
/// being relayed on today's Bitcoin network. /// being relayed on today's Bitcoin network.
#[derive(Debug, Clone, PartialEq, Eq)] #[derive(Debug, Clone, PartialEq, Eq)]
#[non_exhaustive]
pub struct NonStandardSighashTypeError(pub u32); pub struct NonStandardSighashTypeError(pub u32);
impl fmt::Display for NonStandardSighashTypeError { impl fmt::Display for NonStandardSighashTypeError {