Re: [BUG] Colon in remote urls

From: Jeff King <peff@peff.net>
Date: Dec 10, 2016, 08:26 UTC
Message-ID: <20161210082657.zjp52a2zdtqifmg3@sigill.intra.peff.net>
In-Reply-To: <88bed7c9-4d5d-45d5-5d13-6a8ae834e602@kdbg.org>

On Fri, Dec 09, 2016 at 10:32:48PM +0100, Johannes Sixt wrote:

Show 5 quoted lines

> > +			if (*p == '\\')
> > +				literal = 1;
> 
> There are too many systems out there that use a backslash in path names. I
> don't think it is wise to use it also as the quoting character.

Yeah, I picked it arbitrarily as the common quoting character, but I agree it probably makes backwards compatibility (and general usability when you have to double-backslash each instance) pretty gross on Windows.

Most of the printable characters are going to suffer from backwards compatibility issues. Syntactically, we could use any ASCII control code below 0x20. Certainly our C code would be fine with that, but it seems pretty nasty.

There's probably some crazy non-printing UTF-8 sequence we could use, too, but I'm not sure I want to go there.

-Peff