Re: [PATCH v4 8/9] config API: add "string" version of *_value_multi(), fix segfaults

From: Glen Choo <chooglen@google.com>
Date: Feb 6, 2023, 13:04 UTC
Message-ID: <kl6lmt5rgd2r.fsf@chooglen-macbookpro.roam.corp.google.com>
In-Reply-To: <patch-v4-8.9-a391ee17617-20230202T131155Z-avarab@gmail.com>

Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:

Show 8 quoted lines

> There are now three remaining files using the low-level API:
>
> - Two cases in "builtin/submodule--helper.c", where it's used safely
>   to see if any config exists.
>
>   We could refactor these away from "multi" to some "does it exist?"
>   function, as [4] did, but as that's orthogonal to the "string"
>   safety we're introducing here let's leave them for now.

Maybe I'm mistaken, but weren't these removed in 3/9? I couldn't find these calls any more.

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