Here's a v2 of my series. The behavior should be identical, but I've incorporated some comment and small code tweaks based on feedback from the first round. I also added a fourth patch which adds a new comment explaining some of the cases that were alluded to in the earlier round's patch 3. [1/4]: t4212: avoid putting git on left-hand side of pipe [2/4]: parse_commit(): parse timestamp from end of line [3/4]: parse_commit(): handle broken whitespace-only timestamp [4/4]: parse_commit(): describe more date-parsing failure modes commit.c | 47 +++++++++++++++++++++++++++++++++++------- t/t4212-log-corrupt.sh | 39 +++++++++++++++++++++++++++++++++-- 2 files changed, 76 insertions(+), 10 deletions(-) -Peff