From: Jeff King Date: Fri, 21 Feb 2025 09:06:25 GMT Subject: Re: [PATCH v2 02/16] object-name: allow skipping ambiguity checks in `get_oid()` family Message-ID: <20250221090625.GA2518234@coredump.intra.peff.net> In-Reply-To: On Fri, Feb 21, 2025 at 09:36:04AM +0100, Patrick Steinhardt wrote: > Spoiler alert: I do have a patch series locally that gets rid of the > global variable completely, and that series builds on top of the new > flag I'm introducing here. So I'd prefer to keep it so that we can > eventually have less callsites that rely on global state. OK. I do agree that getting rid of the global would be nice. I just wasn't sure how ugly it would be to pass the flags through the stack into handle_revision(). -Peff