scripts: deduplicate leading-whitespace validation in buildtable.pl #2081

pull vtjl10 wants to merge 1 commits into bitcoin:master from vtjl10:patch-1 changing 1 files +4 −2

vtjl10 commented at 3:46 pm on January 14, 2026: none
- Remove duplicated $val =~ /^\s/ validation while parsing BIP preamble fields in scripts/buildtable.pl.
- Keep the exact existing error messages (“Too many spaces…” vs “Extra spaces…”) depending on the parsing branch.
The same leading-whitespace condition was checked in two places, which is easy to let drift over time and makes the parsing logic harder to follow. Consolidating it removes duplicate code without changing behavior.
Update buildtable.pl c7859ff767
murchandamus commented at 7:29 pm on January 14, 2026: contributor

No thanks. This is not a substantial improvement, but a stylistic preference. This is the third busywork pull request you have opened to this repository. Stop wasting our time with AI-powered commit farming.
murchandamus closed this on Jan 14, 2026