partly broken reverts?

[malenki]

What steps will reproduce the problem?
(0. have plugin measurement loaded)

1. revert changeset 12514596
2. press ctrl-a

What is the expected result?

All data gets selected.

What happens instead?

All data gets selected and the user gets asked if he wants to disable plugin measurement since it most probably caused an error.

Most probably the measurement plugin crashes because the reverter plugin created ways of infinite/undefined length (see screenshot).

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2012-11-23 02:31:14
Last Changed Author: stoecker
Revision: 5597
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2012-11-22 17:44:05 +0100 (Thu, 22 Nov 2012)
Last Changed Rev: 5597

Identification: JOSM/1.5 (5597 de)
Memory Usage: 2539 MB / 4711 MB (1741 MB allocated, but free)
Java version: 1.6.0_24, Sun Microsystems Inc., OpenJDK 64-Bit Server VM
Operating system: Linux
Dataset consistency test: No problems found

Plugin: DirectUpload (28807)
Plugin: EditGpx (0.90)
Plugin: FastDraw (28807)
Plugin: FixAddresses (28807)
Plugin: OpeningHoursEditor (28656)
Plugin: PicLayer (28807)
Plugin: buildings_tools (28891)
Plugin: contourmerge (1003)
Plugin: download_along (28807)
Plugin: editgpx (28807)
Plugin: ext_tools (28807)
Plugin: imageryadjust (28807)
Plugin: jts (28945)
Plugin: lakewalker (28807)
Plugin: mapdust (28807)
Plugin: measurement (28898)
Plugin: mirrored_download (28939)
Plugin: openstreetbugs (28807)
Plugin: openvisible (28807)
Plugin: reltoolbox (28857)
Plugin: reverter (28820)
Plugin: surveyor2 (28656)
Plugin: tag2link (28807)
Plugin: tageditor (28807)
Plugin: terracer (28807)
Plugin: turnrestrictions (28807)
Plugin: undelete (28807)
Plugin: utilsplugin2 (28807)
Plugin: waydownloader (28492)
Plugin: wayselector (28807)

java.lang.NullPointerException
	at org.openstreetmap.josm.data.coor.LatLon.greatCircleDistance(LatLon.java:231)
	at org.openstreetmap.josm.plugins.measurement.MeasurementDialog.selectionChanged(MeasurementDialog.java:138)
	at org.openstreetmap.josm.data.osm.DataSet.fireSelectionChanged(DataSet.java:442)
	at org.openstreetmap.josm.data.osm.DataSet.setSelected(DataSet.java:615)
	at org.openstreetmap.josm.data.osm.DataSet.setSelected(DataSet.java:626)
	at org.openstreetmap.josm.actions.SelectAllAction.actionPerformed(SelectAllAction.java:23)
	at javax.swing.SwingUtilities.notifyAction(SwingUtilities.java:1660)
	at javax.swing.JComponent.processKeyBinding(JComponent.java:2852)
	at javax.swing.KeyboardManager.fireBinding(KeyboardManager.java:285)
	at javax.swing.KeyboardManager.fireKeyboardAction(KeyboardManager.java:247)
	at javax.swing.JComponent.processKeyBindingsForAllComponents(JComponent.java:2929)
	at javax.swing.JComponent.processKeyBindings(JComponent.java:2921)
	at javax.swing.JComponent.processKeyEvent(JComponent.java:2815)
	at java.awt.Component.processEvent(Component.java:6166)
	at java.awt.Container.processEvent(Container.java:2045)
	at java.awt.Component.dispatchEventImpl(Component.java:4750)
	at java.awt.Container.dispatchEventImpl(Container.java:2103)
	at java.awt.Component.dispatchEvent(Component.java:4576)
	at java.awt.KeyboardFocusManager.redispatchEvent(KeyboardFocusManager.java:1895)
	at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(DefaultKeyboardFocusManager.java:769)
	at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(DefaultKeyboardFocusManager.java:1045)
	at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(DefaultKeyboardFocusManager.java:912)
	at java.awt.DefaultKeyboardFocusManager.dispatchEvent(DefaultKeyboardFocusManager.java:734)
	at java.awt.Component.dispatchEventImpl(Component.java:4620)
	at java.awt.Container.dispatchEventImpl(Container.java:2103)
	at java.awt.Window.dispatchEventImpl(Window.java:2518)
	at java.awt.Component.dispatchEvent(Component.java:4576)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:672)
	at java.awt.EventQueue.access$400(EventQueue.java:96)
	at java.awt.EventQueue$2.run(EventQueue.java:631)
	at java.awt.EventQueue$2.run(EventQueue.java:629)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:105)
	at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:116)
	at java.awt.EventQueue$3.run(EventQueue.java:645)
	at java.awt.EventQueue$3.run(EventQueue.java:643)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:105)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:642)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:275)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:200)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:190)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:185)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:177)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:13

[malenki]

infinite "restored" ways

[malenki]

Maybe see also #8240

[Don-vip]

Should be fixed in [o29553] (JOSM latest required).