Remove redundant else statements #32886
pull Raimo33 wants to merge 1 commits into bitcoin:master from Raimo33:master changing 1 files +9 −6-
Raimo33 commented at 6:44 pm on July 6, 2025: none
-
Remove redundant else statements 991408f149
-
DrahtBot commented at 6:44 pm on July 6, 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/32886.
Reviews
See the guideline for information on the review process. A summary of reviews will appear here.
-
Raimo33 closed this on Jul 6, 2025
-
Raimo33 reopened this on Jul 6, 2025
-
l0rinc commented at 6:46 pm on July 6, 2025: contributorPlease stop spamming, a lot of people are getting email notifications in these cases.
-
Raimo33 commented at 6:58 pm on July 6, 2025: noneGot it. But I see a lot of bad code through the whole repo
-
pinheadmz commented at 7:11 pm on July 6, 2025: member
Please don’t open new pull requests just for tiny typo fixes, they are a drag on our integration testing infrastructure, maintainer time and reviewer time. See https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#refactoring:
Trivial pull requests or pull requests that refactor the code with no clear benefits may be immediately closed by the maintainers to reduce unnecessary workload on reviewing.
There are many more significant ways to contribute to Bitcoin – for example, look for good first issues.
-
pinheadmz closed this on Jul 6, 2025
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-07-07 21:13 UTC
More mirrored repositories can be found on mirror.b10c.me