diff options
author | Arne H Juul <arnej@yahoo-inc.com> | 2016-11-08 12:00:29 +0100 |
---|---|---|
committer | Arne H Juul <arnej@yahoo-inc.com> | 2016-11-08 12:00:29 +0100 |
commit | bc208603a01b8392360408c5847d0d55ad7d45c1 (patch) | |
tree | b4c77f9994223ff63de5506483ba6400b5e9ac11 /simplemetrics | |
parent | bc148a328e3ac237c02ae6fb6f890c40a7419103 (diff) |
use emptyPoint instead of null where applicable.
* also, expected sizes should reflect that we probably
want to add a dimension or two when creating a PointBuilder.
Diffstat (limited to 'simplemetrics')
-rw-r--r-- | simplemetrics/src/main/java/com/yahoo/metrics/simple/PointBuilder.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/simplemetrics/src/main/java/com/yahoo/metrics/simple/PointBuilder.java b/simplemetrics/src/main/java/com/yahoo/metrics/simple/PointBuilder.java index 7f6b797c601..a4fb0f370be 100644 --- a/simplemetrics/src/main/java/com/yahoo/metrics/simple/PointBuilder.java +++ b/simplemetrics/src/main/java/com/yahoo/metrics/simple/PointBuilder.java @@ -33,8 +33,8 @@ public final class PointBuilder { location = new ArrayList<>(); if (p != null) { int size = p.dimensionality(); - dimensions = new ArrayList<>(size); - location = new ArrayList<>(size); + dimensions = new ArrayList<>(size+2); + location = new ArrayList<>(size+2); for (String dimensionName : p.getDimensions()) { dimensions.add(dimensionName); } @@ -103,7 +103,7 @@ public final class PointBuilder { public Point build() { Point p; if (dimensions.size() == 0) { - p = null; + p = Point.emptyPoint(); } else { p = new Point(dimensions.toArray(dimensionsTypeArgument), location.toArray(locationTypeArgument)); } |