Re: [PATCH v2 0/4] t-ctype: simplify unit test definitions

From: Christian Couder <christian.couder@gmail.com>
Date: Mar 4, 2024, 09:25 UTC
Message-ID: <CAP8UFD395xDjgqxg=S7dZrbnsCNsMEqQbKBipMU4m45CfT8f0w@mail.gmail.com>
In-Reply-To: <20240303101330.20187-1-l.s.r@web.de>

On Sun, Mar 3, 2024 at 11:13 AM René Scharfe <l.s.r@web.de> wrote:

Show 10 quoted lines

>
> Simplify the ctype unit tests to allow combining specification strings
> in any order and no longer require repeating class names.
>
> Changes since v1:
> * Added checks to guard string length calculation using sizeof.
> * Kept the definition string in the error output.
> * Added patches 2 and 3 with further cosmetic changes.
> * Dropped the last patch with the huge output for now, as it needs
>   further thought.

This V2 looks good to me. Acked.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.