This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
[PATCH 2/3] Fix gcvt to always show 'ndigits' of precision
- From: Keith Packard <keithp at keithp dot com>
- To: newlib at sourceware dot org
- Cc: Keith Packard <keithp at keithp dot com>
- Date: Tue, 17 Dec 2019 22:00:48 -0800
- Subject: [PATCH 2/3] Fix gcvt to always show 'ndigits' of precision
- References: <20191218060049.660325-1-keithp@keithp.com>
Leading zeros after the decimal point should not count
towards the 'ndigits' limit.
This makes gcvt match glibc and the posix gcvt man page.
Signed-off-by: Keith Packard <keithp@keithp.com>
---
newlib/libc/stdlib/ecvtbuf.c | 18 +++++++-----------
1 file changed, 7 insertions(+), 11 deletions(-)
diff --git a/newlib/libc/stdlib/ecvtbuf.c b/newlib/libc/stdlib/ecvtbuf.c
index 0a5d41a0f..8936363ce 100644
--- a/newlib/libc/stdlib/ecvtbuf.c
+++ b/newlib/libc/stdlib/ecvtbuf.c
@@ -349,15 +349,10 @@ _gcvt (struct _reent *ptr,
char *end;
char *p;
- if (invalue < 1.0)
- {
- /* what we want is ndigits after the point */
- p = _dtoa_r (ptr, invalue, 3, ndigit, &decpt, &sign, &end);
- }
- else
- {
- p = _dtoa_r (ptr, invalue, 2, ndigit, &decpt, &sign, &end);
- }
+ /* We always want ndigits of precision, even if that means printing
+ * a bunch of leading zeros for numbers < 1.0
+ */
+ p = _dtoa_r (invalue, 2, ndigit, &decpt, &sign, &end);
if (decpt == 9999)
{
@@ -383,11 +378,12 @@ _gcvt (struct _reent *ptr,
if (buf == save)
*buf++ = '0';
*buf++ = '.';
- while (decpt < 0 && ndigit > 0)
+
+ /* Leading zeros don't count towards 'ndigit' */
+ while (decpt < 0)
{
*buf++ = '0';
decpt++;
- ndigit--;
}
/* Print rest of stuff */
--
2.24.0