On 6/7/2021 10:43 AM, Ævar Arnfjörð Bjarmason wrote: > Fix a potential incorrect display of the number of items (off by one) > and stalling of the progress bar in refresh_index(). > > The off-by-one error is minor, we should say we're processing the 1st > item, not the 0th. This along with the next change also allows us to > remove the last display_progress() call outside the loop, as we'll > always have reached 100% now. This "pre-announce the progress" seems correct and is unlikely to have a user sitting at "100%" while the loop is actually doing work on that last cache entry. Thanks, -Stolee