]> git.ipfire.org Git - thirdparty/glibc.git/commit
Handle surrogate pairs in c16rtomb (bug 23794, DR#488, C2X).
authorJoseph Myers <joseph@codesourcery.com>
Fri, 19 Oct 2018 16:31:29 +0000 (16:31 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Fri, 19 Oct 2018 16:31:29 +0000 (16:31 +0000)
commitd0a74159792477e5922f53fa2aa6d58eb8265a14
treefcce06511d2a91afa649f629cde15875b6fe6f6a
parentf997b4be18f7e57d757d39e42f7715db26528aa0
Handle surrogate pairs in c16rtomb (bug 23794, DR#488, C2X).

The c16rtomb implementation has:

  // XXX The ISO C 11 spec I have does not say anything about handling
  // XXX surrogates in this interface.

The DR#488 resolution, as applied to C2X, requires surrogate pairs to
be handled here (so the first call returns 0 and stores the high
surrogate in the mbstate_t, while the second call combines the
surrogates, produces a multibyte character and returns the number of
bytes written).  This patch implements that.  (mbrtoc16 already
handled producing surrogates as output.)

Tested for x86_64.

[BZ #23794]
* wcsmbs/c16rtomb.c (c16rtomb): Save first character of surrogate
pair and return 0 in that case, and use saved character to
interpret following character.
* wcsmbs/tst-c16-surrogate.c: New file.
* wcsmbs/Makefile (tests): Add tst-c16-surrogate.c.
[$(run-built-tests) = yes] ($(objpfx)tst-c16-surrogate.out):
Depend on $(gen-locales)
ChangeLog
wcsmbs/Makefile
wcsmbs/c16rtomb.c
wcsmbs/tst-c16-surrogate.c [new file with mode: 0644]