[Mesa-dev] [PATCH 7/7] c11: Make TIME_UTC a conditional macro.

[Brian Paul]

LGTM.  Reviewed-by: Brian Paul <brianp at vmware.com>

On 01/09/2014 12:32 PM, Jose Fonseca wrote:
> This series is pretty much what I proposed several months ago, rebased on top of master, with minor if any tweaks.
>
> Still to do is remove the deprecated abstractions (which now are mere wrappers of the c11 ones).
>
> That said, this series is already useful as is, not only because of the code de-duplication, but also because the Windows conditional var implemention in master is poll-based so performs quite badly, in particular llvmpipe with many threads.  Which is fixed here.
>
> IMHO the rest can be done as time permits...
>
> Jose
>
> ----- Original Message -----
>> From: José Fonseca <jfonseca at vmware.com>
>>
>> Some system headers already define it, so this prevents conflict.
>> ---
>>   include/c11/threads.h | 6 ++++--
>>   1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/include/c11/threads.h b/include/c11/threads.h
>> index 678774c..4a4079d 100644
>> --- a/include/c11/threads.h
>> +++ b/include/c11/threads.h
>> @@ -10,6 +10,10 @@
>>
>>   #include <time.h>
>>
>> +#ifndef TIME_UTC
>> +#define TIME_UTC 1
>> +#endif
>> +
>>   #include "c99_compat.h" /* for `inline` */
>>
>>   /*---------------------------- types ----------------------------*/
>> @@ -39,8 +43,6 @@ enum {
>>       thrd_nomem        // out of memory
>>   };
>>
>> -enum { TIME_UTC = 1 };
>> -
>>   /*-------------------------- functions --------------------------*/
>>
>>   #if defined(_WIN32) && !defined(__CYGWIN__)
>> --
>> 1.8.3.2
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>>