From: Jonathan Nieder Date: Tue, 11 Jul 2017 23:51:16 GMT Subject: Re: [PATCH 1/3] repo_read_index: don't discard the index Message-ID: <20170711235116.GB93855@aiede.mtv.corp.google.com> In-Reply-To: <20170711220408.173269-2-bmwill@google.com> Brandon Williams wrote: > Have 'repo_read_index()' behave more like the other read_index family of > functions and don't discard the index if it has already been populated. > > Signed-off-by: Brandon Williams > --- > repository.c | 2 -- > 1 file changed, 2 deletions(-) How did you discover this? E.g. was it from code inspection or does this make the function more convenient to use for some kinds of callers? Reviewed-by: Jonathan Nieder