Opened 16 years ago

Closed 15 years ago

Last modified 15 years ago

#176 closed defect (fixed)

Exception when using PFixedWidthStroke

Reported by: mirschel Owned by: mirschel
Priority: major Milestone: Promot 0.8.3
Component: PromotGuiVizNav Version: 0.8.0
Severity: fatal Keywords: visual explorer piccolo PFixedWidthStroke exception
Cc:

Description

Piccolos piccolox.PFixedWidthStroke is buggy and generates missing initial moveto in path definition errors with a IllegalPathStateException? that cannot be caught because it is in the automatic repaint function.

Stacktrace: Exception in thread "AWT-EventQueue?-0" java.awt.geom.IllegalPathStateException?: missing initial moveto in path definition

at java.awt.geom.Path2D$Float.needRoom(Path2D.java:263) at java.awt.geom.Path2D.closePath(Path2D.java:1752) at edu.umd.cs.piccolox.util.PFixedWidthStroke$FillAdapter?.endPath(Unknown Source) at sun.dc.pr.PathStroker?.endPath(Native Method) at edu.umd.cs.piccolox.util.PFixedWidthStroke.createStrokedShape(Unknown Source) at edu.umd.cs.piccolo.nodes.PPath.getPathBoundsWithStroke(Unknown Source) ...

Reproducibility: It is not reproducible very well. Best test case is: Loading a logical model (toymodel), export layout, select mode 'Editing', move nodes around, import layout. Sometimes the exception is thrown.

This bug in known by the piccolo team (see issue tracker of piccolo) and other people, e.g. http://www.sfr-fresh.com/unix/privat/tesseract-2.03.tar.gz:a/tesseract-2.03/java/SVWindow.java. In the next piccolo version the bug might be fixed (it is on the agenda).

Change History (3)

comment:1 Changed 16 years ago by mirschel

  • Owner changed from promot to mirschel
  • Status changed from new to assigned
  • Summary changed from Exception when using PFIxedWidthStroke to Exception when using PFixedWidthStroke

This issue needs more testing. Maybe it can be avoided in some cases. This bug is also discussed in #117.

comment:2 Changed 15 years ago by mirschel

  • Resolution set to fixed
  • Status changed from assigned to closed

Should be fixed with release of piccolo1.3 that we use now in promot.

comment:3 Changed 15 years ago by mirschel

  • Milestone changed from Promot 1.0+ to Promot 0.8.3
Note: See TracTickets for help on using tickets.