Re: [PATCH v4 02/16] ref-filter.h: provide `REF_FILTER_INIT`

From: Taylor Blau <me@ttaylorr.com>
Date: Jul 3, 2023, 17:07 UTC
Message-ID: <ZKMAaI3I7NDDatXk@nand.local>
In-Reply-To: <20230703051504.GB3502534@coredump.intra.peff.net>

On Mon, Jul 03, 2023 at 01:15:04AM -0400, Jeff King wrote:

Show 6 quoted lines

> > In the meantime, fix a case in test-reach.c where its `ref_filter` is
> > not even zero-initialized.
>
> This test-reach case scared me, but it happens to work now because
> commit_contains() only looks at the one field that we set. So we're not
> fixing a bug, but more like a bug waiting to happen. :)

Good point, I updated the commit message to more clearly signal that this isn't fixing a bug, but rather preventing one.

Thanks, Taylor

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