[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: time_t in gnulib
Re: time_t in gnulib
Thu, 05 Jul 2018 22:50:04 +0200
KMail/5.1.3 (Linux/4.4.0-128-generic; KDE/5.18.0; x86_64; ; )
> I have had a look at gnulib in the meantime, and I would like to know
> if the following assumptions are correct:
> - gnulib contains a year2038 module which is only intended to check
> whether time_t is limited to Y2038 or not.
Yes and no.
Yes, gnulib contains a year2038 module .
No, it does not "check" something: It attempts to ensure that time_t is a
64-bit type, by defining certain preprocessor symbols.
No, it does not guarantee a configuration error or compilation error if
time_t is only 32-bit. It produces a configuration error ONLY if time_t
is only 32-bit AND this can be avoided by choosing a different ABI.
> - gnulib does not contain any module which provides the time_t type, but
> some of gnulib's modules assume there is such a type, and it might be
> wider than 32 bits.
Correct. All portability targets of gnulib do have a time_t type. Since
it is not possible to support a wider time_t type without kernel support,
gnulib does not attempt to redefine time_t type via typedef or #define.
> - gnulib does not provide difftime either, so ATM a difftime patch
> would only make sense in glibc, not gnulib.
difftime is only the least problem. The bigger ones are gettimeofday,
stat, fstat. But if on some platform, difftime is the only missing piece
for supporting 64-bit time_t, gnulib could easily provide a replacement.
> - gnulib makes no assumption on how it is used in object form; notably,
> it makes no assumption that it will be compiled into a shared object
> form which will provide the same functionalities for both 64-bit and
> 32-bit callers.
Correct. Gnulib provides an "application source interface" only, not an
"application binary interface".
Re: [PATCH 1/1] Y2038: add function __difftime64, Joseph Myers, 2018/07/17
Re: time_t in gnulib,
Bruno Haible <=
Re: [PATCH 1/1] Y2038: add function __difftime64, Paul Eggert, 2018/07/05
- Re: [PATCH 1/1] Y2038: add function __difftime64, (continued)