.
. #33835
pull 777genius wants to merge 1 commits into bitcoin:master from 777genius:refactor/wallet-redundant-ternary changing 1 files +1 −1-
777genius commented at 6:15 PM on November 9, 2025: none
-
dce7ba0970
refactor: Remove redundant ternary operator in wallet HaveChain()
Replace 'return m_chain ? true : false' with 'return m_chain != nullptr' in the HaveChain() method. The ternary operator is redundant since the condition already evaluates to a boolean. The explicit nullptr comparison makes the intent clearer and is more idiomatic C++. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- DrahtBot added the label Refactoring on Nov 9, 2025
-
DrahtBot commented at 6:15 PM on November 9, 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/33835.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process. A summary of reviews will appear here.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
- bitcoin locked this on Nov 9, 2025
- achow101 renamed this:
refactor: Remove redundant ternary operator in wallet HaveChain()
.
on Nov 9, 2025 - bitcoin deleted a comment on Nov 9, 2025
- fanquake removed the label Refactoring on Nov 10, 2025
