[PATCH] Exclude all but ways with highway=* from Lanes.java test

[skyper]

For better performance, Lanes.java​ should only be run on ways with highway=*.

r16198 included some general restrictions but it could be stricter. Or do I misread the code?

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2020-08-06 21:47:58 +0200 (Thu, 06 Aug 2020)
Revision:16851
Build-Date:2020-08-07 01:30:47
URL:https://josm.openstreetmap.de/svn/trunk

[Gabe]

so an addition like p.hasTag("highway") is what you are looking for in the isPrimitiveUsable function to be more strict about when this validation is used?

[Gabe]

patch for Lanes.java

[Gabe]

correct extension

[Gabe]

REAL correct extension.

[GerdP]

I was not sure if highways are the only objects for which this test should be executed, but it seems true.

[GerdP]

In 17861/josm:

fix #19647: Exclude all but ways with highway=* from Lanes.java test

• make isPrimitiveUsable() more restrictive (patch by Gabe, modified)

Unit tests bypass this method so they need no changes.

[skyper]

Replying to GerdP:

I was not sure if highways are the only objects for which this test should be executed, but it seems true.

There might be few more tags but only checking the majority is fine, as there are only low numbers of use beside highway=*.