[PATCH] Changes the default size of time_t to 64 bit.

[Sebastian Huber]

On 25/08/17 14:06, Corinna Vinschen wrote:

> On Aug 25 08:15, Sebastian Huber wrote:
>> Currently we have:
>>
>> #define    _TIME_T_    long        /* time() */
>>
>> If we change this type, it is probably nice to add a configure option to
>> restore this definition for those who trade off binary compatibility with a
>> year 2038 readiness.
>>
>> I am still in favour of --enable-long-time_t, since this corresponds exactly
>> to the current definition.
>>
>> We should make the type change only if necessary, e.g. something like this:
>>
>> #if defined(_USE_LONG_TIME_T) || __LONG_MAX__ > 0x7fffffffL
>>
>> #define _TIME_T_ long
>>
>> #else
>>
>> #define _TIME_T_ __int_least64_t
>>
>> #endif
> Fine with me, unless Cygwin.

We should probably add this to <sys/config.h>:

#ifdef _WANT_USE_LONG_TIME_T
#ifndef _USE_LONG_TIME_T
#define _USE_LONG_TIME_T
#endif
#endif

The _WANT_USE_LONG_TIME_T is defined by --enable-long-time_t. Cygwin (or 
any other system) can use <cygwin/config.h> to define _USE_LONG_TIME_T. 
Is this ok?

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.