From: Rafal Luzynski <digitalfreak@lingonborough.com>
To: Zack Weinberg <zackw@panix.com>, TAMUKI Shoichi <tamuki@linet.gr.jp>
Cc: GNU C Library <libc-alpha@sourceware.org>
Subject: Re: [PATCH v6 2/2] strftime: Pass the additional flags from "%EY" to "%Ey" [BZ #23758]
Date: Thu, 17 Jan 2019 18:55:00 -0000 [thread overview]
Message-ID: <184032555.404911.1547751249485@poczta.nazwa.pl> (raw)
In-Reply-To: <CAKCAbMh3=uxMyzMY0k=LcnWQZsyedHJuhtCKfDrpNwPFpfkhaQ@mail.gmail.com>
16.01.2019 17:17 Zack Weinberg <zackw@panix.com> wrote:
>
> This review covers only the documentation and commit message.
Thank you Zack and Paul for your reviews. They were the most
missing part. I believe we will help ourselves with the rest
of the patches.
> [...]
> (It seems to me that this extension ought to be generalized to all of
> the "macro" formats (%c, %D, %F, %r, %R, %T, %x, %X, %Ec, %Ex, %EX),
> and to all format flags and field widths, but that would be a separate
> patch and not appropriate for 2.29 at this point.)
This may be too ambiguous and therefore impossible to implement.
> [...]
> Paul's suggested revision of this addition is technically incorrect;
> he got confused about which way around the flag propagates. I would
> recommend using a separate bullet point for this change, and I would
> also recommend not describing the behavior in terms of implementation
> details:
>
> | * As a GNU extension, the '-' and '_' flags can now be applied to '%EY'
> | to control how the year number is formatted; they have the same effect
> | that they would on %Ey.
"they would" or "they would have"?
Also, shouldn't all format specifiers be consequently quoted, like "%EY"
and "%Ey"? I don't mind single quotes, especially for the flags, I just
think that %Ey (without any quotes) may not be absolutely clear.
Regards,
Rafal
next prev parent reply other threads:[~2019-01-17 18:55 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-11 4:48 [PATCH v6 0/2] strftime: Improve the width of alternative representation for year " TAMUKI Shoichi
2019-01-11 4:50 ` [PATCH v6 1/2] strftime: Set the default width of "%Ey" to 2 " TAMUKI Shoichi
2019-01-16 16:17 ` Zack Weinberg
2019-01-17 6:31 ` TAMUKI Shoichi
2019-01-17 17:57 ` Rafal Luzynski
2019-01-18 2:46 ` Zack Weinberg
2019-01-18 14:01 ` TAMUKI Shoichi
2019-01-18 18:35 ` Rafal Luzynski
2019-01-18 14:01 ` TAMUKI Shoichi
2019-01-18 18:44 ` Rafal Luzynski
2019-01-19 3:52 ` TAMUKI Shoichi
2019-01-19 17:08 ` Zack Weinberg
2019-01-19 21:42 ` Rafal Luzynski
2019-01-11 4:53 ` [PATCH v6 2/2] strftime: Pass the additional flags from "%EY" to "%Ey" " TAMUKI Shoichi
2019-01-16 16:18 ` Zack Weinberg
2019-01-17 18:55 ` Rafal Luzynski [this message]
2019-01-18 2:32 ` Zack Weinberg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=184032555.404911.1547751249485@poczta.nazwa.pl \
--to=digitalfreak@lingonborough.com \
--cc=libc-alpha@sourceware.org \
--cc=tamuki@linet.gr.jp \
--cc=zackw@panix.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).