Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce crate level decoding functions #166

Merged
merged 1 commit into from
Mar 8, 2025

Conversation

tcharding
Copy link
Member

We would like to release a bare-bones 1.0 version of this crate that excludes the FromHex trait. So that users can still decode vectors and arrays without the trait introduced two new decoding functions.

Intentionally do not use decode as the name incase we later want to add a more general decode function.

@tcharding tcharding marked this pull request as ready for review March 3, 2025 22:03
@tcharding
Copy link
Member Author

Pulled out of #162

tcharding added a commit to tcharding/hex-conservative that referenced this pull request Mar 3, 2025
WIP because this should come from `master` but we are waiting on rust-bitcoin#166 to
merge.

`master` is at tip: TOOD

Introduce decoding logic from `master`, this means:

Public API consisting of:

- The crate level decoding functions
- The associated error types

Private code:

- Code in private modules `parse` and `iter` as needed.
@tcharding tcharding force-pushed the 03-04-decodeing-api branch 2 times, most recently from c9531ac to e50d71c Compare March 6, 2025 01:32
We would like to release a bare-bones 1.0 version of this crate that
excludes the `FromHex` trait. So that users can still decode vectors
and arrays without the trait introduced two new decoding functions.

Intentionally do not use `decode` as the name incase we later want to
add a more general `decode` function.
@tcharding tcharding force-pushed the 03-04-decodeing-api branch from e50d71c to 97222f2 Compare March 6, 2025 01:34
Copy link
Collaborator

@Kixunil Kixunil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 97222f2

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 97222f2; successfully ran local tests

@apoelstra apoelstra merged commit 6de170e into rust-bitcoin:master Mar 8, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants