Message ID | d6ac90a2-7bb3-3f21-abd7-8695a1cec835@honermann.net (mailing list archive) |
---|---|
Headers |
Return-Path: <libc-alpha-bounces+patchwork=sourceware.org@sourceware.org> X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 38AD6385843F for <patchwork@sourceware.org>; Sun, 27 Feb 2022 16:53:38 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 38AD6385843F DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1645980818; bh=1t3nsFgCqznYyA1lh2GhGWSD1NOo0idMRqpUd8QzCYE=; h=Date:Subject:To:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:From; b=d66/E5/INDnqPBJMSUGDzyu0GqIaye2lzHK2i4d4F2yP3QUhBcvDz9jF1Ffql7g9n 56M/TG4OcTQcpCE4N2aCobj8NRXjF5mBuYq0JuSYaaTFPn3DZs9iSBMpDB8N9krCIf F9MEWBMpugYwnZltZP2HAAD3e19awiY4SDTTtaUg= X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from smtp116.ord1c.emailsrvr.com (smtp116.ord1c.emailsrvr.com [108.166.43.116]) by sourceware.org (Postfix) with ESMTPS id 60B763858D35 for <libc-alpha@sourceware.org>; Sun, 27 Feb 2022 16:53:15 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 60B763858D35 X-Auth-ID: tom@honermann.net Received: by smtp15.relay.ord1c.emailsrvr.com (Authenticated sender: tom-AT-honermann.net) with ESMTPSA id 47D912009E for <libc-alpha@sourceware.org>; Sun, 27 Feb 2022 11:53:13 -0500 (EST) Message-ID: <d6ac90a2-7bb3-3f21-abd7-8695a1cec835@honermann.net> Date: Sun, 27 Feb 2022 11:53:12 -0500 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 Content-Language: en-US Subject: [PATCH 0/3]: C++20 P0482R6 and C2X N2653: support for char8_t, mbrtoc8(), and c8rtomb(). To: libc-alpha <libc-alpha@sourceware.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Classification-ID: 50ad1b57-328c-4c8b-87bc-9a9e92733657-1-1 X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list <libc-alpha.sourceware.org> List-Unsubscribe: <https://sourceware.org/mailman/options/libc-alpha>, <mailto:libc-alpha-request@sourceware.org?subject=unsubscribe> List-Archive: <https://sourceware.org/pipermail/libc-alpha/> List-Post: <mailto:libc-alpha@sourceware.org> List-Help: <mailto:libc-alpha-request@sourceware.org?subject=help> List-Subscribe: <https://sourceware.org/mailman/listinfo/libc-alpha>, <mailto:libc-alpha-request@sourceware.org?subject=subscribe> From: Tom Honermann via Libc-alpha <libc-alpha@sourceware.org> Reply-To: Tom Honermann <tom@honermann.net> Errors-To: libc-alpha-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libc-alpha" <libc-alpha-bounces+patchwork=sourceware.org@sourceware.org> |
Series |
: C++20 P0482R6 and C2X N2653: support for char8_t, mbrtoc8(), and c8rtomb().
|
|
Message
Tom Honermann
Feb. 27, 2022, 4:53 p.m. UTC
This series of patches provides the following: - A fix for bug 25744 [1]. - Implementations of the mbrtoc8 and c8rtomb functions adopted for C++20 via WG21 P0482R6 [2] and for C2X via WG14 N2653 [3]. - A char8_t typedef as adopted for C2X via WG14 N2653 [3]. These patches addresses feedback provided in response to a previous submission [4]. Patch 1: A fix and test for bug 25744 [1]. Patch 2: Definitions of the mbrtoc8 and c8rtomb functions and the char8_t typedef. Patch 3: Tests for the mbrtoc8 and c8rtomb functions and the char8_t typedef. The fix for bug 25744 [1] is included in this patch series because the tests for mbrtoc8 and c8rtomb depend on it for exercising the special case where a pair of Unicode code points is converted to/from a single double byte sequence. Such conversion cases exist for Big5-HKSCS. N2653 was adopted by WG14 for C2X during their recent meeting. This patch series enables the new declarations in C2X mode and when _GNU_SOURCE is defined. Thank you to Joseph Myers and Carlos O'Donell for their prior reviews of this patch series. Tom. [1]: Bug 25744 "mbrtowc with Big5-HKSCS returns 2 instead of 1 when consuming the second byte of certain double byte characters" https://sourceware.org/bugzilla/show_bug.cgi?id=25744 [2]: WG21 P0482R6 "char8_t: A type for UTF-8 characters and strings (Revision 6)" https://wg21.link/p0482r6 [3]: WG14 N2653 "char8_t: A type for UTF-8 characters and strings (Revision 1)" http://www.open-std.org/jtc1/sc22/wg14/www/docs/n2653.htm [4]: "[PATCH 2/3]: C++20 P0482R6 and C2X N2653: Implement mbrtoc8, c8rtomb, char8_t" https://sourceware.org/pipermail/libc-alpha/2022-February/136558.html
Comments
Is there anything I can do to help land this patch series? I believe all review comments have been addressed. Tom. On 2/27/22 11:53 AM, Tom Honermann via Libc-alpha wrote: > This series of patches provides the following: > - A fix for bug 25744 [1]. > - Implementations of the mbrtoc8 and c8rtomb functions adopted for > C++20 via WG21 P0482R6 [2] and for C2X via WG14 N2653 [3]. > - A char8_t typedef as adopted for C2X via WG14 N2653 [3]. > > These patches addresses feedback provided in response to a previous > submission [4]. > > Patch 1: A fix and test for bug 25744 [1]. > Patch 2: Definitions of the mbrtoc8 and c8rtomb functions and the > char8_t typedef. > Patch 3: Tests for the mbrtoc8 and c8rtomb functions and the char8_t > typedef. > > The fix for bug 25744 [1] is included in this patch series because the > tests for mbrtoc8 and c8rtomb depend on it for exercising the special > case where a pair of Unicode code points is converted to/from a single > double byte sequence. Such conversion cases exist for Big5-HKSCS. > > N2653 was adopted by WG14 for C2X during their recent meeting. This > patch series enables the new declarations in C2X mode and when > _GNU_SOURCE is defined. > > Thank you to Joseph Myers and Carlos O'Donell for their prior reviews > of this patch series. > > Tom. > > [1]: Bug 25744 > "mbrtowc with Big5-HKSCS returns 2 instead of 1 when consuming the > second byte of certain double byte characters" > https://sourceware.org/bugzilla/show_bug.cgi?id=25744 > > [2]: WG21 P0482R6 > "char8_t: A type for UTF-8 characters and strings (Revision 6)" > https://wg21.link/p0482r6 > > [3]: WG14 N2653 > "char8_t: A type for UTF-8 characters and strings (Revision 1)" > http://www.open-std.org/jtc1/sc22/wg14/www/docs/n2653.htm > > [4]: "[PATCH 2/3]: C++20 P0482R6 and C2X N2653: Implement mbrtoc8, > c8rtomb, char8_t" > https://sourceware.org/pipermail/libc-alpha/2022-February/136558.html
Ping. Could someone please assist me in landing these changes? Tom. On 3/23/22 11:32 AM, Tom Honermann via Libc-alpha wrote: > Is there anything I can do to help land this patch series? I believe > all review comments have been addressed. > > Tom. > > On 2/27/22 11:53 AM, Tom Honermann via Libc-alpha wrote: >> This series of patches provides the following: >> - A fix for bug 25744 [1]. >> - Implementations of the mbrtoc8 and c8rtomb functions adopted for >> C++20 via WG21 P0482R6 [2] and for C2X via WG14 N2653 [3]. >> - A char8_t typedef as adopted for C2X via WG14 N2653 [3]. >> >> These patches addresses feedback provided in response to a previous >> submission [4]. >> >> Patch 1: A fix and test for bug 25744 [1]. >> Patch 2: Definitions of the mbrtoc8 and c8rtomb functions and the >> char8_t typedef. >> Patch 3: Tests for the mbrtoc8 and c8rtomb functions and the char8_t >> typedef. >> >> The fix for bug 25744 [1] is included in this patch series because >> the tests for mbrtoc8 and c8rtomb depend on it for exercising the >> special case where a pair of Unicode code points is converted to/from >> a single double byte sequence. Such conversion cases exist for >> Big5-HKSCS. >> >> N2653 was adopted by WG14 for C2X during their recent meeting. This >> patch series enables the new declarations in C2X mode and when >> _GNU_SOURCE is defined. >> >> Thank you to Joseph Myers and Carlos O'Donell for their prior reviews >> of this patch series. >> >> Tom. >> >> [1]: Bug 25744 >> "mbrtowc with Big5-HKSCS returns 2 instead of 1 when consuming the >> second byte of certain double byte characters" >> https://sourceware.org/bugzilla/show_bug.cgi?id=25744 >> >> [2]: WG21 P0482R6 >> "char8_t: A type for UTF-8 characters and strings (Revision 6)" >> https://wg21.link/p0482r6 >> >> [3]: WG14 N2653 >> "char8_t: A type for UTF-8 characters and strings (Revision 1)" >> http://www.open-std.org/jtc1/sc22/wg14/www/docs/n2653.htm >> >> [4]: "[PATCH 2/3]: C++20 P0482R6 and C2X N2653: Implement mbrtoc8, >> c8rtomb, char8_t" >> https://sourceware.org/pipermail/libc-alpha/2022-February/136558.html