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

On 07/11, Jonathan Nieder wrote:

Brandon Williams wrote:

quoted

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 <redacted>
---
 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?

When working on another series I realized that some code paths may end
up calling read_index() a bunch and I want to prevent discarding and
then re-reading the same index over and over again if/when those calls
are migrated to using repo_read_index.

Reviewed-by: Jonathan Nieder <redacted>

-- 
Brandon Williams