Re: [PATCH] localedata: GBK: add mapping for 0x80->Euro sign [BZ #20864]

[Florian Weimer <fweimer at redhat dot com>]

On 11/29/2016 06:04 PM, Mike Frysinger wrote:
> On Tue, Nov 29, 2016 at 5:08 AM, Florian Weimer <fweimer@redhat.com> wrote:
> > On 11/28/2016 10:23 AM, Andreas Schwab wrote:
> > > FAIL: iconvdata/tst-tables
> > > failed: ./tst-table.sh
> > > /home/abuild/rpmbuild/BUILD/glibc-2.24.90.20161127.gb964e06/cc-base/
> > > /home/abuild/rpmbuild/BUILD/glibc-2.24.90.20161127.gb964e06/cc-base/iconvdata/
> > > GBK
> >
> > Here's one way to fix it.  Doesn't look really pretty, but it seems that
> > this new 0x80 mapping is quite an outlier.
>
> i'm not super familiar with the gconv modules, but this seems to make sense
>
> should use lowercase hex constants though to match surrounding code style

Right, I'll fix this before committing it.

Thanks,
Florian