From: Glen Choo Date: Mon, 06 Feb 2023 13:04:12 GMT Subject: Re: [PATCH v4 8/9] config API: add "string" version of *_value_multi(), fix segfaults Message-ID: In-Reply-To: Ævar Arnfjörð Bjarmason writes: > 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.