From: Linus Torvalds <torvalds@osdl.org>
Date: Sun, 19 Nov 2006 21:14:34 GMT
Subject: Re: [WISH] Store also tag dereferences in packed-refs
Message-ID: <Pine.LNX.4.64.0611191310430.3692@woody.osdl.org>
In-Reply-To: <7vbqn38831.fsf@assigned-by-dhcp.cox.net>



On Sun, 19 Nov 2006, Junio C Hamano wrote:
> 
> Did you do this "if (isspace(*line)) return NULL" to have
> extensibility later, I wonder...

No, but I try to have a policy of not accepting anything that is even 
slightly questionable, so it was unintentional in the details but a result 
of a higher-level principle..

Of course, even when I wrote it I _also_ knew that I should check 
everything else too (not just the first character), but I was lazy. That 
would have been even better - then we could have made the thing be 
something like

	<sha1><space><name>[<space><sha1-of-deref>]*

instead.