Change Details

Seems a bit confusing to happily accept random bytes and pretend they are hex encoded strings Added a new TryParseHex function returning an optional. The old function remains an alias, with the fallback to an empty vector. This avoid having to change all existing code and making it needlessly verbose, because it is already properly handling empty vectors. This is a backport of [[https://github.com/bitcoin/bitcoin/pull/25227 | core#25227]] Fix a hex string in checkpoints_tests that was obviously wrong (odd number of hex chars) and now causes ParseHex to return an empty vector.

Seems a bit confusing to happily accept random bytes and pretend they are hex encoded strings Added a new TryParseHex function returning an optional. The old function remains an alias, with the fallback to an empty vector. This avoid having to change all existing code and making it needlessly verbose, because it is already properly handling empty vectors. This is a backport of [[https://github.com/bitcoin/bitcoin/pull/25227 | core#25227]] Fix a hex string in checkpoints_tests that was obviously wrong (odd number of hex chars) and now causes ParseHex to return an empty vector. Depends on D17157

Seems a bit confusing to happily accept random bytes and pretend they are hex encoded strings Added a new TryParseHex function returning an optional. The old function remains an alias, with the fallback to an empty vector. This avoid having to change all existing code and making it needlessly verbose, because it is already properly handling empty vectors. This is a backport of [[https://github.com/bitcoin/bitcoin/pull/25227 | core#25227]] Fix a hex string in checkpoints_tests that was obviously wrong (odd number of hex chars) and now causes ParseHex to return an empty vector. Depends on D17157