Re: [PATCH 1/4] convert: Safer handling of $Id$ contraction.

Henrik Grubbström [off-list ref] writes:

From: Henrik Grubbström (Grubba) <redacted>

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?