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: [PATCH 14/14] Fix UTF-16 surrogate handling.

From: Stefan Liebler <stli at linux dot vnet dot ibm dot com>
To: libc-alpha at sourceware dot org
Cc: "Joseph S. Myers" <joseph at codesourcery dot com>
Date: Fri, 18 Mar 2016 14:04:45 +0100
Subject: Re: [PATCH 14/14] Fix UTF-16 surrogate handling.
Authentication-results: sourceware.org; auth=none
References: <1456219278-5258-1-git-send-email-stli at linux dot vnet dot ibm dot com> <1456219278-5258-15-git-send-email-stli at linux dot vnet dot ibm dot com> <alpine dot DEB dot 2 dot 10 dot 1602231742030 dot 17168 at digraph dot polyomino dot org dot uk> <namfnd$mrp$2 at ger dot gmane dot org>

On 02/25/2016 09:58 AM, Stefan Liebler wrote:

On 02/23/2016 06:42 PM, Joseph Myers wrote:

If this is user-visible in a release, there should be a bug filed in
Bugzilla (if there isn't one already open), and a testcase added to the
testsuite.

okay.

I've filed the bug
"Bug 19727 - Converting from/to UTF-xx with iconv() does not always
report errors on UTF-16 surrogates values."
(https://sourceware.org/bugzilla/show_bug.cgi?id=19727)

This patch also adds a new testcase, which checks UTF conversions with
input values in range of UTF16 surrogates. The test converts from
UTF-xx to INTERNAL, INTERNAL to UTF-xx and directly between
UTF-xx to UTF-yy. The latter conversion is needed because s390 has
iconv-modules, which converts from/to UTF in one step.
The new testcase was tested on a s390, power and intel machine.

Is the previously attached test-case okay?

Follow-Ups:
- Re: [PATCH 14/14] Fix UTF-16 surrogate handling.
  - From: Stefan Liebler

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