diff options
author | Arne H Juul <arnej@yahoo-inc.com> | 2016-11-08 12:08:09 +0100 |
---|---|---|
committer | Arne H Juul <arnej@yahoo-inc.com> | 2016-11-08 12:08:09 +0100 |
commit | fa865f55ba4639b36559a263c73d259dfb1ab9c3 (patch) | |
tree | 86317a79c2741a641b0814b64efe6effa9da7ba6 /simplemetrics/src/main/java/com/yahoo/metrics | |
parent | bc208603a01b8392360408c5847d0d55ad7d45c1 (diff) |
simplify
Diffstat (limited to 'simplemetrics/src/main/java/com/yahoo/metrics')
-rw-r--r-- | simplemetrics/src/main/java/com/yahoo/metrics/simple/PointBuilder.java | 11 |
1 files changed, 4 insertions, 7 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 a4fb0f370be..5e6edaf8b50 100644 --- a/simplemetrics/src/main/java/com/yahoo/metrics/simple/PointBuilder.java +++ b/simplemetrics/src/main/java/com/yahoo/metrics/simple/PointBuilder.java @@ -17,8 +17,6 @@ import com.google.common.annotations.Beta; public final class PointBuilder { private ArrayList<String> dimensions; private ArrayList<Value> location; - private static final String[] dimensionsTypeArgument = new String[0]; - private static final Value[] locationTypeArgument = new Value[0]; public enum Discriminator { LONG, DOUBLE, STRING; @@ -101,11 +99,10 @@ public final class PointBuilder { * @return a Point instance reflecting this builder */ public Point build() { - Point p; - if (dimensions.size() == 0) { - p = Point.emptyPoint(); - } else { - p = new Point(dimensions.toArray(dimensionsTypeArgument), location.toArray(locationTypeArgument)); + Point p = Point.emptyPoint(); + int size = dimensions.size(); + if (size != 0) { + p = new Point(dimensions.toArray(new String[size]), location.toArray(new Value[size])); } // deny builder re-use dimensions = null; |