Removed the Vector random methods and moved to their own function as they're swappable between each other

Author: photonstorm

v3/src/math/RandomXY.js

@@ -0,0 +1,13 @@
+var RandomXY = function (vector, scale)
+{
+    if (scale === undefined) { scale = 1; }
+
+    var r = Math.random() * 2 * Math.PI;
+
+    vector.x = Math.cos(r) * scale;
+    vector.y = Math.sin(r) * scale;
+
+    return vector;
+};
+
+module.exports = RandomXY;

v3/src/math/RandomXYZ.js

@@ -0,0 +1,17 @@
+//  Position Vector randomly in a spherical area defined by the given radius
+var RandomXYZ = function (vector, radius)
+{
+    if (radius === undefined) { radius = 1; }
+
+    var r = Math.random() * 2 * Math.PI;
+    var z = (Math.random() * 2) - 1;
+    var zScale = Math.sqrt(1 - z * z) * radius;
+    
+    vector.x = Math.cos(r) * zScale;
+    vector.y = Math.sin(r) * zScale;
+    vector.z = z * radius;
+
+    return vector;
+};
+
+module.exports = RandomXYZ;

v3/src/math/RandomXYZW.js

@@ -0,0 +1,14 @@
+var RandomXYZW = function (vector, scale)
+{
+    if (scale === undefined) { scale = 1; }
+
+    // Not spherical; should fix this for more uniform distribution
+    vector.x = (Math.random() * 2 - 1) * scale;
+    vector.y = (Math.random() * 2 - 1) * scale;
+    vector.z = (Math.random() * 2 - 1) * scale;
+    vector.w = (Math.random() * 2 - 1) * scale;
+
+    return vector;
+};
+
+module.exports = RandomXYZW;

v3/src/math/Vector2.js

@@ -198,18 +198,6 @@ var Vector2 = new Class({
         this.x = 0;
         this.y = 0;
 
-        return this;
-    },
-
-    random: function (scale)
-    {
-        if (scale === undefined) { scale = 1; }
-
-        var r = Math.random() * 2 * Math.PI;
-
-        this.x = Math.cos(r) * scale;
-        this.y = Math.sin(r) * scale;
-
         return this;
     }
 

v3/src/math/Vector3.js

@@ -327,22 +327,6 @@ var Vector3 = new Class({
         return this.project(invProjectionView);
     },
 
-    //  Position Vector randomly in a spherical area defined by the given radius
-    random: function (radius)
-    {
-        if (radius === undefined) { radius = 1; }
-
-        var r = Math.random() * 2 * Math.PI;
-        var z = (Math.random() * 2) - 1;
-        var zScale = Math.sqrt(1 - z * z) * radius;
-        
-        this.x = Math.cos(r) * zScale;
-        this.y = Math.sin(r) * zScale;
-        this.z = z * radius;
-
-        return this;
-    },
-
     reset: function ()
     {
         this.x = 0;

v3/src/math/Vector4.js

@@ -246,19 +246,6 @@ var Vector4 = new Class({
         return this;
     },
 
-    random: function (scale)
-    {
-        if (scale === undefined) { scale = 1; }
-
-        // Not spherical; should fix this for more uniform distribution
-        this.x = (Math.random() * 2 - 1) * scale;
-        this.y = (Math.random() * 2 - 1) * scale;
-        this.z = (Math.random() * 2 - 1) * scale;
-        this.w = (Math.random() * 2 - 1) * scale;
-
-        return this;
-    },
-
     reset: function ()
     {
         this.x = 0;

v3/src/math/index.js

@@ -29,6 +29,9 @@ module.exports = {
     MinSub: require('./MinSub'),
     Percent: require('./Percent'),
     RadToDeg: require('./RadToDeg'),
+    RandomXY: require('./RandomXY'),
+    RandomXYZ: require('./RandomXYZ'),
+    RandomXYZW: require('./RandomXYZW'),
     Rotate: require('./Rotate'),
     RotateAround: require('./RotateAround'),
     RotateAroundDistance: require('./RotateAroundDistance'),