René Scharfe [off-list ref] writes:
Am 12.06.2012 23:00, schrieb Jonathan Nieder:
quoted
Jonathan Nieder wrote:
quoted
René Scharfe wrote:
quoted
quoted
How about this instead?
Looks good to me.
Oh, hold on a second. Won't this get confused by
ssh://[::1]/foo/bar/baz:80/qux
?
It shouldn't, because the host part is NUL-terminated before
get_port() is called. Let's see (with the patch):
$ git clone ssh://[::1]/foo/bar/baz:80/qux
Cloning into 'qux'...
ssh: connect to host ::1 port 22: Connection refused
fatal: The remote end hung up unexpectedly
René
Yeah, I was wondering how that would get confused myself. Jonathan,
ack again?