jpb-phet wrote:
> I believe I have found a bug in the XYArray class when I was using it
> to create a PLine. Here is a simple main routine that duplicates the
> bug. In the version of Piccolo that I'm using, this throws an
> exception, and I've looked at the trunk and believe that the bug is
> still there.
>
> public class PiccoloBugDemo {
>
> public static void main( String[] args ) {
> XYArray pointArray = new XYArray( 10 );
> pointArray.setX( 0, 5 );
> System.out.println("Point count is: " +
> pointArray.getPointCount());
> System.out.println("X value of first point in array is: " +
> pointArray.getX( 0 ));
> }
> }
>
> The exception is thrown on the line "pointArray.setX" because the
> number of points is incorrectly set when using this constructor.
Thanks for the note, might you be able to create an issue for this?
http://code.google.com/p/piccolo2d/issues/list
> I was able to work around this easily by using a PPath and creating a
> shape, which makes me wonder if it makes sense to maintain the PLIne
> class and the related classes (such as this XYArray class). Perhaps
> they should be deprecated.
It might make sense to keep PLine around if it is more efficient than
PPath for lines. I can't say that it is.
michael
--
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
