Re: [cp-patches] Geom classes

[Mark Wielaard]

Hi,

On Mon, 2004-08-23 at 06:44, Sven de Marothy wrote:
> Here's a patch for the remaining unimplemented parts of java.awt.geom,
> with the exception of Area, which will be along in a day or two. David
> Gilbert supplied the nice fix for linesIntersect().
> 
> 2004-08-23 Sven de Marothy <address@hidden>
>       ArcIterator: Set to private.

But ArcIterator is used by Ellipse2D (Since ellipses are a subclass of
arcs, this is used by Ellipse2D as well):

  public PathIterator getPathIterator(AffineTransform at)
  {
    // An ellipse is just a complete arc.
    return new Arc2D.ArcIterator(this, at);
  }

And jikes correctly complains about ArcIterator being private.
This is a gcj bug.

(See also the autobuilder report on commit-classpath mailinglist).

Fixed as:

2004-08-25  Mark Wielaard  <address@hidden>

        * java/awt/geom/Arc2D.java (ArcIterator): Make package private.

Cheers,

Mark

--- java/awt/geom/Arc2D.java    25 Aug 2004 01:00:48 -0000      1.6
+++ java/awt/geom/Arc2D.java    25 Aug 2004 07:00:26 -0000
@@ -692,7 +692,7 @@
    *
    * @author Eric Blake (address@hidden)
    */
-  private static final class ArcIterator implements PathIterator
+  static final class ArcIterator implements PathIterator
   {
     /** The current iteration. */
     private int current;

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