Files
bitcoin/src/wallet
merge-script 20ad7c9eab Merge bitcoin/bitcoin#35955: wallet: remove orphaned GetAffectedKeys and LegacyScriptPubKeyMan declarations
02de12b1e6 wallet: remove remaining LegacyScriptPubKeyMan references (Laxman Acharya)
d194be69d6 wallet: remove orphaned GetAffectedKeys declaration (Laxman Acharya)

Pull request description:

  Follow-up to #28710, which removed `GetAffectedKeys()` and `LegacyScriptPubKeyMan` but left their declarations behind.

  Remove both orphaned declarations, rename `SetupLegacyScriptPubKeyMan()` to `SetupLegacyDataSPKM()`, and update related comments and logging to reflect the minimal `LegacyDataSPKM` retained for legacy wallet loading and migration.

  ## Testing

  ```bash
  cmake --build build --target bitcoin_wallet -j 8
  cmake --build build --target test_bitcoin -j 8
  build/bin/test_bitcoin --run_test=scriptpubkeyman_tests,walletdb_tests,wallet_tests --catch_system_errors=no --log_level=error
  ```

ACKs for top commit:
  achow101:
    ACK 02de12b1e6
  polespinasa:
    ACK 02de12b1e6

Tree-SHA512: 0d340291c969f013fd6ec55158f6797d4a1478ff7a819c2d250d1b3f345131db99a8b7b4b3ff0da1fa0e06182701abed31dcd317f8ef12c81837c1aed57b21a5
2026-08-18 09:45:34 +01:00
..