fsck: remove redundant date timestamp check
After calling "parse_timestamp(p, &end, 10)", we complain if "p == end", which would imply that we did not see any digits at all. But we know this cannot be the case, since we would have bailed already if we did not see any digits, courtesy of extra checks added by 8e4309038f (fsck: do not assume NUL-termination of buffers, 2023-01-19). Since then, checking "p == end" is redundant and we can drop it. This will make our lives a little easier as we refactor further. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Jeff King committed
Nov 18, 2025 at 04:12 UTC
f05df7ffca492b37d604ad6beed788055eb56ebd
1 file changed
+1
-1
fsck.c
+1
-1
@@ -919,7 +919,7 @@ static int fsck_ident(const char **ident, const char *ident_end,
919
return report(options, oid, type, FSCK_MSG_ZERO_PADDED_DATE, "invalid author/committer line - zero-padded date");
920
if (date_overflows(parse_timestamp(p, &end, 10)))
921
return report(options, oid, type, FSCK_MSG_BAD_DATE_OVERFLOW, "invalid author/committer line - date causes integer overflow");
922
- if ((end == p || *end != ' '))
922
+ if (*end != ' ')
923
return report(options, oid, type, FSCK_MSG_BAD_DATE, "invalid author/committer line - bad date");
924
p = end + 1;
925
if ((*p != '+' && *p != '-') ||