diff options
| author | pfg <pfg@FreeBSD.org> | 2014-07-19 15:12:38 +0000 |
|---|---|---|
| committer | pfg <pfg@FreeBSD.org> | 2014-07-19 15:12:38 +0000 |
| commit | c62a7a5557c69ee25d4bb0aff7674251e1a34ad2 (patch) | |
| tree | 0f915ff5f70272d70d7b134974d581d52884207e /lib | |
| parent | 46a564acf940ca3c2d7c926cfa87df7839ff5fb2 (diff) | |
| download | FreeBSD-src-c62a7a5557c69ee25d4bb0aff7674251e1a34ad2.zip FreeBSD-src-c62a7a5557c69ee25d4bb0aff7674251e1a34ad2.tar.gz | |
MFC r268569:
strfmon: reduce unnecessary snprintf.
No need for the snprintf/asprintf dance; use fixed width formats.
Obtained from: NetBSD (CVS rev. 1.8)
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/libc/stdlib/strfmon.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/lib/libc/stdlib/strfmon.c b/lib/libc/stdlib/strfmon.c index b82797d..689c78b 100644 --- a/lib/libc/stdlib/strfmon.c +++ b/lib/libc/stdlib/strfmon.c @@ -526,7 +526,6 @@ __format_grouped_double(double value, int *flags, char *rslt; char *avalue; int avalue_size; - char fmt[32]; size_t bufsize; char *bufend; @@ -567,14 +566,13 @@ __format_grouped_double(double value, int *flags, left_prec += get_groups(left_prec, grouping); /* convert to string */ - snprintf(fmt, sizeof(fmt), "%%%d.%df", left_prec + right_prec + 1, - right_prec); - avalue_size = asprintf(&avalue, fmt, value); + avalue_size = asprintf(&avalue, "%*.*f", left_prec + right_prec + 1, + right_prec, value); if (avalue_size < 0) return (NULL); /* make sure that we've enough space for result string */ - bufsize = strlen(avalue)*2+1; + bufsize = avalue_size * 2 + 1; rslt = calloc(1, bufsize); if (rslt == NULL) { free(avalue); |
