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

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

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
4 changes: 4 additions & 0 deletions Misc/NEWS.d/next/Library/2024-02-11-20-23-36.gh-issue-114563.RzxNYT.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Fix several `format()` bugs when using the C implementation of `Decimal`:
* memory leak in some rare cases when using the `z` format option (coerce negative 0)
* incorrect output when applying the `z` format option to type `F` (fixed-point with capital `NAN` / `INF`)
* incorrect output when applying the `#` format option (alternate form)