Skip to content

bpo-45995: add "z" format specifer to coerce negative 0 to zero #30049

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
mdickinson merged 25 commits into from
Apr 11, 2022
Merged

bpo-45995: add "z" format specifer to coerce negative 0 to zero #30049

Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
a633e88
bpo-45995: add "z" format specifer to coerce negative 0 to zero
belm0 Dec 7, 2021
636da05
formatting
belm0 Dec 11, 2021
3f4085d
implementation for Decimal
belm0 Dec 13, 2021
68a049e
📜🤖 Added by blurb_it.
blurb-it[bot] Dec 14, 2021
ad32be5
consistent flag names
belm0 Dec 14, 2021
3dcaf5f
add test case for integer value with z option
belm0 Feb 3, 2022
6b9ab3b
reference pending PEP
belm0 Feb 4, 2022
e243568
Apply some formatting and doc suggestions
belm0 Mar 16, 2022
be4fda2
revise "z" option description
belm0 Mar 19, 2022
043d76a
add test cases for explicit sign option
belm0 Mar 19, 2022
104e023
revise tests for format options expected to fail on floats
belm0 Mar 19, 2022
61c64df
"float presentation" -> "floating-point presentation"
belm0 Mar 19, 2022
76d61ae
news file terminating newline
belm0 Mar 19, 2022
33fe72c
add test coverage for Decimal bugs
belm0 Mar 19, 2022
9393136
Decimal: handle 'z' fill character correctly
belm0 Mar 21, 2022
f88f7fc
Decimal: const qualifier on fmt variable
belm0 Mar 21, 2022
20c9cf1
fix rounding of 'e', 'g', and '%' presentation types for Decimal
belm0 Mar 23, 2022
bf1a891
fix Decimal directed rounding
belm0 Mar 23, 2022
3f5b392
consistency among tests
belm0 Mar 23, 2022
8d7a745
fix stack-use-after-scope sanitizer error
belm0 Mar 23, 2022
2a24e61
clarify Decimal strategy
belm0 Mar 23, 2022
0cbff6a
fix Decimal format parsing
belm0 Apr 6, 2022
8e7b51c
fix Decimal when no precision is specified
belm0 Apr 7, 2022
418ab76
fix comment typo
belm0 Apr 7, 2022
3ee6f6b
add attribution to news blurb
belm0 Apr 11, 2022
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
Decimal: const qualifier on fmt variable
  • Loading branch information
@belm0
belm0 committed Mar 21, 2022
commit f88f7fcaa2426aaf4d8fd8f903e1100fa1a4e00a
6 changes: 3 additions & 3 deletions Modules/_decimal/_decimal.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3220,7 +3220,7 @@ dec_format(PyObject *dec, PyObject *args)
PyObject *fmtarg;
PyObject *context;
mpd_spec_t spec;
char *fmt;
char const *fmt;
char *fmt_copy = NULL;
char *decstring = NULL;
uint32_t status = 0;
Expand Down Expand Up @@ -3252,8 +3252,8 @@ dec_format(PyObject *dec, PyObject *args)
/* Strip 'z' option, which isn't understood by mpd_parse_fmt_str().
* First, skip [[fill]align], since 'fill' itself may be 'z'.
* NOTE: fmt is always null terminated by PyUnicode_AsUTF8AndSize() */
char *fmt_offset = size >= 2 && strchr("<>=^", fmt[1]) != NULL ?
fmt + 2 : fmt;
char const *fmt_offset = size >= 2 && strchr("<>=^", fmt[1]) != NULL ?
fmt + 2 : fmt;
char *z_start = strchr(fmt_offset, 'z');
Comment thread
belm0 marked this conversation as resolved.
Outdated
if (z_start != NULL) {
no_neg_0 = 1;
Expand Down