8000 gh-127740: For odd-length input to bytes.fromhex(...) change the error message to ValueError: fromhex() arg must be of even length by srinivasreddy · Pull Request #127756 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-127740: For odd-length input to bytes.fromhex(...) change the error message to ValueError: fromhex() arg must be of even length #127756

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 23 commits into from
Dec 11, 2024
Merged

gh-127740: For odd-length input to bytes.fromhex(...) change the error message to ValueError: fromhex() arg must be of even length #127756

Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
b5b0da7
gh-127740: For odd-length input to bytes.fromhex(...) change the erro…
srinivasreddy Dec 9, 2024
14b7d5e
Introduce space between variables
srinivasreddy Dec 9, 2024
f3bfe85
gh-127740: Improve code, consolidate two variables into one
srinivasreddy Dec 9, 2024
5a55e24
Check the logic inside of the loop, instead of checking eagerly
srinivasreddy Dec 9, 2024
a3ba0e7
Remove dead variable
srinivasreddy Dec 9, 2024
ee7947a
Fix a bug in logic
srinivasreddy Dec 9, 2024
879994e
Add some more tests
srinivasreddy Dec 9, 2024
ad96b89
Improve tests
srinivasreddy Dec 9, 2024
95744bd
Revert the change that is not necessary
srinivasreddy Dec 9, 2024
6929a4d
Move everthing to error labels
srinivasreddy Dec 9, 2024
aa6684b
Remove comments
srinivasreddy Dec 9, 2024
bf3ace9
Add tests and update logic
srinivasreddy Dec 9, 2024
3588d18
Update tests
srinivasreddy Dec 9, 2024
504f4ed
Add tests and update logic
srinivasreddy Dec 9, 2024
7bab683
Change the invalid_char value to -1
srinivasreddy Dec 9, 2024
7c819c1
Update test case
srinivasreddy Dec 9, 2024
862bfd5
gh-127740: Improve error messages as suggested by @hauntsaninja
srinivasreddy Dec 10, 2024
6cc9305
Update test cases
srinivasreddy Dec 10, 2024
c34696e
gh-127740: NULL at the end is mapped to 37
srinivasreddy Dec 10, 2024
82762db
gh-127740: Reintroduce the space back
srinivasreddy Dec 10, 2024
6628ea7
gh-127740: Check if we had a second digit
srinivasreddy Dec 10, 2024
b20e248
Add News entry
srinivasreddy Dec 10, 2024
cbf50b4
Fix lint
srinivasreddy Dec 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update tests
  • Loading branch information
@srinivasreddy
srinivasreddy committed Dec 9, 2024
commit 3588d181dc8c611a094397b754b2d6b830c7b0cc
2 changes: 1 addition & 1 deletion Lib/test/test_bytes.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -460,7 +460,7 @@ def test_fromhex(self):
self.assertRaises(ValueError, self.type2test.fromhex, '12 \x00 34')

# For odd number of character(s)
for value in ("a", "a ", " a"," a ", "a a a", "aa a ", " aa a", " aaa "):
for value in ("a", "a ", " a"," a ", "aaa", "aaa ", " aaa", " aaa "):
with self.assertRaises(ValueError) as cm:
self.type2test.fromhex(value)
self.assertIn("fromhex() arg must be of even length", str(cm.exception))
Expand Down
Loading
0