This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFC][PATCH v8 00/16][BZ 10871] Month names in alternative (nominative/genitive) case


3.07.2017 23:35 Zack Weinberg <zackw@panix.com> wrote:
>
>
> On 06/28/2017 05:56 AM, Rafal Luzynski wrote:
> > This is almost complete repost of my patches. The previous series was
> > posted on May 29. [1] There are not many changes in this set, mostly it
> > is rebase against the current master and applying some recent changes
> > like using locale_t instead of __locale_t [2].
>
> Having now read through the entire patch series, I would like to suggest
> that you squash patches 1, 4, 5, and 8 together into one patch that does
> the complete change for %OB, just as patch 9 does the complete change
> for %Ob. They are _logically_ all the same change and should be
> reviewed as a unit.
>
> zw

Thank you again for your review, Zack.  I always meant that these patches
should be squashed before the final commit.  I have split them only for
the easier review and to give the reviewers/committers choice which parts
will be accepted and which will be rejected.  For example now we have
decided to reject the backward compatibility.

Regards,

Rafal


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]