[cairo] New per-scaled_font mutex to fix locking bugs

Carl Worth cworth at cworth.org
Tue Feb 6 17:55:48 PST 2007


On Mon, 05 Feb 2007 17:00:25 -0800, Carl Worth wrote:
> We (Keith came up with all the good parts of this patch) hope this
> solves the problems, and would be interested in having people that hit
> the problems test this patch and report back to us.

I got an affirmative report from Monty who had reported the original
bug. Thanks!

> For win32, I took a guess and came up with the following
>
> 	typedef CRITICAL_SECTION cairo_mutex_t;
> 	# define CAIRO_MUTEX_INIT(mutex) InitializeCriticalSection (mutex)
> 	# define CAIRO_MUTEX_FINI(mutex) DeleteCriticalSection (mutex)
> 	# define CAIRO_MUTEX_NIL_INITIALIZER {}

Vlad suggested I use "{ 0 }" there instead, which I've done now.

> And for BeOS and OS/2 I just plain broke the compilation since I had
> no idea what the correct stuff is there. Patches welcome.

And thanks to Peter for the OS/2 stuff.

So all that's in now. I even cleaned up the patch into 4 separate
commits and I've pushed it out now.

Have fun, (especially with the new, more thread-friendly cairo),

-Carl
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/cairo/attachments/20070206/cf74f1a6/attachment.pgp


More information about the cairo mailing list