On Sat, May 02, 2020 at 01:47:09PM -0700, Junio C Hamano wrote: > > What I am curious is if anything breaks around this test if you lost > the extra "!strchr() &&" check. I suspect that this test will pass. correct and with the strchr I am introducing a regression (compared against 2.26.0) in cases where the '\r' gets appended to the path and therefore gets ignored for matching (unless useHttpPath is true, which is not the default) will add 2 more test cases to cover those, and guess we are going to 11 :( Carlo PS. thanks for all your patience and reviewing