Re: [patch] libstdc++/69703 ignore endianness in codecvt_utf8

2016-05-26 Thread Jonathan Wakely
On 26/05/16 14:02 +0200, Christophe Lyon wrote: I've seen you've backported the main patch to the gcc-6 branch, you forgot to add the follow-up "Add dg-require-filesystem-ts directive to test". And likewise for the gcc-5 branch. Both fixed now, sorry about that, again.

Re: [patch] libstdc++/69703 ignore endianness in codecvt_utf8

2016-05-26 Thread Christophe Lyon
On 26 May 2016 at 13:58, Christophe Lyon wrote: > On 5 May 2016 at 12:12, Jonathan Wakely wrote: >> On 04/05/16 17:19 +0100, Andre Vieira (lists) wrote: >>> >>> On 20/04/16 18:40, Jonathan Wakely wrote: On 19/04/16 19:07 +0100, Jonathan

Re: [patch] libstdc++/69703 ignore endianness in codecvt_utf8

2016-05-26 Thread Christophe Lyon
On 5 May 2016 at 12:12, Jonathan Wakely wrote: > On 04/05/16 17:19 +0100, Andre Vieira (lists) wrote: >> >> On 20/04/16 18:40, Jonathan Wakely wrote: >>> >>> On 19/04/16 19:07 +0100, Jonathan Wakely wrote: This was reported as a bug in the Filesystem library, but

Re: [patch] libstdc++/69703 ignore endianness in codecvt_utf8

2016-05-05 Thread Jonathan Wakely
On 04/05/16 17:19 +0100, Andre Vieira (lists) wrote: On 20/04/16 18:40, Jonathan Wakely wrote: On 19/04/16 19:07 +0100, Jonathan Wakely wrote: This was reported as a bug in the Filesystem library, but it's actually a problem in the codecvt_utf8 facet that it uses. The fix had a silly typo

Re: [patch] libstdc++/69703 ignore endianness in codecvt_utf8

2016-05-04 Thread Andre Vieira (lists)
On 20/04/16 18:40, Jonathan Wakely wrote: > On 19/04/16 19:07 +0100, Jonathan Wakely wrote: >> This was reported as a bug in the Filesystem library, but it's >> actually a problem in the codecvt_utf8 facet that it uses. > > The fix had a silly typo meaning it didn't work for big endian > targets,

Re: [patch] libstdc++/69703 ignore endianness in codecvt_utf8

2016-04-20 Thread Jonathan Wakely
On 19/04/16 19:07 +0100, Jonathan Wakely wrote: This was reported as a bug in the Filesystem library, but it's actually a problem in the codecvt_utf8 facet that it uses. The fix had a silly typo meaning it didn't work for big endian targets, which was revealed by the improved tests I added.

[patch] libstdc++/69703 ignore endianness in codecvt_utf8

2016-04-19 Thread Jonathan Wakely
This was reported as a bug in the Filesystem library, but it's actually a problem in the codecvt_utf8 facet that it uses. Tested x86_64-linux, committed to trunk. commit 7f3a547a9e80556030e77ac090e2ad8e04e44abc Author: Jonathan Wakely Date: Tue Apr 19 18:32:17 2016 +0100