Re: [PATCH v2 13/21] config: plug various memory leaks

From: Jeff King <peff@peff.net>
Date: May 29, 2024, 09:20 UTC
Message-ID: <20240529092032.GC1098944@coredump.intra.peff.net>
In-Reply-To: <ZlQsKp2IHt2wankn@tanuki>

On Mon, May 27, 2024 at 08:46:02AM +0200, Patrick Steinhardt wrote:

Show 8 quoted lines

> Indeed, I had the exact same observation. I've already got a patch
> series that enables `-Wwrite-strings` and that adapts our codebase to
> compile cleanly with it. I'll send that series once this one here has
> landed.
> 
> So my proposal would be to leave this patch as-is for the time being,
> but revisit it once both patch series have landed. Does that work for
> you?

Yeah, I can certainly live with that. One of my biggest concerns was that your patch would invalidate the investigation and work I had already done, and I'd fail to follow up. But if you are putting it onto your todo list, then that is even less work for me. ;)

-Peff