Re: [PATCH v2 6/6] t-strbuf: use for_test

From: Phillip Wood <phillip.wood123@gmail.com>
Date: Jul 23, 2024, 13:23 UTC
Message-ID: <d51c9764-e11e-452d-8974-7de80ac34195@gmail.com>
In-Reply-To: <b179d004-d00b-4ed7-8cc9-f3b388f101f4@web.de>

Hi René

On 21/07/2024 07:26, René Scharfe wrote:

I'm still ambivalent about patches 3-6 but if we do this can we visually separate the initialization and cleanup from the main test body please. For example

> +	for_test ("strbuf_addch adds char") {
> +		struct strbuf sb = STRBUF_INIT;

> +		t_addch(&sb, 'a');

> +		t_release(&sb);
> +	}

Best Wishes

Phillip

$ 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.