W dniu 05.11.2018 o 16:53, Earnie via Mingw-w64-public pisze:
> On 11/4/2018 2:02 PM, Mateusz wrote:
> 
>> (but in _mingw.h we always define _USE_32BIT_TIME_T for 32-bit)
> 
> This is a wrong implementation.  32-bit Windows supports 64-bit time_t.
> 32-bit time_t should only be used if _USE_32BIT_TIME_T is set.

Yes, I agree -- in the future we should not define _USE_32BIT_TIME_T. I didn't 
change this because the patch is only for _ftime functions. If the patch 
survives in real life we could make next step and check/fix all time functions 
that depend on _USE_32BIT_TIME_T and not define _USE_32BIT_TIME_T.

> _USE_32BIT_TIME_T functions might not be in the MSVCRT.DLL residing on
> the OS which includes Windows 7.  This
> https://sourceforge.net/p/mingw/bugs/1973/ might be helpful.

It is quite long thread. In msvcrt.dll from WinXPsp2 we have only
_ftime
_ftime64
functions. In my patch we use _ftime32 and _ftime64 functions and we define 
_ftime to _ftime32 or _ftime64. It should work on WinXP because _ftime32 is 
aliased as _ftime for 32-bit libmsvcrt.a.

Regards,
Mateusz



_______________________________________________
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public

Reply via email to