From: Jeff King Date: Wed, 29 May 2024 09:20:32 GMT Subject: Re: [PATCH v2 13/21] config: plug various memory leaks Message-ID: <20240529092032.GC1098944@coredump.intra.peff.net> In-Reply-To: On Mon, May 27, 2024 at 08:46:02AM +0200, Patrick Steinhardt wrote: > 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