Skip to content

gh-114563: C decimal falls back to pydecimal for unsupported format strings #114879

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
serhiy-storchaka merged 12 commits into from
Feb 12, 2024
Merged

gh-114563: C decimal falls back to pydecimal for unsupported format strings #114879

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
d49d679
add decimal tests
belm0 Feb 1, 2024
b526004
revert 'z' format support in C decimal
belm0 Feb 1, 2024
6ac60d7
gh-114563: C decimal falls back to pydecimal for unsupported format s…
belm0 Feb 1, 2024
d37b5b8
Apply suggestions from code review
belm0 Feb 10, 2024
d2b6893
disable failing null fill test
belm0 Feb 10, 2024
feb7469
only override message string on ValueError
belm0 Feb 10, 2024
5803d9c
corrected null fill char test
belm0 Feb 10, 2024
dcee58f
honor round and capitals context in fallback
belm0 Feb 11, 2024
7a2ba4b
format fallback context tests
belm0 Feb 11, 2024
ab4f2d7
pass context directly to __format__()
belm0 Feb 11, 2024
3b1b8f4
📜🤖 Added by blurb_it.
blurb-it[bot] Feb 11, 2024
06e371c
fix NEWS formatting
belm0 Feb 11, 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
Apply suggestions from code review 
Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
  • Loading branch information
@belm0 @serhiy-storchaka
belm0 and serhiy-storchaka authored Feb 10, 2024
commit d37b5b81c582a8964fe83683e88660e9c4d4017b
1 change: 1 addition & 0 deletions Lib/test/test_decimal.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1110,6 +1110,7 @@ def test_formatting(self):
('z>z6.1f', '-0.', 'zzz0.0'),
('x>z6.1f', '-0.', 'xxx0.0'),
('🖤>z6.1f', '-0.', '🖤🖤🖤0.0'), # multi-byte fill char
Comment thread
belm0 marked this conversation as resolved.
('\0>z6.1f', '-0.', '\0\0\00.0'), # null fill char
Comment thread
belm0 marked this conversation as resolved.
Outdated

# issue 114563 ('z' format on F type in cdecimal)
('z3,.10F', '-6.24E-323', '0.0000000000'),
Expand Down
13 changes: 3 additions & 10 deletions Modules/_decimal/_decimal.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3377,14 +3377,7 @@ pydec_format(PyObject *dec, PyObject *fmt, decimal_state *state)
PyObject *u;

if (state->PyDecimal == NULL) {
PyObject *obj = PyImport_ImportModule("_pydecimal");
if (obj == NULL) {
return NULL;
}

state->PyDecimal = PyObject_GetAttrString(obj, "Decimal");
Py_DECREF(obj);

state->PyDecimal = _PyImport_GetModuleAttrString("_pydecimal", "Decimal");
if (state->PyDecimal == NULL) {
return NULL;
}
Expand All @@ -3395,13 +3388,13 @@ pydec_format(PyObject *dec, PyObject *fmt, decimal_state *state)
return NULL;
}

pydec = PyObject_CallFunctionObjArgs(state->PyDecimal, u, NULL);
pydec = PyObject_CallOneArg(state->PyDecimal, u);
Comment thread
belm0 marked this conversation as resolved.
Py_DECREF(u);
if (pydec == NULL) {
return NULL;
}

result = PyObject_CallMethod(pydec, "__format__", "(O)", fmt);
result = PyObject_Format(pydec, fmt);
Py_DECREF(pydec);

if (result == NULL) {
Comment thread
belm0 marked this conversation as resolved.
Outdated
Expand Down