From: Earnie B. <ea...@us...> - 2012-07-22 00:01:31
|
I'm reviewing our headers in preparation for _WIN64 use. I came upon the following definition and believe that we should remove the __MSVCRT_VERSION__ filter based on the fact that MSDN states it should default to __time64_t. See the notes at http://msdn.microsoft.com/en-us/library/w4ddyt9h(v=vs.80).aspx for this. #ifndef _TIME_T_DEFINED /* FIXME __STRICT_ANSI__ ! */ #if __MSVCRT_VERSION__ >= 0x0800 #ifndef _USE_32BIT_TIME_T typedef __time64_t time_t; #else typedef __time32_t time_t; #endif /* !_USE_32BIT_TIME_T */ #else typedef __time32_t time_t; #endif /* __MSVCRT_VERSION__ >= 0x0800 */ #define _TIME_T_DEFINED #endif I want to change it to #ifndef _TIME_T_DEFINED #ifndef _USE_32BIT_TIME_T typedef __time64_t time_t; #else typedef __time32_t time_t; #endif /* !_USE_32BIT_TIME_T */ #define _TIME_T_DEFINED #endif Thoughts? -- Earnie -- https://sites.google.com/site/earnieboyd |