From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 04 Feb 2008 00:07:48 GMT
Subject: Re: [PATCH] Add platform-independent .git "symlink"
Message-ID: <7vwsplporv.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <8c5c35580802021047o41a7de9fi84a6b33f0f779e2e@mail.gmail.com>

"Lars Hjemli" <hjemli@gmail.com> writes:

> Maybe this as a compromise?
>
> 	static char buf[PATH_MAX + 9];  /* "GITDIR: " + "\n"  */
> ...
> 	if (!len || buf[len - 1] != '\n')
> 		return NULL;
> 	buf[len - 1] = '\0';
> 	if (prefixcmp(buf, "GITDIR: "))
> 		return NULL;

Perhaps that is easier to read.

For now I haven't picked up any of the "how about this way"
revisions.