[v2,2/2] Fix time/mktime.c and time/gmtime.c for gnulib compatibility
Commit Message
From: Paul Eggert <eggert@cs.ucla.edu>
Tested with 'make check' on x86_64-linux-gnu and i686-linux.gnu.
* time/mktime.c
(__mktime64): Guard weak_alias with #ifdef
(__mktime64): Guard libc_hidden_def with #if _LIBC
(__mktime64): Guard libc_hidden_weak with #if _LIBC
(mktime): Only build when _LIBC is defined
* time/timegm.c:
(timegm): Only build when _LIBC is defined
---
time/mktime.c | 15 +++------------
time/timegm.c | 2 +-
2 files changed, 4 insertions(+), 13 deletions(-)
Comments
On 2/27/19 3:20 AM, Lukasz Majewski wrote:
> -#ifdef weak_alias
> weak_alias (__mktime64, __timelocal64)
> -#endif
This part of the patch won't be needed if my suggestion about removing
timelocal64 (from the previous patch) is taken. Otherwise this patch
looks OK to me once the earlier patch is settled on.
@@ -549,18 +549,14 @@ __mktime64 (struct tm *tp)
return mktime (tp);
# endif
}
-#endif /* _LIBC || NEED_MKTIME_WORKING || NEED_MKTIME_WINDOWS */
-#ifdef weak_alias
weak_alias (__mktime64, __timelocal64)
-#endif
-
-#ifdef _LIBC
libc_hidden_def (__mktime64)
libc_hidden_weak (__timelocal64)
-#endif
-#if __TIMESIZE != 64
+#endif /* _LIBC || NEED_MKTIME_WORKING || NEED_MKTIME_WINDOWS */
+
+#if defined _LIBC && __TIMESIZE != 64
/* The 32-bit-time wrapper. */
time_t
@@ -573,13 +569,8 @@ mktime (struct tm *tp)
return -1;
}
-#ifdef weak_alias
weak_alias (mktime, timelocal)
-#endif
-
-#ifdef _LIBC
libc_hidden_def (mktime)
libc_hidden_weak (timelocal)
-#endif
#endif
@@ -35,7 +35,7 @@ __timegm64 (struct tm *tmp)
return __mktime_internal (tmp, __gmtime64_r, &gmtime_offset);
}
-#if __TIMESIZE != 64
+#if defined _LIBC && __TIMESIZE != 64
time_t
timegm (struct tm *tmp)