[RFC][PATCH v2 00/11][BZ 10871] Month names in genitive case

Rical Jasan ricaljasan@pacific.net
Tue Oct 18 02:01:00 GMT 2016


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



More information about the Libc-alpha mailing list