Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix pollard serialization/deserialization (#46), Debug/Display for empty pollard (#48) #47

Merged
merged 1 commit into from
May 20, 2024

Conversation

Ash-L2L
Copy link
Contributor

@Ash-L2L Ash-L2L commented Apr 24, 2024

fixes #46, #48.
This fix is not backwards-compatible as the serialization encoding is changed. I have bumped the version number to reflect this.

@Ash-L2L Ash-L2L force-pushed the main branch 2 times, most recently from 2652dd7 to 64d5cf0 Compare April 24, 2024 13:31
@Ash-L2L Ash-L2L changed the title Fix pollard serialization/deserialization ( #46) Fix pollard serialization/deserialization (#46), Debug/Display for empty pollard (#47) Apr 29, 2024
@Ash-L2L Ash-L2L changed the title Fix pollard serialization/deserialization (#46), Debug/Display for empty pollard (#47) Fix pollard serialization/deserialization (#46), Debug/Display for empty pollard (#48) Apr 29, 2024
@Davidson-Souza
Copy link
Collaborator

Thank you for this PR! Just some comments

Cargo.toml Outdated Show resolved Hide resolved
src/accumulator/pollard.rs Outdated Show resolved Hide resolved
src/accumulator/pollard.rs Show resolved Hide resolved
@Davidson-Souza Davidson-Souza merged commit d8a1aea into mit-dci:main May 20, 2024
20 checks passed
Davidson-Souza added a commit to Davidson-Souza/rustreexo that referenced this pull request Jun 8, 2024
After we broadcast a transaction, this should reflect our address's
balance
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Serialization/Deserialization fails for empty node hashes
2 participants