From 1820703045f8974bc5320d08a3611f4e29c83bf9 Mon Sep 17 00:00:00 2001 From: Derrick Stolee Date: Tue, 21 Aug 2018 20:54:12 +0000 Subject: commit: use timestamp_t for author_date_slab The author_date_slab is used to store the author date of a commit when walking with the --author-date flag in rev-list or log. This was added as an 'unsigned long' in 81c6b38b "log: --author-date-order" Since 'unsigned long' is ambiguous in its bit-ness across platforms (64-bit in Linux, 32-bit in Windows, for example), most references to the author dates in commit.c were converted to timestamp_t in dddbad72 "timestamp_t: a new data type for timestamps" However, the slab definition was missed, leading to a mismatch in the data types in Windows. This would not reveal itself as a bug unless someone authors a commit after February 2106, but commits can store anything as their author date. Signed-off-by: Derrick Stolee Signed-off-by: Junio C Hamano diff --git a/commit.c b/commit.c index 99a62b9..801b6dc 100644 --- a/commit.c +++ b/commit.c @@ -588,7 +588,7 @@ struct commit *pop_commit(struct commit_list **stack) define_commit_slab(indegree_slab, int); /* record author-date for each commit object */ -define_commit_slab(author_date_slab, unsigned long); +define_commit_slab(author_date_slab, timestamp_t); static void record_author_date(struct author_date_slab *author_date, struct commit *commit) -- cgit v0.10.2-6-g49f6