On Mon, Mar 1, 2021 at 11:26 AM Junio C Hamano <gitster@xxxxxxxxx> wrote: > Before the problematic patch, this used to be > > } else if (cache_name_pos(src, length) < 0) > > I wonder if we should revert the change to use cache_file_exists() > in the first place (and rewrite the subsequent use of ce to match), > though. For what it's worth, that was what I did originally; the change to look up the ce "up front" was because someone objected to the double search implied by calling cache_name_pos once, then cache_file_exists to determine the correct error message... (I've never been 100% on how the ignore-case stuff works internally.) Chris
