From: Jeff King Date: Tue, 25 Apr 2023 05:52:44 GMT Subject: [PATCH v2 0/3] fixing some parse_commit() timestamp corner cases Message-ID: <20230425055244.GA4014505@coredump.intra.peff.net> In-Reply-To: <20230422134150.GA3516940@coredump.intra.peff.net> 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