[PATCH v2 1/1] Add new codepage charmaps/IBM858

Rafal Luzynski digitalfreak@lingonborough.com
Thu Sep 14 09:16:00 GMT 2017


9.09.2017 11:43 Mike FABIAN <mfabian@redhat.com> wrote:
> [...]
> [BZ #21084]
> * benchtests/strcoll-inputs/filelist#en_US.UTF-8: Add IBM858 and ibm858.c.
> * iconvdata/Makefile: Add IBM858.
> * iconvdata/gconv-modules: Add IBM858.

I think there should be "Likewise." if there is the same comment in
multiple lines.

> * iconvdata/ibm858.c: New file.
> * iconvdata/tst-tables.sh: Add IBM858
> * localedata/charmaps/IBM858: New file.

Can we reorder the lines and use "Likewise." once again?

Just these nitpicks, otherwise no objections.

Regards,

Rafal



More information about the Libc-alpha mailing list