On Thu, Jul 9, 2015 at 11:21 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
>> - if (is_bare) {
>> - struct strbuf result = STRBUF_INIT;
>> - strbuf_addf(&result, "%.*s.git", (int)(end - start), start);
>> - dir = strbuf_detach(&result, NULL);
>> - } else
>> + if (is_bare)
>> + dir = xstrfmt("%.*s.git", (int)len, start);
>> + else
>> dir = xstrndup(start, end - start);
>
> The last one needs to be adjusted with s/end - start/len/. The
> last-minute rewrite without testing shows; your first two patches
> correctly used "len" ;-)
Doh, you're right, sorry for that.
> No need to resend. Will locally tweak before queuing.
Thanks!
--
Sebastian Schuberth
--
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