|
From: | Han-Wen Nienhuys |
Subject: | Re: More improvements to PostScript backend |
Date: | Mon, 03 Apr 2006 20:50:38 +0200 |
User-agent: | Thunderbird 1.5 (X11/20060313) |
David Feuer wrote:
In my nonexistent free time I've done some more work. I'd really like to do something about the compulsive gsave x y translate 0 0 moveto grestore wrapping that everything gets. Could someone give me an idea of what could get stuffed into placebox? Is there actually some kind of global graphics state that we need to maintain as a default? It looks like pretty much everything sets all the variables it cares about. I'd like to change all the drawing procedures to draw at the current point. If they want to translate to make their lives easierthey should clean up after themselves.
Sounds good.
Then the box wrapper becomes simply x y moveto, and we don't have to waste our time pushing and popping graphics states for no reason. By the way, in the future, should I include patches inline or attach them? This time I did both. The attached one is less likely to have line wrapping problems. This patch is against the versions in CVS.
Your patch looks fine, but can you make a proper changelog entry (see eg. http://www.gnu.org/prep/standards/html_node/Style-of-Change-Logs.html#Style-of-Change-LogsYou can put the log in the entry of the email. If you use emacs, just press C-x 4 a to add it.
-- Han-Wen Nienhuys - address@hidden - http://www.xs4all.nl/~hanwen LilyPond Software Design -- Code for Music Notation http://www.lilypond-design.com
[Prev in Thread] | Current Thread | [Next in Thread] |