From b105298f2e3bb667721e3073b09029072d8d5e76 Mon Sep 17 00:00:00 2001 From: md_5 Date: Mon, 13 Jul 2015 19:00:33 +1000 Subject: Simplify getPlayerExact / try for exact match first in getPlayer. --- src/main/java/org/bukkit/craftbukkit/CraftServer.java | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) (limited to 'src') diff --git a/src/main/java/org/bukkit/craftbukkit/CraftServer.java b/src/main/java/org/bukkit/craftbukkit/CraftServer.java index 88490873..f5722c6c 100644 --- a/src/main/java/org/bukkit/craftbukkit/CraftServer.java +++ b/src/main/java/org/bukkit/craftbukkit/CraftServer.java @@ -386,7 +386,12 @@ public final class CraftServer implements Server { public Player getPlayer(final String name) { Validate.notNull(name, "Name cannot be null"); - Player found = null; + Player found = getPlayerExact(name); + // Try for an exact match first. + if (found != null) { + return found; + } + String lowerName = name.toLowerCase(); int delta = Integer.MAX_VALUE; for (Player player : getOnlinePlayers()) { @@ -407,15 +412,8 @@ public final class CraftServer implements Server { public Player getPlayerExact(String name) { Validate.notNull(name, "Name cannot be null"); - String lname = name.toLowerCase(); - - for (Player player : getOnlinePlayers()) { - if (player.getName().equalsIgnoreCase(lname)) { - return player; - } - } - - return null; + EntityPlayer player = playerList.getPlayer(name); + return (player != null) ? player.getBukkitEntity() : null; } @Override -- cgit v1.2.3