Issue #22090: Fix '%' formatting for infinities and NaNs.

This commit is contained in:
Stefan Krah 2014-08-26 20:46:49 +02:00
parent d84fd73de2
commit 298131a448
3 changed files with 14 additions and 4 deletions

View File

@ -3769,6 +3769,8 @@ class Decimal(object):
if self._is_special: if self._is_special:
sign = _format_sign(self._sign, spec) sign = _format_sign(self._sign, spec)
body = str(self.copy_abs()) body = str(self.copy_abs())
if spec['type'] == '%':
body += '%'
return _format_align(sign, body, spec) return _format_align(sign, body, spec)
# a type of None defaults to 'g' or 'G', depending on context # a type of None defaults to 'g' or 'G', depending on context

View File

@ -1057,6 +1057,11 @@ class FormatTest(unittest.TestCase):
# issue 6850 # issue 6850
('a=-7.0', '0.12345', 'aaaa0.1'), ('a=-7.0', '0.12345', 'aaaa0.1'),
# issue 22090
('<^+15.20%', 'inf', '<<+Infinity%<<<'),
('\x07>,%', 'sNaN1234567', 'sNaN1234567%'),
('=10.10%', 'NaN123', ' NaN123%'),
] ]
for fmt, d, result in test_values: for fmt, d, result in test_values:
self.assertEqual(format(Decimal(d), fmt), result) self.assertEqual(format(Decimal(d), fmt), result)

View File

@ -446,7 +446,7 @@ _mpd_to_string(char **result, const mpd_t *dec, int flags, mpd_ssize_t dplace)
if (mpd_isspecial(dec)) { if (mpd_isspecial(dec)) {
mem = sizeof "-Infinity"; mem = sizeof "-Infinity%";
if (mpd_isnan(dec) && dec->len > 0) { if (mpd_isnan(dec) && dec->len > 0) {
/* diagnostic code */ /* diagnostic code */
mem += dec->digits; mem += dec->digits;
@ -609,10 +609,10 @@ _mpd_to_string(char **result, const mpd_t *dec, int flags, mpd_ssize_t dplace)
*cp++ = (flags&MPD_FMT_UPPER) ? 'E' : 'e'; *cp++ = (flags&MPD_FMT_UPPER) ? 'E' : 'e';
cp = exp_to_string(cp, ldigits-dplace); cp = exp_to_string(cp, ldigits-dplace);
} }
}
if (flags&MPD_FMT_PERCENT) { if (flags&MPD_FMT_PERCENT) {
*cp++ = '%'; *cp++ = '%';
}
} }
assert(cp < decstring+mem); assert(cp < decstring+mem);
@ -1260,6 +1260,9 @@ mpd_qformat_spec(const mpd_t *dec, const mpd_spec_t *spec,
stackspec.align = '>'; stackspec.align = '>';
spec = &stackspec; spec = &stackspec;
} }
if (type == '%') {
flags |= MPD_FMT_PERCENT;
}
} }
else { else {
uint32_t workstatus = 0; uint32_t workstatus = 0;