Re: [cp-patches] RFA: CairoGraphics2D.java fixlet

[Mark Wielaard]

Hi,

On Thu, 2006-06-15 at 02:19 +0200, Sven de Marothy wrote:
> On Wed, 14 Jun 2006, David Gilbert wrote:
> > As I understand the getPathIterator(AffineTransform) method defined in the
> > Shape interface, if you want the path returned without transformation, you 
> > can
> > pass in null for the AffineTransform.  So it is wasteful to create a new
> > identity transform as we seem to be doing here.
> 
> Seems sane to me.

Found another one while debugging some other code in BasicStroke:

2006-06-15  Mark Wielaard  <[EMAIL PROTECTED]>

    * java/awt/BasicStroke.java (createStrokedShape): Call
    getPathIterator with null argument.

Committed,

Mark

diff -u -r1.12 BasicStroke.java
--- java/awt/BasicStroke.java   10 May 2006 14:42:26 -0000      1.12
+++ java/awt/BasicStroke.java   15 Jun 2006 18:06:13 -0000
@@ -260,7 +260,7 @@
    */
   public Shape createStrokedShape(Shape s)
   {
-    PathIterator pi = s.getPathIterator( new AffineTransform() );
+    PathIterator pi = s.getPathIterator(null);

     if( dash == null )
       return solidStroke( pi );

signature.asc
Description: This is a digitally signed message part