This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [RFC][PATCH v2 00/11][BZ 10871] Month names in genitive case
- From: Rical Jasan <ricaljasan at pacific dot net>
- To: Rafal Luzynski <digitalfreak at lingonborough dot com>
- Cc: libc-alpha at sourceware dot org
- Date: Mon, 17 Oct 2016 19:01:28 -0700
- Subject: Re: [RFC][PATCH v2 00/11][BZ 10871] Month names in genitive case
- Authentication-results: sourceware.org; auth=none
- References: <1848268023.118102.1476748481399@poczta.nazwa.pl>
Hi Rafal,
I can't comment on much of the mechanical changes, but it would be nice
if the new feature was documented.
On 10/17/2016 04:54 PM, Rafal Luzynski wrote:
> 1. Add the alternative month names support to nl_langinfo(3) (the
> ALTMON_... constants.) There are no major differences between the
> previous [6] version and this one except that it has been rebased.
I think the ALTMON_* constants should be added to the list of constants
in the description of nl_langinfo in manual/locale.texi.
> 5. Implement the %OB specifier. Actually the same as [9], the only
> difference is the Subject line which says it's PATCH 05/11 rather
> than PATCH 3/6.
I think OB should be added to the list of conversion specifiers in the
description of strftime in manual/time.texi.
nl_langinfo is annotated as being part of XOPEN, and strftime is marked
as ISO, so if OB and the associated ALTMON_* constants aren't parts of
those standards, they should have an "@comment GNU" line placed above
the "@item" lines. It should say so in their descriptions as well;
e.g., "This is a GNU extension."
Your introductions to patches 01/11 and 05/11 are good, so modifying
them to sound right in the context of the manual should be fine.
Thanks,
Rical