From 254ed12b8f80bf3b3846281bae981d10fefc92c1 Mon Sep 17 00:00:00 2001 From: hniksic Date: Wed, 10 Aug 2005 05:40:31 -0700 Subject: [PATCH] [svn] Doc fix. --- src/utils.c | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/src/utils.c b/src/utils.c index be0dd97a..697ebcf2 100644 --- a/src/utils.c +++ b/src/utils.c @@ -1210,11 +1210,10 @@ free_keys_and_values (struct hash_table *ht) { hash_table_map (ht, free_keys_and_values_mapper, NULL); } - -/* Get grouping data, the separator and grouping info, by calling - localeconv(). The information is cached after the first call to - the function. +/* Get digit grouping data for thousand separors by calling + localeconv(). The data includes separator string and grouping info + and is cached after the first call to the function. In locales that don't set a thousand separator (such as the "C" locale), this forces it to be ",". We are now only showing @@ -1258,8 +1257,8 @@ get_grouping_data (const char **sep, const char **grouping) Unfortunately, we cannot use %'d (in fact it would be %'j) to get the separators because it's too non-portable, and it's hard to test - for this feature at configure time. Besides, it wouldn't work in - the "C" locale, which many Unix users still work in. */ + for this feature at configure time. Besides, it wouldn't display + separators in the "C" locale, still used by many Unix users. */ const char * with_thousand_seps (wgint n) @@ -1283,8 +1282,8 @@ with_thousand_seps (wgint n) atgroup = grouping; groupsize = *atgroup++; - /* This will overflow on WGINT_MIN, but we're not using this to - print negative numbers anyway. */ + /* This would overflow on WGINT_MIN, but printing negative numbers + is not an important goal of this fuinction. */ if (negative) n = -n;