This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: Fix float128 uses of xlocale.h [committed]
- From: Zack Weinberg <zackw at panix dot com>
- To: "Gabriel F. T. Gomes" <gftg at linux dot vnet dot ibm dot com>
- Cc: Joseph Myers <joseph at codesourcery dot com>, GNU C Library <libc-alpha at sourceware dot org>
- Date: Fri, 23 Jun 2017 10:31:39 -0400
- Subject: Re: Fix float128 uses of xlocale.h [committed]
- Authentication-results: sourceware.org; auth=none
- References: <alpine.DEB.2.20.1706222235070.18518@digraph.polyomino.org.uk> <20170623100059.120b3e56@keller>
On Fri, Jun 23, 2017 at 9:00 AM, Gabriel F. T. Gomes
<gftg@linux.vnet.ibm.com> wrote:
> On Thu, 22 Jun 2017 22:35:25 +0000
> Joseph Myers <joseph@codesourcery.com> wrote:
>
>> Three float128 files still include xlocale.h after it was removed. I
>> don't know why this didn't cause problems for powerpc64le float128
>> testing; it did cause problems for my x86_64 float128 testing. This
>> patch changes the includes to use bits/types/locale_t.h.
>
> It caused problems for powerpc64le testing, indeed. But, my rebasing was
> a little outdated. Thanks for doing this.
It's not a big deal, but .c files should probably be using <locale.h>
instead (on the usual "don't directly include bits headers without a
compelling reason" principles).
zw