From f41efdec6018948c0eff68719582d25af0aba94e Mon Sep 17 00:00:00 2001
From: "Kristian S. Stangeland" <kr_stang@hotmail.com>
Date: Sat, 14 Dec 2013 17:35:29 +0100
Subject: [PATCH] Make it possible to modify the player list in the ping
 packet.

---
 .../com/comphenix/protocol/wrappers/BukkitConverters.java     | 4 ++--
 .../com/comphenix/protocol/wrappers/WrappedDataWatcher.java   | 3 ++-
 .../com/comphenix/protocol/wrappers/WrappedServerPing.java    | 2 +-
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/BukkitConverters.java b/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/BukkitConverters.java
index 9b6fbee1..61b03ae2 100644
--- a/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/BukkitConverters.java
+++ b/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/BukkitConverters.java
@@ -256,7 +256,7 @@ public class BukkitConverters {
 	 * @return An equivalent converter.
 	 */
 	public static <T> EquivalentConverter<Iterable<? extends T>> getArrayConverter(
-			final Class<?> genericItemType, final EquivalentConverter<T> itemConverter) {
+	  final Class<?> genericItemType, final EquivalentConverter<T> itemConverter) {
 		// Convert to and from the wrapper
 		return new IgnoreNullConverter<Iterable<? extends T>>() {
 				@Override
@@ -279,7 +279,7 @@ public class BukkitConverters {
 				@Override
 				protected Object getGenericValue(Class<?> genericType, Iterable<? extends T> specific) {
 					List<T> list = Lists.newArrayList(specific);
-					Object[] output = (Object[]) Array.newInstance(genericType, list.size());
+					Object[] output = (Object[]) Array.newInstance(genericItemType, list.size());
 					
 					// Convert each object
 					for (int i = 0; i < output.length; i++) {
diff --git a/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/WrappedDataWatcher.java b/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/WrappedDataWatcher.java
index bf66f630..8f992320 100644
--- a/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/WrappedDataWatcher.java
+++ b/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/WrappedDataWatcher.java
@@ -568,7 +568,7 @@ public class WrappedDataWatcher extends AbstractWrapper implements Iterable<Wrap
 				VALUE_MAP_ACCESSOR = Accessors.getFieldAccessor(lookup, true);
 			}
 		}
-		// Spigot workaround
+		// Spigot workaround (not necessary
 		initializeSpigot(fuzzy);
 		
 		// Initialize static type type
@@ -588,6 +588,7 @@ public class WrappedDataWatcher extends AbstractWrapper implements Iterable<Wrap
 		initializeMethods(fuzzy);
 	}
 	
+	// TODO: Remove, as this was fixed in build #1189 of Spigot
 	private static void initializeSpigot(FuzzyReflection fuzzy) {
 		// See if the workaround is needed
 		if (TYPE_MAP_ACCESSOR != null && VALUE_MAP_ACCESSOR != null)
diff --git a/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/WrappedServerPing.java b/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/WrappedServerPing.java
index f3d5460d..02ab8348 100644
--- a/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/WrappedServerPing.java
+++ b/ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/WrappedServerPing.java
@@ -46,7 +46,7 @@ public class WrappedServerPing extends AbstractWrapper {
 	
 	// For converting to the underlying array
 	private static EquivalentConverter<Iterable<? extends WrappedGameProfile>> PROFILE_CONVERT = 
-		BukkitConverters.getArrayConverter(GameProfile[].class, BukkitConverters.getWrappedGameProfileConverter());
+		BukkitConverters.getArrayConverter(GameProfile.class, BukkitConverters.getWrappedGameProfileConverter());
 	
 	// Server ping player sample fields
 	private static Class<?> PLAYERS_CLASS = MinecraftReflection.getServerPingPlayerSampleClass();