Modify ↓

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#20810 closed defect (fixed)

NPE at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$Factory.lambda$ofNumberVarArgs$1

Reported by: skyper Owned by: simon04
Priority: normal Milestone: 21.04
Component: Core mappaint Version: latest
Keywords: template_report mapcss expression factory regression Cc: simon04

Description (last modified by skyper)

On startup I got a silent NPE, only found it by chance in the console log.
As the deb package was not updated last night, I am still running r17821, so this could be already fixed.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2021-04-25 09:16:28 +0200 (Sun, 25 Apr 2021)
Revision:17821
Build-Date:2021-04-26 01:31:00
URL:https://josm.openstreetmap.de/svn/trunk

Map paint styles:
+ https://josm.openstreetmap.de/josmfile?page=Styles/Lane_and_Road_Attributes&zip=1

Last errors/warnings:
- 00134.955 E: java.lang.NullPointerException
- 00134.958 E: java.lang.NullPointerException

2021-04-27 14:39:06.263 SEVERE: java.lang.NullPointerException
java.lang.NullPointerException
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$Factory.lambda$ofNumberVarArgs$1(ExpressionFactory.java:62)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$Factory.lambda$of$11(ExpressionFactory.java:86)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:337)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$Factory.lambda$of$11(ExpressionFactory.java:86)
	at org.openstreetmap.josm.gui.mappaint.mapcss.Instruction$AssignmentInstruction.execute(Instruction.java:95)
	at org.openstreetmap.josm.gui.mappaint.mapcss.Declaration.execute(Declaration.java:43)
	at org.openstreetmap.josm.gui.mappaint.mapcss.MapCSSRule.execute(MapCSSRule.java:58)
	at org.openstreetmap.josm.gui.mappaint.mapcss.MapCSSStyleSource.apply(MapCSSStyleSource.java:387)
	at org.openstreetmap.josm.gui.mappaint.ElemStyles.generateStyles(ElemStyles.java:386)
	at org.openstreetmap.josm.gui.mappaint.ElemStyles.getImpl(ElemStyles.java:248)
	at org.openstreetmap.josm.gui.mappaint.ElemStyles.getStyleCacheWithRange(ElemStyles.java:161)
	at org.openstreetmap.josm.gui.mappaint.ElemStyles.get(ElemStyles.java:139)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.add(ComputeStyleListWorker.java:171)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.visit(ComputeStyleListWorker.java:143)
	at org.openstreetmap.josm.data.osm.Way.accept(Way.java:182)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.acceptDrawable(ComputeStyleListWorker.java:129)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.computeDirectly(ComputeStyleListWorker.java:116)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.compute(ComputeStyleListWorker.java:93)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.compute(ComputeStyleListWorker.java:34)
	at java.base/java.util.concurrent.RecursiveTask.exec(RecursiveTask.java:94)
	at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:290)
	at java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1020)
	at java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1656)
	at java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1594)
	at java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:183)

2021-04-27 14:39:06.266 SEVERE: java.lang.NullPointerException
java.lang.NullPointerException
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$Factory.lambda$ofNumberVarArgs$1(ExpressionFactory.java:62)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$Factory.lambda$of$11(ExpressionFactory.java:86)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:337)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$CondOperator.evaluate(ExpressionFactory.java:339)
	at org.openstreetmap.josm.gui.mappaint.mapcss.ExpressionFactory$Factory.lambda$of$11(ExpressionFactory.java:86)
	at org.openstreetmap.josm.gui.mappaint.mapcss.Instruction$AssignmentInstruction.execute(Instruction.java:95)
	at org.openstreetmap.josm.gui.mappaint.mapcss.Declaration.execute(Declaration.java:43)
	at org.openstreetmap.josm.gui.mappaint.mapcss.MapCSSRule.execute(MapCSSRule.java:58)
	at org.openstreetmap.josm.gui.mappaint.mapcss.MapCSSStyleSource.apply(MapCSSStyleSource.java:387)
	at org.openstreetmap.josm.gui.mappaint.ElemStyles.generateStyles(ElemStyles.java:386)
	at org.openstreetmap.josm.gui.mappaint.ElemStyles.getImpl(ElemStyles.java:248)
	at org.openstreetmap.josm.gui.mappaint.ElemStyles.getStyleCacheWithRange(ElemStyles.java:161)
	at org.openstreetmap.josm.gui.mappaint.ElemStyles.get(ElemStyles.java:139)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.add(ComputeStyleListWorker.java:171)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.visit(ComputeStyleListWorker.java:143)
	at org.openstreetmap.josm.data.osm.Way.accept(Way.java:182)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.acceptDrawable(ComputeStyleListWorker.java:129)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.computeDirectly(ComputeStyleListWorker.java:116)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.compute(ComputeStyleListWorker.java:93)
	at org.openstreetmap.josm.data.osm.visitor.paint.ComputeStyleListWorker.compute(ComputeStyleListWorker.java:34)
	at java.base/java.util.concurrent.RecursiveTask.exec(RecursiveTask.java:94)
	at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:290)
	at java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1020)
	at java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1656)
	at java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1594)
	at java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:183)

Attachments (0)

Change History (4)

comment:1 by skyper, 4 years ago

Description: modified (diff)

comment:2 by simon04, 4 years ago

Milestone: → 21.04
Owner: changed from team to simon04
Status: new → assigned

comment:3 by simon04, 4 years ago

Resolution: → fixed
Status: assigned → closed

In 17832/josm:

see #20744, fix #20810 - MapCSS: fix NPE

comment:4 by simon04, 4 years ago

Keywords: regression added

Regression of r17758.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain simon04.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.