Re: [PATCH 02/20] path: expose `do_git_common_path()` as `strbuf_git_common_pathv()`
Patrick Steinhardt <ps@pks.im> writes:
Show 10 quoted lines
>> Yes, if the function is about obtaining the path for a file in a
>> given repository's metadata directory, and its association with
>> "strbuf" is that it merely happens to use it instead of "char *",
>> it should not be named as if "strbuf_" ness is the primary
>> characteristics of the function.
>>
>> strbuf_cleanup_path() should also be renamed for the same reason.
>
> Agreed. I was doing it for consistency's sake in this case, but let's
> rather not make the overall interface any weirder than it already is.
Thanks, then we are in agreement.
Junio C Hamano <gitster@pobox.com>