[Pixman] [PATCH 1/8] Add platform specific mutex implementations

Taekyun Kim podain77 at gmail.com
Tue Sep 27 20:27:59 PDT 2011


This is definitely a mistake.
I will fix it up

Thanks,
Taekyun Kim

On 09/27/2011 11:03 PM, Søren Sandmann wrote:
> Taekyun Kim<podain77 at gmail.com>  writes:
>
>> +#else
>> +
>> +typedef int pixman_mutex_t;
>> +
>> +#   define PIXMAN_MUTEX_INIT(mutex)	do while (0) {}
>> +#   define PIXMAN_MUTEX_LOCK(mutex)	do while (0) {}
>> +#   define PIXMAN_MUTEX_UNLOCK(mutex)	do while (0) {}
>> +#   define PIXMAN_MUTEX_FINI(mutex)	do while (0) {}
>> +
>> +#   warning "Unknown mutex support for this system. Pixman will work
>> without mutex support."
> Does
>
>       do while (0) {}
>
> actually compile? You probably mean
>
>       do { } while (0)
>
>
> Soren

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/pixman/attachments/20110928/331d7b16/attachment.htm>


More information about the Pixman mailing list