Re: [PATCH 1/2] Introduce the function strip_path_suffix()
Johannes Schindelin schrieb:
Show 12 quoted lines
> On Wed, 18 Feb 2009, Johannes Sixt wrote:
>> Johannes Schindelin schrieb:
>>> strip_path_suffix("C:\\msysgit/\\libexec\\git-core",
>>> "libexec///git-core", &prefix)
>>>
>>> will set prefix to "C:\\msysgit" and return 0.
>> But you implemented it so that prefix is actually "C:\\msysgit/\\"
>> (unless, of course, I'm reading the code wrong).
>
> This is supposed to handle that case:
>
> *prefix = xstrndup(path, chomp_trailing_dir_sep(path, path_len));Ah, I missed that final call of chomp_trailing_dir_sep.
-- Hannes