[cairo] Another Gradient rendering speedup patch

Carl Worth cworth at cworth.org
Fri Feb 2 16:49:33 PST 2007


On Fri, 2 Feb 2007 09:32:30 UT, "David Turner" wrote:
> sorry, this bug report escaped me. Here's a patch that fixes the
> problem and includes the test case provided in the original bug
> report.

No worries.

Thanks for the very quick fix! Asking you for that was much easier
than if I had tried to diagnose and fix this myself.

This fix is all pushed out now, (still need to finish up a radial
gradient test case before pushing out those performance improvements).

With this fix, we've now got 2 out of the 3 bad bugs fixed that were
originally known in 1.3.12, and that I said we wanted fixed before
1.3.14.

The third bug is the locking problems as described here:

	text rendering lacking locking in multithreaded apps [PATCH]
	https://bugs.freedesktop.org/show_bug.cgi?id=8801

I've decided that we want to do a little more cleanup than in the
patches currently there in bugzilla. And if we're having to touch this
glyph cache code to fix its locking, we might as well fix it to
address some performance limitations already documented in the
implementation.

So Keith and I are planning to work on those things on Monday.

Meanwhile, one other nasty bug has also cropped up that's in 1.3.12,
and that's the spacing problem in cairo_show_text with cairo-win32. Is
there anybody out there that is planning on looking into that more
closely?

-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/20070202/dbe3636a/attachment.pgp


More information about the cairo mailing list