FundTx was used by the wallet_notifications target which we recently removed. So it's now unused and can be removed.
fuzz: wallet: remove `FundTx` from `FuzzedWallet` #32772
pull brunoerg wants to merge 1 commits into bitcoin:master from brunoerg:2025-06-fuzz-delete-fundtx changing 1 files +0 −53-
brunoerg commented at 7:21 PM on June 18, 2025: contributor
-
fuzz: wallet: remove FundTx from FuzzedWallet cd1ae1b4df
- DrahtBot added the label Tests on Jun 18, 2025
-
DrahtBot commented at 7:21 PM on June 18, 2025: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32772.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK maflcko, dergoegge, kevkevinpal If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #32750 (refactor: CFeeRate encapsulates FeeFrac internally by polespinasa)
- #32618 (wallet: Remove ISMINE_WATCHONLY and watchonly from RPCs by achow101)
- #32523 (wallet: Remove watchonly behavior and isminetypes by achow101)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
-
-
maflcko commented at 7:55 PM on June 18, 2025: member
lgtm ACK cd1ae1b4dfdb1bb11dec2558448a453f43cb2ac3
- dergoegge approved
-
dergoegge commented at 10:05 AM on June 19, 2025: member
utACK cd1ae1b4dfdb1bb11dec2558448a453f43cb2ac3
-
kevkevinpal commented at 1:25 PM on June 19, 2025: contributor
ACK cd1ae1b
makes sense to remove dead code
- fanquake merged this on Jun 19, 2025
- fanquake closed this on Jun 19, 2025
- brunoerg deleted the branch on Jun 19, 2025
- stringintech referenced this in commit f19e7819e9 on Jul 19, 2025
- alexanderwiederin referenced this in commit 9997c3d993 on Jul 25, 2025
- alexanderwiederin referenced this in commit 8d6ebb9c4e on Jul 28, 2025
- alexanderwiederin referenced this in commit 533a4585b5 on Jul 28, 2025
- yuvicc referenced this in commit 22f55cf11d on Aug 26, 2025
- bug-castercv502 referenced this in commit d4bd8a1259 on Sep 28, 2025
- fanquake added the label Fuzzing on Oct 30, 2025
Contributors