Larry wrote: "The fix is:

               private int getSegType(){
                       if (closed && (iterate == numPoints-1))
                               return PathIterator.SEG_CLOSE;
                       return (iterate==0) ? PathIterator.SEG_MOVETO :
PathIterator.SEG_LINETO;
               }"

We should commit this, if you haven't done so already. :]

Larry wrote: "I would also like to increase the accuracy of the rendering system
from float to double by replacing all references to GeneralPath with
Path2D.Double.  There is no speed or memory penalty for this change."

This sounds reasonable to me. Let's go for it if there are no other
objections. I think you and Sascha know more about OpenJUMP's
rendering system than anyone else at this point.

The Sunburned Surveyor

On 7/12/07, Larry Becker <[EMAIL PROTECTED]> wrote:
> We recently implemented some modifications to Java2DConverter and
> PolygonShape to use PathIterator.  The implementation was slightly
> flawed and resulted in paths that didn't use a SEG_CLOSE which causes
> closed paths to fail to connect start and end with a JOIN_BEVEL.  You
> can see the problem very easily if you start OJ, drag out a rectangle,
> and then increase the line width to 10.
>
> The fix is:
>
>                private int getSegType(){
>                        if (closed && (iterate == numPoints-1))
>                                return PathIterator.SEG_CLOSE;
>                        return (iterate==0) ? PathIterator.SEG_MOVETO : 
> PathIterator.SEG_LINETO;
>                }
>
> I would also like to increase the accuracy of the rendering system
> from float to double by replacing all references to GeneralPath with
> Path2D.Double.  There is no speed or memory penalty for this change.
>
> regards,
> Larry Becker
>
> --
> http://amusingprogrammer.blogspot.com/
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to