[cairo] [cairo-commit] 7 commits - src/cairo.c src/cairo-color.c src/cairo-gl-glyphs.c src/cairo-gl-private.h src/cairo-gl-surface.c src/cairo-gstate.c src/cairo-image-surface.c test/Makefile.sources test/zero-mask.c test/zero-mask.ref.png test/zero-mask.rgb24.ref.png

[Eric Anholt]

On Thu, 13 May 2010 13:15:11 +0200, Benjamin Otte <otte at redhat.com> wrote:
> On Thu, 2010-05-13 at 12:00 +0100, Chris Wilson wrote:
> > Eric thinks this a necessary as creating surfaces in GL is not cheap. The
> > canonical place to clean up caches and break circular-references is in
> > cairo_device_finish().
> > 
> I benchmarked this with gnome-terminal-vim, which should hit that case a
> lot and didn't notice any differences. That said, if Eric thinks it's a
> good thing to have, I'll readd it.

Profile driven optimization, imo.  Sounds like you've found that it
didn't matter, so I'm not going to contradict that without bringing
numbers.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.cairographics.org/archives/cairo/attachments/20100513/aa227f30/attachment.pgp>