|
From: | Fred Kiefer |
Subject: | Re: GSGState DPSrectclip: |
Date: | Wed, 01 Oct 2003 23:29:37 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20021204 |
Hi Adam,thank you for fixing this. I had looked at the bug report a few times but could not make up my mind on the best way to handle this. Your solution is of course correct, but may not be the most efficient one, as we could also reuse the temp bezier path we just created, if there has been none before. But than again, we don't know if we will ever need this path later on.
So this patch is fine for me, but we should also fix all the real implementations in the subclasses (the fixed code actually never gets executed). This should be easy for winlib and xlib (just the saem two lines), but I don't see if it is needed for art. I never did understand any of the code in that backend.
Cheers Fred Banlu Kemiyatorn wrote:
Pete French wrote:Why would it keep old path there rather destroy it?because DPSrectclip is not supposed to alter the current path ?From PostScript® language reference“Rectangles”). ***After computing the new clipping path, rectclip clears the currentpath with an implicit newpath operation.***
[Prev in Thread] | Current Thread | [Next in Thread] |