[PATCH 1/2] print: Use cairo_save/cairo_restore to manage temporary cairo changes

Dirk Hohndel dirk at hohndel.org
Thu Jan 31 21:08:31 PST 2013


Carl Worth <cworth at cworth.org> writes:
>
> Pretty much every drawing function in print.c should have similar
> treatment applied. I'm just doing this one to enable my next patch,
> (which fixes the line spacing in the "weight system" box), and to
> teach you guys. So it should be straightforward to similarly fix the
> other functions.
>
> Also, this commit has a side effect of changing the layout of the text
> in the box just to the left of the weight-system box. Apparently, the
> "undo translation" code here was buggy and was not returning to the
> exact starting point. So the rendering code for the box to the left
> was relying on a buggy starting point. It should be fixed, of course.

Salvador, is this something you can take a look at in the next few days?
I'm planning to apply Carl's patches and would love to see this applied
to all of print.c in the hope that this gives us more robust print
rendering.

Thanks

/D


More information about the subsurface mailing list