Format pom.xml and plugin.yml, completely remove updater

Author: dmulloy2

ProtocolLib/pom.xml

@@ -4,8 +4,11 @@
   <groupId>com.comphenix.protocol</groupId>
   <artifactId>ProtocolLib</artifactId>
   <version>3.6.0-SNAPSHOT</version>
-  <packaging>jar</packaging>
+  <name>ProtocolLib</name>
+
   <description>Provides read/write access to the Minecraft protocol.</description>
+  <url>http://www.spigotmc.org/resources/protocollib.1997/</url>
+  <packaging>jar</packaging>
 
   <properties>
     <project.build.sourceEncoding>cp1252</project.build.sourceEncoding>
@@ -14,15 +17,10 @@
 
   <distributionManagement>
     <repository>
-      <id>comphenix-releases</id>
-      <name>Comphenix Maven Releases</name>
-      <url>http://repo.comphenix.net/content/repositories/releases/</url>
+      <id>shadowvolt-repo</id>
+      <name>Shadowvolt Maven Repository</name>
+      <url>http://ci.shadowvolt.com/plugin/repository/everything/</url>
     </repository>
-    <snapshotRepository>
-      <id>comphenix-snapshots</id>
-      <name>Comphenix Maven Snapshots</name>
-      <url>http://repo.comphenix.net/content/repositories/snapshots/</url>
-    </snapshotRepository>
   </distributionManagement>
 
   <repositories>
@@ -32,7 +30,6 @@
     </repository>
     <repository>
       <id>comphenix-releases</id>
-      <name>Comphenix Maven Releases</name>
       <url>http://repo.comphenix.net/content/repositories/releases/</url>
     </repository>
   </repositories>
@@ -94,26 +91,6 @@
         </configuration>
       </plugin>
 
-      <!--
-      <plugin>
-        <groupId>org.apache.maven.plugins</groupId>
-        <artifactId>maven-failsafe-plugin</artifactId>
-        <version>2.12.4</version>
-        <configuration>
-          <workingDirectory>${basedir}/target/server/</workingDirectory>
-          <argLine>-Xmx1024m -Xms1024M -Dnojline=true</argLine>
-        </configuration>
-        <executions>
-          <execution>
-            <goals>
-              <goal>integration-test</goal>
-              <goal>verify</goal>
-            </goals>
-          </execution>
-        </executions>
-      </plugin>
-      -->
-
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-jar-plugin</artifactId>
@@ -202,9 +179,6 @@
     </profile>
   </profiles>
 
-  <url>http://dev.bukkit.org/server-mods/protocollib/</url>
-  <name>ProtocolLib</name>
-
   <scm>
     <connection>scm:git:git://github.com/aadnk/ProtocolLib.git</connection>
     <developerConnection>scm:git:[email protected]:aadnk/ProtocolLib.git</developerConnection>

ProtocolLib/src/main/java/com/comphenix/protocol/CommandProtocol.java

@@ -26,8 +26,6 @@
 
 import com.comphenix.protocol.error.ErrorReporter;
 import com.comphenix.protocol.events.PacketListener;
-import com.comphenix.protocol.metrics.Updater;
-import com.comphenix.protocol.metrics.Updater.UpdateType;
 import com.comphenix.protocol.timing.TimedListenerManager;
 import com.comphenix.protocol.timing.TimingReportGenerator;
 
@@ -43,13 +41,11 @@ class CommandProtocol extends CommandBase {
 	public static final String NAME = "protocol";
 
 	private Plugin plugin;
-	private Updater updater;
 	private ProtocolConfig config;
 
-	public CommandProtocol(ErrorReporter reporter, Plugin plugin, Updater updater, ProtocolConfig config) {
+	public CommandProtocol(ErrorReporter reporter, Plugin plugin, ProtocolConfig config) {
 		super(reporter, CommandBase.PERMISSION_ADMIN, NAME, 1);
 		this.plugin = plugin;
-		this.updater = updater;
 		this.config = config;
 	}
 
@@ -60,10 +56,6 @@ protected boolean handleCommand(CommandSender sender, String[] args) {
 		// Only return TRUE if we executed the correct command
 		if (subCommand.equalsIgnoreCase("config") || subCommand.equalsIgnoreCase("reload"))
 			reloadConfiguration(sender);
-		else if (subCommand.equalsIgnoreCase("check"))
-			checkVersion(sender);
-		// else if (subCommand.equalsIgnoreCase("update"))
-		//	updateVersion(sender);
 		else if (subCommand.equalsIgnoreCase("timings"))
 			toggleTimings(sender, args);
 		else if (subCommand.equalsIgnoreCase("listeners"))
@@ -73,14 +65,6 @@ else if (subCommand.equalsIgnoreCase("listeners"))
 		return true;
 	}
 
-	public void checkVersion(final CommandSender sender) {
-		performUpdate(sender, UpdateType.NO_DOWNLOAD);
-	}
-
-	public void updateVersion(final CommandSender sender) {
-		performUpdate(sender, UpdateType.DEFAULT);
-	}
-
 	// Display every listener on the server
 	private void printListeners(final CommandSender sender, String[] args) {
 		ProtocolManager manager = ProtocolLibrary.getProtocolManager();
@@ -96,26 +80,6 @@ private void printListeners(final CommandSender sender, String[] args) {
 		}
 	}
 
-	private void performUpdate(final CommandSender sender, UpdateType type) {
-		if (updater.isChecking()) {
-			sender.sendMessage(ChatColor.RED + "Already checking for an update.");
-			return;
-		}
-
-		// Perform on an async thread
-		Runnable notify = new Runnable() {
-			@Override
-			public void run() {
-				sender.sendMessage(ChatColor.YELLOW + "[ProtocolLib] " + updater.getResult());
-
-				updater.removeListener(this);
-				updateFinished();
-			}
-		};
-		updater.start(type);
-		updater.addListener(notify);
-	}
-
 	private void toggleTimings(CommandSender sender, String[] args) {
 		TimedListenerManager manager = TimedListenerManager.getInstance();
 		boolean state = !manager.isTiming(); // toggle