Subject: Re: [PATCH v2] safe_create_leading_directories(): on Windows, \ can separate path components

From: Junio C Hamano <[email-masked]>
Date: Tue, 21 Jan 2014 11:35:27 -0800

Sebastian Schuberth <[email-masked]> writes:

> On Sun, Jan 19, 2014 at 12:40 AM, Michael Haggerty <[email-masked]> wrote:
>
>> This patch applies on top of v3 of mh/safe-create-leading-directories.
>>
>> The only logical change from Sebastian's patch is that this version
>> restores the original slash character rather than always restoring it
>> to '/' (as suggested by Junio).
>
> Thanks Michael. This is very similar to Junio's current merge conflict
> resultion in pu between your original series and my patch (c2fec83). I
> like this patch of yours slightly better, however, as it uses a
> "while" instead of a "for" loop and the more conclusive
> "slash_character" than the "was_slash" variable name.

Yeah, I agree this patch that uses more descriptive names is a much
more desirable end-result than the tentative merge in 'pu'.