Re: [PATCH 04/12] pack-objects: widen `free_unpacked()` return to `size_t`
Patrick Steinhardt <[email protected]> Wed, 5 Aug 2026 11:23:07 +0200
| Newsgroups | org.kernel.vger.git |
|---|---|
| Message-ID | <[email protected]> |
On Thu, Jul 09, 2026 at 04:49:31PM +0000, Johannes Schindelin via GitGitGadget wrote: > From: Johannes Schindelin <[email protected]> > diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c > index f89628a760..4737a6a32c 100644 > --- a/builtin/pack-objects.c > +++ b/builtin/pack-objects.c > @@ -2972,9 +2972,9 @@ static unsigned int check_delta_limit(struct object_entry *me, unsigned int n) > return m; > } > > -static unsigned long free_unpacked(struct unpacked *n) > +static size_t free_unpacked(struct unpacked *n) > { > - unsigned long freed_mem = sizeof_delta_index(n->index); > + size_t freed_mem = sizeof_delta_index(n->index); Okay. As mentioned on a preceding patch, the function itself still returns `unsigned long`, which should probably also be corrected in this patch series. Patrick