Re: [PATCH v4 12/24] read-cache: read index-v5
- From
Thomas Gummerer <t.gummerer@gmail.com>
- Date
- Nov 30, 2013, 20:10 UTC
- Message-ID
- <87ob51y7l0.fsf@gmail.com>
- In-Reply-To
- <CALWbr2ybgpdDJLRKA4zwPmRE4LQv4VWLJM6Jv0dO-GyHREGW7g@mail.gmail.com>
Antoine Pelisse <apelisse@gmail.com> writes:
Show 18 quoted lines
> On Wed, Nov 27, 2013 at 1:00 PM, Thomas Gummerer <t.gummerer@gmail.com> wrote: >> Make git read the index file version 5 without complaining. >> >> This version of the reader reads neither the cache-tree >> nor the resolve undo data, however, it won't choke on an >> index that includes such data. >> >> Helped-by: Junio C Hamano <gitster@pobox.com> >> Helped-by: Nguyen Thai Ngoc Duy <pclouds@gmail.com> >> Helped-by: Thomas Rast <trast@student.ethz.ch> >> Signed-off-by: Thomas Gummerer <t.gummerer@gmail.com> >> --- >> [...] >> +static struct directory_entry *read_directories(unsigned int *dir_offset, >> + unsigned int *dir_table_offset, >> + void *mmap, int mmap_size) > > Minor nit: why is this mmap_size "int" while all others are "unsigned long" ?
Thanks for catching that. It should be "unsigned long" here to. Will fix in the re-roll.
Show 13 quoted lines
>> [...] >> +static int read_entry(struct cache_entry **ce, char *pathname, size_t pathlen, >> + void *mmap, unsigned long mmap_size, >> + unsigned int first_entry_offset, >> + unsigned int foffsetblock) >> [...] >> +static int read_entries(struct index_state *istate, struct directory_entry *de, >> + unsigned int first_entry_offset, void *mmap, >> + unsigned long mmap_size, unsigned int *nr, >> + unsigned int foffsetblock) >> [...] >> +static int read_index_v5(struct index_state *istate, void *mmap, >> + unsigned long mmap_size, struct filter_opts *opts)