From: Mark C. Chu-Carroll Date: Fri, 02 Oct 2026 19:02:30 GMT Subject: Re: [PATCH 1/2] packfile: move around `close_pack()` Message-ID: In-Reply-To: <20261002-pks-packfile-stale-delta-base-cache-v1-1-7592a3e31ae0@pks.im> On Fri Oct 2, 2026 at 3:34 AM EDT, Patrick Steinhardt wrote: > In the next commit we'll want to access the delta base cache in > `close_pack()`. Move the function after the declaration of the cache to > prepare for this. Maybe I'm just being clueless, but how does moving an unmodified function help with the subsequent change? -- Mark Craig Chu-Carroll (@MarkChuCarroll at gitlab) *** Software Tools/Math Geek - Software Engineer at Gitlab *** Work Email: mcarroll@gitlab.com / markchucarroll@fastmail.com *** Personal Blog: http://goodmath.org/blog / Personal email: markcc@gmail.com