diff --git a/paper-api/src/main/java/org/bukkit/Location.java b/paper-api/src/main/java/org/bukkit/Location.java index e7af316d60..5c5b3ba507 100644 --- a/paper-api/src/main/java/org/bukkit/Location.java +++ b/paper-api/src/main/java/org/bukkit/Location.java @@ -6,7 +6,6 @@ import java.util.Map; import org.bukkit.block.Block; import org.bukkit.configuration.serialization.ConfigurationSerializable; import org.bukkit.util.NumberConversions; -import static org.bukkit.util.NumberConversions.checkFinite; import org.bukkit.util.Vector; /** @@ -555,6 +554,19 @@ public class Location implements Cloneable, ConfigurationSerializable { } } + /** + * Check if each component of this Location is finite. + * + * @throws IllegalArgumentException if any component is not finite + */ + public void checkFinite() throws IllegalArgumentException { + NumberConversions.checkFinite(x, "x not finite"); + NumberConversions.checkFinite(y, "y not finite"); + NumberConversions.checkFinite(z, "z not finite"); + NumberConversions.checkFinite(pitch, "pitch not finite"); + NumberConversions.checkFinite(yaw, "yaw not finite"); + } + /** * Safely converts a double (location coordinate) to an int (block * coordinate) @@ -566,22 +578,22 @@ public class Location implements Cloneable, ConfigurationSerializable { return NumberConversions.floor(loc); } - @Utility - public Map serialize() { - Map data = new HashMap(); - data.put("world", this.world.getName()); + @Utility + public Map serialize() { + Map data = new HashMap(); + data.put("world", this.world.getName()); - data.put("x", this.x); - data.put("y", this.y); - data.put("z", this.z); + data.put("x", this.x); + data.put("y", this.y); + data.put("z", this.z); - data.put("yaw", this.yaw); - data.put("pitch", this.pitch); + data.put("yaw", this.yaw); + data.put("pitch", this.pitch); - return data; - } - - /** + return data; + } + + /** * Required method for deserialization * * @param args map to deserialize @@ -589,12 +601,12 @@ public class Location implements Cloneable, ConfigurationSerializable { * @throws IllegalArgumentException if the world don't exists * @see ConfigurationSerializable */ - public static Location deserialize(Map args) { - World world = Bukkit.getWorld((String) args.get("world")); - if (world == null) { - throw new IllegalArgumentException("unknown world"); - } + public static Location deserialize(Map args) { + World world = Bukkit.getWorld((String) args.get("world")); + if (world == null) { + throw new IllegalArgumentException("unknown world"); + } - return new Location(world, NumberConversions.toDouble(args.get("x")), NumberConversions.toDouble(args.get("y")), NumberConversions.toDouble(args.get("z")), NumberConversions.toFloat(args.get("yaw")), NumberConversions.toFloat(args.get("pitch"))); - } + return new Location(world, NumberConversions.toDouble(args.get("x")), NumberConversions.toDouble(args.get("y")), NumberConversions.toDouble(args.get("z")), NumberConversions.toFloat(args.get("yaw")), NumberConversions.toFloat(args.get("pitch"))); + } } diff --git a/paper-api/src/main/java/org/bukkit/util/Vector.java b/paper-api/src/main/java/org/bukkit/util/Vector.java index cebd3ccb7f..068361f58c 100644 --- a/paper-api/src/main/java/org/bukkit/util/Vector.java +++ b/paper-api/src/main/java/org/bukkit/util/Vector.java @@ -7,7 +7,6 @@ import org.bukkit.Location; import org.bukkit.World; import org.bukkit.configuration.serialization.ConfigurationSerializable; import org.bukkit.configuration.serialization.SerializableAs; -import static org.bukkit.util.NumberConversions.checkFinite; /** * Represents a mutable vector. Because the components of Vectors are mutable, @@ -616,6 +615,17 @@ public class Vector implements Cloneable, ConfigurationSerializable { return new BlockVector(x, y, z); } + /** + * Check if each component of this Vector is finite. + * + * @throws IllegalArgumentException if any component is not finite + */ + public void checkFinite() throws IllegalArgumentException { + NumberConversions.checkFinite(x, "x not finite"); + NumberConversions.checkFinite(y, "y not finite"); + NumberConversions.checkFinite(z, "z not finite"); + } + /** * Get the threshold used for equals(). *