Re: [PATCH v2 3/4] bundle: keep a copy of bundle file name in the in-core bundle header

2016-03-02 Thread Jeff King
On Wed, Mar 02, 2016 at 12:32:40PM -0800, Junio C Hamano wrote: > + free((void *)header->filename); This cast is necessary, because... > diff --git a/bundle.h b/bundle.h > index f7ce23b..e059ccf 100644 > --- a/bundle.h > +++ b/bundle.h > @@ -10,12 +10,14 @@ struct ref_list { > }; > > str

[PATCH v2 3/4] bundle: keep a copy of bundle file name in the in-core bundle header

2016-03-02 Thread Junio C Hamano
This will be necessary when we start reading from a split bundle where the header and the thin-pack data live in different files. The in-core bundle header will read from a file that has the header, and will record the path to that file. We would find the name of the file that hosts the thin-pack