Re: [PATCH 1/3] repo_read_index: don't discard the index

2017-07-12 Thread Brandon Williams
On 07/11, Jonathan Nieder wrote: > 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 fil

Re: [PATCH 1/3] repo_read_index: don't discard the index

2017-07-12 Thread Brandon Williams
On 07/11, Stefan Beller wrote: > On Tue, Jul 11, 2017 at 3:04 PM, 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. > > instead rely on the quick return of read_index_fro

Re: [PATCH 1/3] repo_read_index: don't discard the index

2017-07-11 Thread Stefan Beller
On Tue, Jul 11, 2017 at 3:04 PM, 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. instead rely on the quick return of read_index_from which has /* istate->initialized c

Re: [PATCH 1/3] repo_read_index: don't discard the index

2017-07-11 Thread Jonathan Nieder
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 thi

[PATCH 1/3] repo_read_index: don't discard the index

2017-07-11 Thread Brandon Williams
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(-) diff --git a/repository.c b/repository.c index edca90740..8e60