This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [COMMITTED] Fix a few whitespace arrangement inconsistencies in time/strftime_l.c
- From: Carlos O'Donell <carlos at redhat dot com>
- To: TAMUKI Shoichi <tamuki at linet dot gr dot jp>, libc-alpha at sourceware dot org
- Date: Mon, 11 Feb 2019 08:26:05 -0500
- Subject: Re: [COMMITTED] Fix a few whitespace arrangement inconsistencies in time/strftime_l.c
- References: <201902110331.AA04238@tamuki.linet.gr.jp>
On 2/10/19 10:31 PM, TAMUKI Shoichi wrote:
> Having checked the arrangement of whitespace in time/strftime_l.c
> using "unexpand" and "unexpand -a" command, three inconsistencies are
> detected. So fix them for consistency.
>
> ChangeLog:
>
> * time/strftime_l.c: Fix a few whitespace arrangement inconsistencies.
Thank you for fixing such obvious issues! :-)
> ---
> time/strftime_l.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/time/strftime_l.c b/time/strftime_l.c
> index cb3b8d3..98c35d5 100644
> --- a/time/strftime_l.c
> +++ b/time/strftime_l.c
> @@ -562,7 +562,7 @@ __strftime_internal (CHAR_T *s, size_t maxsize, const CHAR_T *format,
> int pad = 0; /* Padding for number ('-', '_', or 0). */
> int modifier; /* Field modifier ('E', 'O', or 0). */
> int digits; /* Max digits for numeric format. */
> - int number_value; /* Numeric value to be printed. */
> + int number_value; /* Numeric value to be printed. */
> int negative_number; /* 1 if the number is negative. */
> const CHAR_T *subfmt;
> CHAR_T *bufp;
> @@ -992,7 +992,7 @@ __strftime_internal (CHAR_T *s, size_t maxsize, const CHAR_T *format,
> do
> *--bufp = u % 10 + L_('0');
> while ((u /= 10) != 0);
> - }
> + }
>
> do_number_sign_and_padding:
> if (negative_number)
> @@ -1136,7 +1136,7 @@ __strftime_internal (CHAR_T *s, size_t maxsize, const CHAR_T *format,
> DO_NUMBER (2, tp->tm_sec);
>
> case L_('s'): /* GNU extension. */
> - {
> + {
> struct tm ltm;
> time_t t;
>
>
--
Cheers,
Carlos.