Updates to FlxRandom

CHANGELOG.md

@@ -61,7 +61,10 @@
 * FlxGamepad:
  * added a connected flag
  * fixed a bug that would prevent gamepad buttons from being updated
-* FlxRandom: exposed internalSeed as a read-only property
+* FlxRandom:
+ * exposed internalSeed as a read-only property
+ * removed intRanged() and floatRanged(), int() and float() now provide optional ranges
+* FlxArrayUtil: removed randomness-related functions, please use FlxRandom instead
 
 3.3.4
 ------------------------------

flixel/math/FlxRandom.hx

@@ -108,32 +108,16 @@ class FlxRandom
 		return globalSeed = Std.int(Math.random() * MODULUS);
 	}
 
-	/**
-	 * Returns a pseudorandom number between 0 and 2,147,483,647, inclusive.
-	 */
-	public static inline function int():Int
-	{
-		return generate();
-	}
-
-	/**
-	 * Returns a pseudorandom number between 0 and 1, inclusive.
-	 */
-	public static inline function float():Float
-	{
-		return generate() / MODULUS;
-	}
-
 	/**
 	 * Returns a pseudorandom integer between Min and Max, inclusive.
 	 * Will not return a number in the Excludes array, if provided.
 	 * Please note that large Excludes arrays can slow calculations.
 	 * 
 	 * @param   Min        The minimum value that should be returned. 0 by default.
 	 * @param   Max        The maximum value that should be returned. 2,147,483,647 by default.
-	 * @param   Excludes   An optional array of values that should not be returned.
+	 * @param   Excludes   An optional array of values that should not be returned. Optional.
 	 */
-	public static function intRanged(Min:Int = 0, Max:Int = MODULUS, ?Excludes:Array<Int>):Int
+	public static function int(Min:Int = 0, Max:Int = MODULUS, ?Excludes:Array<Int>):Int
 	{
 		if (Min == Max)
 		{
@@ -173,10 +157,10 @@ class FlxRandom
 	 * Please note that large Excludes arrays can slow calculations.
 	 * 
 	 * @param   Min        The minimum value that should be returned. 0 by default.
-	 * @param   Max        The maximum value that should be returned. 33,554,429 by default.
-	 * @param   Excludes   An optional array of values that should not be returned.
+	 * @param   Max        The maximum value that should be returned. 1 by default.
+	 * @param   Excludes   An optional array of values that should not be returned. Optional.
 	 */
-	public static function floatRanged(Min:Float = 0, Max:Float = 1, ?Excludes:Array<Float>):Float
+	public static function float(Min:Float = 0, Max:Float = 1, ?Excludes:Array<Float>):Float
 	{
 		var result:Float = 0;
 
@@ -444,4 +428,4 @@ class FlxRandom
 	{
 		return internalSeed = ((internalSeed * MULTIPLIER) % MODULUS) & MODULUS;
 	}
-}
+}

flixel/util/FlxArrayUtil.hx

@@ -1,5 +1,4 @@
 package flixel.util;
-import flixel.math.FlxRandom;
 
 /**
  * A set of functions for array manipulation.
@@ -32,35 +31,6 @@ class FlxArrayUtil
 		}
 	}
 
-	/**
-	 * Deprecated; please use FlxRandom.shuffleArray() instead.
-	 * Shuffles the entries in an array into a new random order.
-	 * 
-	 * @param	Objects			An array to shuffle.
-	 * @param	HowManyTimes	How many swaps to perform during the shuffle operation.  A good rule of thumb is 2-4 times the number of objects in the list.
-	 * @return	The newly shuffled array.
-	 */
-	@:generic
-	public static inline function shuffle<T>(Objects:Array<T>, HowManyTimes:Int):Array<T>
-	{
-		return FlxRandom.shuffleArray(Objects, HowManyTimes);
-	}
-
-	/**
-	 * Deprecated; please use FlxRandom.getObject() instead.
-	 * Fetch a random entry from the given array from StartIndex to EndIndex.
-	 * 
-	 * @param	Objects			An array from which to select a random entry.
-	 * @param	StartIndex		Optional index from which to restrict selection. Default value is 0, or the beginning of the array.
-	 * @param	EndIndex		Optional index at which to restrict selection. Ignored if 0, which is the default value.
-	 * @return	The random object that was selected.
-	 */
-	@:generic
-	public static inline function getRandom<T>(Objects:Array<T>, StartIndex:Int = 0, EndIndex:Int = 0):T
-	{
-		return FlxRandom.getObject(Objects, StartIndex, EndIndex);
-	}
-
 	/**
 	 * Safely removes an element from an array by swapping it with the last element and calling pop()
 	 * (won't do anything if the element is not in the array). This is a lot faster than regular splice(),