Signed-off-by: Ramsay Jones <>

Hi Thomas,

I note that index_changed(), which has no side effects, is called with
unchanged parameters twice each time around the loop. I was about to
suggest saving the result of a single call and using it in both places ...

However, on first sight, it looks like the expression in the while
condition will always be true (otherwise you would not reach that point
in the code - you would already have returned), so we can simply remove
that part of the expression (however, I didn't look too closely, so ...)

Ramsay Jones

 read-cache.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/read-cache.c b/read-cache.c
index 36f0877..5176d7a 100644
--- a/read-cache.c
+++ b/read-cache.c
@@ -1278,7 +1278,7 @@ int read_index_from(struct index_state *istate, const 
char *path)
-       } while ((err || index_changed(&st_old, &st_new)) && i < 50);
+       } while (i < 50);
        munmap(mmap, mmap_size);
        die("index file corrupt");

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to
More majordomo info at

Reply via email to