refactor: Remove unused HaveKey and HaveWatchOnly #32476

pull maflcko wants to merge 2 commits into bitcoin:master from maflcko:2505-bdb-removal-2 changing 3 files +4 −21
  1. maflcko commented at 7:46 am on May 13, 2025: member
    removes some dead code
  2. Remove HaveKey helper, unused after sethdseed removal fa7b7f796a
  3. Remove unused LegacyDataSPKM::HaveWatchOnly() fabdc5ad06
  4. DrahtBot commented at 7:46 am on May 13, 2025: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32476.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK Sjors

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  5. DrahtBot added the label Refactoring on May 13, 2025
  6. maflcko commented at 7:47 am on May 13, 2025: member
    (follow-up to #32438)
  7. fanquake requested review from Sjors on May 13, 2025
  8. Sjors commented at 9:32 am on May 13, 2025: member
    utACK fabdc5ad06bce8cd3b0e525071d10f616a30bb04
  9. fanquake merged this on May 13, 2025
  10. fanquake closed this on May 13, 2025

  11. maflcko deleted the branch on May 13, 2025


maflcko DrahtBot Sjors


Sjors

Labels
Refactoring


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-05-30 00:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me