Skip to content

Conversation

@ValuedMammal
Copy link
Collaborator

This patch improves readability and maintains logical consistency with the use of spk_cache throughout the keychain_txout module.

While it might offer a performance benefit, the results are mostly comparable with the current benchmarks as far as I can tell. At least there's no indication that it would negatively impact performance.

fixes #1975

Changelog notice

Checklists

All Submissions:

… last revealed

This change improves readability and maintains logical consistency with
the use of `spk_cache` throughout the `keychain_txout` module.
@ValuedMammal ValuedMammal moved this to Needs Review in BDK Chain Jul 14, 2025
@ValuedMammal ValuedMammal self-assigned this Jul 14, 2025
@ValuedMammal ValuedMammal added this to the Wallet 2.1.0 milestone Jul 14, 2025
@ValuedMammal ValuedMammal moved this to Needs Review in BDK Wallet Jul 14, 2025
Copy link
Contributor

@LagginTimes LagginTimes left a comment

Choose a reason for hiding this comment

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

ACK 6ddecc9

@ValuedMammal ValuedMammal merged commit edf0279 into bitcoindevkit:master Jul 17, 2025
19 checks passed
@github-project-automation github-project-automation bot moved this from Needs Review to Done in BDK Chain Jul 17, 2025
@github-project-automation github-project-automation bot moved this from Needs Review to Done in BDK Wallet Jul 17, 2025
@ValuedMammal ValuedMammal deleted the fix/keychain_txout_apply_changeset branch July 17, 2025 18:27
@notmandatory notmandatory added the bug Something isn't working label Jul 28, 2025
@oleonardolima oleonardolima mentioned this pull request Jul 31, 2025
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Archived in project
Archived in project

Development

Successfully merging this pull request may close these issues.

keychain_txout: apply_changeset should restore cache before applying last revealed

3 participants