Re: [PATCH v2 2/2] attr: more matching optimizations from .gitignore

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Junio C Hamano <gitster@xxxxxxxxx> writes:

> Nguyễn Thái Ngọc Duy  <pclouds@xxxxxxxxx> writes:
>
>>    @@ -690,16 +689,18 @@ static int path_matches(const char *pathname, int pathlen,
>>     	 * contain the trailing slash
>>     	 */
>>     
>>    -	if (pathlen < baselen ||
>>    +	if (pathlen < baselen + 1 ||
>>     	    (baselen && pathname[baselen] != '/') ||
>>    -	    strncmp(pathname, base, baselen))
>>    +	    strncmp_icase(pathname, base, baselen))
>
> Shouldn't the last comparison be
>
> 	strncmp_icase(pathname, base, baselen + 1)
>
> instead, if you are trying to match this part from dir.c where
> baselen does count the trailing slash?
>
> 		if (pathlen < x->baselen ||
> 		    (x->baselen && pathname[x->baselen-1] != '/') ||
> 		    strncmp_icase(pathname, x->base, x->baselen))
> 			continue;
>
> In other words, relative to what was queued to 'pu', something like
> this instead....

Aaaaand,... it doesn't work and breaks t0003.sh.  Sigh...
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]