Re: [PATCH v3 01/12] builtin/gc: use designated field initializers for maintenance tasks

From: Karthik Nayak <karthik.188@gmail.com>
Date: Jun 2, 2025, 10:35 UTC
Message-ID: <CAOLa=ZSy5XoKJ2OaKv3DukY9ko-pkpa2zHLQneLwq=9+wJnYWg@mail.gmail.com>
In-Reply-To: <20250602-b4-pks-maintenance-ref-lock-race-v3-1-587d44252dcb@pks.im>

Patrick Steinhardt <ps@pks.im> writes:

> Convert the array of maintenance tasks to use designated field
> initializers. This makes it easier to add more fields to the struct
> without having to modify all tasks.
>

I was wondering why we don't catch missing field initializers when not using designators. Seems like we explicitly set '-Wno-missing-field-initializers'. Perhaps something to slowly tackle over time.

The patch looks good.

[snip]

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