git: f0a15aafcb86 - main - strfmon: Remove XXX marks
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 29 Oct 2022 21:10:55 UTC
The branch main has been updated by kib:
URL: https://cgit.FreeBSD.org/src/commit/?id=f0a15aafcb863f796e2a7f94d0fd8a857fb56103
commit f0a15aafcb863f796e2a7f94d0fd8a857fb56103
Author: Jose Luis Duran <jlduran@gmail.com>
AuthorDate: 2022-10-28 09:20:33 +0000
Commit: Konstantin Belousov <kib@FreeBSD.org>
CommitDate: 2022-10-29 21:09:32 +0000
strfmon: Remove XXX marks
phantom@'s HDD crashed with the final version of strfmon.c, as explained
in 9d430a5991d3f64a75fee951a1efab3593207832.
Now there are tests in place that cover these code paths.
Reviewed by: kib
PR: 267410
Github PR: #620
MFC after: 1 week
---
lib/libc/stdlib/strfmon.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/libc/stdlib/strfmon.c b/lib/libc/stdlib/strfmon.c
index aee053a26ec8..ba02544eb2ec 100644
--- a/lib/libc/stdlib/strfmon.c
+++ b/lib/libc/stdlib/strfmon.c
@@ -324,7 +324,7 @@ vstrfmon_l(char * __restrict s, size_t maxsize, locale_t loc,
if (cs_precedes == 1) {
if (sign_posn == 1 || sign_posn == 3) {
PRINTS(signstr);
- if (sep_by_space == 2) /* XXX: ? */
+ if (sep_by_space == 2)
PRINT(' ');
}
@@ -363,7 +363,7 @@ vstrfmon_l(char * __restrict s, size_t maxsize, locale_t loc,
|| sign_posn == 2
|| sign_posn == 4)))
PRINT(space_char);
- PRINTS(currency_symbol); /* XXX: len */
+ PRINTS(currency_symbol);
if (sign_posn == 4) {
if (sep_by_space == 2)
PRINT(' ');
@@ -597,7 +597,7 @@ __format_grouped_double(double value, int *flags,
}
if ((*flags & NEED_GROUPING) &&
- thousands_sep_size > 0 && /* XXX: need investigation */
+ thousands_sep_size > 0 &&
*grouping != CHAR_MAX &&
*grouping > 0) {
while (avalue_size > (int)*grouping) {