Changeset 17780 in josm for trunk/src/org

Timestamp:

2021-04-14T20:00:37+02:00 (5 years ago)

Author:

simon04

Message:

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

File:

• trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/ExpressionFactory.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/ExpressionFactory.java

@@ -56 +56 @@
 
         static Factory ofNumberVarArgs(double identity, DoubleUnaryOperator unaryOperator, DoubleBinaryOperator operator) {
-            return args -> env -> args.isEmpty()
-                    ? identity
-                    : args.size() == 1
-                    ? unaryOperator.applyAsDouble(Cascade.convertTo(args.get(0).evaluate(env), Double.class))
-                    : args.stream()
-                    .map(arg -> Cascade.convertTo(arg.evaluate(env), Double.class))
-                    .filter(Objects::nonNull)
-                    .reduce(operator::applyAsDouble)
-                    .orElse(null);
+            return args -> env -> {
+                if (args.isEmpty()) {
+                    return identity;
+                } else if (args.size() == 1) {
+                    return unaryOperator.applyAsDouble(Cascade.convertTo(args.get(0).evaluate(env), Double.class));
+                } else {
+                    return args.stream()
+                            .map(arg -> Cascade.convertTo(arg.evaluate(env), Double.class))
+                            .filter(Objects::nonNull)
+                            .reduce(operator::applyAsDouble).orElse(null);
+                }
+            };
         }