hex_str_to_bytes helper
#22593
Use the built-in class method bytes.fromhex() instead, which is available since Python 3.0 (https://docs.python.org/3.0/library/stdtypes.html?highlight=fromhex#bytes.fromhex).
This would be nice to solve with a scripted-diff, but it’s tricky to also tackle the imports (which need to be removed in the same commit, otherwise the linter would complain).
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
No conflicts as of last run.
Use the built-in class method bytes.fromhex() instead,
which is available since Python 3.0.
Concept ACK, cr ACK ca6c154ef116e8d2e0484cdb1af13b34a0c86c17
Silly question, shouldn’t we also test bytes.fromhex()? Or maybe not if there’s enough constants checked in other places.
unhexlify can take str as well. There was already no need for the encode(). Though I guess using a built-in is slightly better.
bytes.fromhex name over the cryptic a2b_hex or the negation unhexlify. In any case it could make sense to remove the a2b_hex and unhexlify uses. It doesn’t really make sense to use four different ways (three after this pull), sometimes even mixed within the same file, to achieve the same outcome.