From: Junio C Hamano Date: Wed, 03 Mar 2010 01:10:57 GMT Subject: Re: [PATCH 1/4] convert: Safer handling of $Id$ contraction. Message-ID: <7vljeamcpq.fsf@alter.siamese.dyndns.org> In-Reply-To: <1267460218-1172-1-git-send-email-grubba@grubba.org> Henrik Grubbström writes: > From: Henrik Grubbström (Grubba) You can omit this line, as it is the same as your From: header. > The code to contract $Id:xxxxx$ strings could eat an arbitrary amount > of source text if the terminating $ was lost. It now refuses to > contract $Id:xxxxx$ strings spanning multiple lines. Hmm, at least when going from working tree to the index, shouldn't the code refuse _and_ die(), instead of silently pass the garbage through?