On 06/21, Jonathan Tan wrote:
> On Tue, 20 Jun 2017 12:19:46 -0700
> Brandon Williams <bmw...@google.com> wrote:
>
> > +int repo_read_index(struct repository *repo)
> > +{
> > + if (!repo->index)
> > + repo->index = xcalloc(1, sizeof(struct index_state));
>
> sizeof(*repo->index)?
Is there a reason to prefer one versus the other? Either way I can
change.
>
> [snip]
>
> > + /* Repository's in-memory index */
> > + struct index_state *index;
> > +
>
> Might be worth commenting that repo_read_index() can populate this.
--
Brandon Williams
