Re: [PATCH 17/24] revisions API: have release_revisions() release "cmdline"

From: Derrick Stolee <derrickstolee@github.com>
Date: Mar 9, 2022, 19:19 UTC
Message-ID: <4e564691-0ef6-a867-647e-c26064b0bb36@github.com>
In-Reply-To: <patch-17.24-b5687ed1355-20220309T123321Z-avarab@gmail.com>

On 3/9/2022 8:16 AM, Ævar Arnfjörð Bjarmason wrote:

> Extend the the release_revisions() function so that it frees the
> "cmdline" in the "struct ref_info". This in combination with a
> preceding change to free "commits" and "mailmap" means that we can
> whitelist another test under "TEST_PASSES_SANITIZE_LEAK=true".

Could you consider changing your vocabulary to use "allowlist" (with the opposite term being "denylist")?

Thanks, -Stolee

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