diff options
author | Patrick Wendell <pwendell@gmail.com> | 2013-02-23 15:40:41 -0800 |
---|---|---|
committer | Patrick Wendell <pwendell@gmail.com> | 2013-02-23 15:40:41 -0800 |
commit | 931f439be9048fde244b81e3eae4ad5ff9de4adf (patch) | |
tree | 8e0b62a7126b9fe675f8bdc5adeee6cd9883adaf /core/src | |
parent | f51b0f93f20e23804b2f95edfb1d86b9c9cee493 (diff) | |
download | spark-931f439be9048fde244b81e3eae4ad5ff9de4adf.tar.gz spark-931f439be9048fde244b81e3eae4ad5ff9de4adf.tar.bz2 spark-931f439be9048fde244b81e3eae4ad5ff9de4adf.zip |
Responding to code review
Diffstat (limited to 'core/src')
-rw-r--r-- | core/src/main/scala/spark/util/Vector.scala | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/core/src/main/scala/spark/util/Vector.scala b/core/src/main/scala/spark/util/Vector.scala index d03cebeea9..835822edb2 100644 --- a/core/src/main/scala/spark/util/Vector.scala +++ b/core/src/main/scala/spark/util/Vector.scala @@ -10,14 +10,16 @@ class Vector(val elements: Array[Double]) extends Serializable { throw new IllegalArgumentException("Vectors of different length") return Vector(length, i => this(i) + other(i)) } - def add(other: Vector) = +(other) + + def add(other: Vector) = this + other def - (other: Vector): Vector = { if (length != other.length) throw new IllegalArgumentException("Vectors of different length") return Vector(length, i => this(i) - other(i)) } - def subtract(other: Vector) = -(other) + + def subtract(other: Vector) = this - other def dot(other: Vector): Double = { if (length != other.length) @@ -62,13 +64,16 @@ class Vector(val elements: Array[Double]) extends Serializable { } this } - def addInPlace(other: Vector) = +=(other) + + def addInPlace(other: Vector) = this +=other def * (scale: Double): Vector = Vector(length, i => this(i) * scale) - def multiply (d: Double) = *(d) + + def multiply (d: Double) = this * d def / (d: Double): Vector = this * (1 / d) - def divide (d: Double) = /(d) + + def divide (d: Double) = this / d def unary_- = this * -1 |