summaryrefslogtreecommitdiffstats
path: root/simplemetrics
diff options
context:
space:
mode:
authorArne H Juul <arnej@yahoo-inc.com>2016-11-08 12:00:29 +0100
committerArne H Juul <arnej@yahoo-inc.com>2016-11-08 12:00:29 +0100
commitbc208603a01b8392360408c5847d0d55ad7d45c1 (patch)
treeb4c77f9994223ff63de5506483ba6400b5e9ac11 /simplemetrics
parentbc148a328e3ac237c02ae6fb6f890c40a7419103 (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.java6
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));
}