Skip to content

gh-127787: refactor helpers for PyUnicodeErrorObject internal interface #127789

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
encukou merged 21 commits into from
Jan 3, 2025
Merged

gh-127787: refactor helpers for PyUnicodeErrorObject internal interface #127789

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
0cc7bc7
Refactor helpers for `PyUnicodeErrorObject`.
picnixz Dec 10, 2024
32a199d
put comment section headers
picnixz Dec 10, 2024
2583095
add comments
picnixz Dec 10, 2024
f0893b7
simpler checks
picnixz Dec 10, 2024
a4b01f0
fix tests
picnixz Dec 10, 2024
01b5f22
unify even more the interface using a generic getter
picnixz Dec 10, 2024
be982a0
remove useless `consistent` parameter
picnixz Dec 10, 2024
d4dc9a6
Simplify call
picnixz Dec 11, 2024
3078f23
Merge branch 'main' into feat/exc/unicode-error-refactor-127787
picnixz Dec 13, 2024
94800fd
remove unused function
picnixz Dec 13, 2024
e5709fa
remove un-necessary macros
picnixz Dec 13, 2024
83eb24d
Update Objects/exceptions.c
picnixz Jan 2, 2025
7c9fd99
style update
picnixz Jan 3, 2025
8219be9
use macro for repeated names to avoid typos
picnixz Jan 3, 2025
4a5e4e3
specialize _PyUnicodeError_GetParams for start and end attributes
picnixz Jan 3, 2025
f7a2efa
update comments
picnixz Jan 3, 2025
17367ff
put 2 blank lines before sections
picnixz Jan 3, 2025
c05f2ad
add NULL assertions to avoid obscure segmentation faults
picnixz Jan 3, 2025
6c07afc
update comments
picnixz Jan 3, 2025
ebf86c7
fixup comment
picnixz Jan 3, 2025
3fb81c2
fixup comment
picnixz Jan 3, 2025
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
Simplify call
  • Loading branch information
@picnixz
picnixz committed Dec 11, 2024
commit d4dc9a6814e3006c6e7a6d59e9a6c4e33e8398f7
2 changes: 1 addition & 1 deletion Objects/exceptions.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2753,7 +2753,7 @@ unicode_error_set_reason_impl(PyObject *self, const char *reason)
// TODO(picnixz): do an assert-only type-check when gh-127694 is merged
// (the caller function must do an eager type-check)
PyUnicodeErrorObject *exc = (PyUnicodeErrorObject *)self;
Py_XSETREF(*&exc->reason, value);
Py_XSETREF(exc->reason, value);
return 0;
}

Expand Down