From: Alex Riesen Date: Mon, 30 Jun 2008 17:41:07 GMT Subject: Re: [PATCH 13/13] Build in merge Message-ID: <81b0412b0806301041w70431da0u775d56920bf4a521@mail.gmail.com> In-Reply-To: <7vd4lz4gtw.fsf@gitster.siamese.dyndns.org> 2008/6/30 Junio C Hamano : > Miklos Vajna writes: >> + /* See if remote matches ~, or ^ */ >> + ptr = strrchr(remote, '^'); >> + if (ptr && ptr[1] == '\0') { >> + len = strlen(remote); >> + while ((ptr = (char *)memrchr(remote, '^', len))) >> + if (ptr && ptr[1] == '\0') >> + len = ptr - remote - 1; >> + else >> + break; > > That's a funny way to say: > > for (len = 0, ptr = remote + strlen(remote); > remote < ptr && ptr[-1] == '^'; > ptr--) > len++; > Besides, Cygwin has no memrchr