changeset 39530:461b806a26b5

mktime, timegm: simplify glibc time64_t * lib/mktime.c, lib/timegm.c (mktime_offset_t) [_LIBC]: Now long int, not time_t, since long int is the longstanding type for this in glibc and there is no need to change it even if time_t becomes 64 bits - even int would do, though this would be a change to the glibc generated code. When this change is merged into glibc, it should simplify the time_t vs time64_t situation.
author Paul Eggert <eggert@cs.ucla.edu>
date Fri, 31 Aug 2018 18:02:48 -0700
parents 747a5774e1e0
children e886d9ea220c
files ChangeLog lib/mktime.c lib/timegm.c
diffstat 3 files changed, 10 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Fri Aug 31 18:02:48 2018 -0700
+++ b/ChangeLog	Fri Aug 31 18:02:48 2018 -0700
@@ -1,5 +1,13 @@
 2018-08-31  Paul Eggert  <eggert@cs.ucla.edu>
 
+	mktime, timegm: simplify glibc time64_t
+	* lib/mktime.c, lib/timegm.c (mktime_offset_t) [_LIBC]:
+	Now long int, not time_t, since long int is the longstanding type
+	for this in glibc and there is no need to change it even if time_t
+	becomes 64 bits - even int would do, though this would be a change
+	to the glibc generated code.  When this change is merged into
+	glibc, it should simplify the time_t vs time64_t situation.
+
 	mktime, timegm: simplify merge to glibc
 	Move code around to make a merge to glibc easier to audit.
 	This should not change behavior.
--- a/lib/mktime.c	Fri Aug 31 18:02:48 2018 -0700
+++ b/lib/mktime.c	Fri Aug 31 18:02:48 2018 -0700
@@ -77,7 +77,7 @@
 #endif
 
 #ifdef _LIBC
-typedef time_t mktime_offset_t;
+typedef long int mktime_offset_t;
 #else
 # include "mktime-internal.h"
 #endif
--- a/lib/timegm.c	Fri Aug 31 18:02:48 2018 -0700
+++ b/lib/timegm.c	Fri Aug 31 18:02:48 2018 -0700
@@ -24,7 +24,7 @@
 #include <time.h>
 
 #ifdef _LIBC
-typedef time_t mktime_offset_t;
+typedef long int mktime_offset_t;
 #else
 # include "mktime-internal.h"
 #endif