NPE on zero node ways

[anonymous]

Path: trunk
URL: ​http://josm.openstreetmap.de/svn/trunk
Repository Root: ​http://josm.openstreetmap.de/svn
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Revision: 1583
Node Kind: directory
Last Changed Author: stoecker
Last Changed Rev: 1583
Last Changed Date: 2009-05-07 20:00:43 +0200 (Thu, 07 May 2009)

Java version: 1.6.0_13

java.lang.IndexOutOfBoundsException: Index: 0, Size: 0

at java.util.ArrayList.RangeCheck(Unknown Source)
at java.util.ArrayList.get(Unknown Source)
at org.openstreetmap.josm.data.osm.Way.firstNode(Way.java:180)
at org.openstreetmap.josm.data.osm.Way.isFirstLastNode(Way.java:184)
at org.openstreetmap.josm.actions.mapmode.DrawAction.computeHelperLine(DrawAction.java:673)
at org.openstreetmap.josm.actions.mapmode.DrawAction.mouseMoved(DrawAction.java:589)
at java.awt.AWTEventMulticaster.mouseMoved(Unknown Source)
at java.awt.Component.processMouseMotionEvent(Unknown Source)
at javax.swing.JComponent.processMouseMotionEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

[stoecker]

What did you do before this? Is this reproducable?

[xeen]

lastNode/firstNode both assume the way consists of at least 1 node. So… where does a zero-node way come from?

[stoecker]

Zero node ways are buggy but nevertheless they can exist. They should be removed from database thought.

[stoecker]

Fixed in r1611.