A few build improvements

.travis.yml

@@ -3,7 +3,7 @@ jdk:
   - oraclejdk8
   - oraclejdk7
   - openjdk6
-script: mvn clean install -U
+script: mvn clean test -U
 before_install: cd ProtocolLib
 install: true
 notifications:

ProtocolLib/pom.xml

@@ -90,7 +90,7 @@
         </configuration>
       </plugin>
 
-      <plugin>
+      <!--<plugin>
         <artifactId>maven-javadoc-plugin</artifactId>
         <version>2.7</version>
         <executions>
@@ -104,12 +104,12 @@
           </execution>
 
         </executions>
-      </plugin>
+      </plugin>-->
 
     </plugins>
   </build>
 
-  <reporting>
+  <!--<reporting>
     <plugins>
       <plugin>
         <groupId>org.codehaus.mojo</groupId>
@@ -133,7 +133,7 @@
       </plugin>
     </plugins>
 
-  </reporting>
+  </reporting>-->
 
   <profiles>
     <profile>

ProtocolLib/src/main/java/com/comphenix/protocol/compat/netty/independent/IndependentNetty.java

@@ -45,14 +45,11 @@ public class IndependentNetty implements NettyCompat {
 
 	@Override
 	public WrappedByteBuf createPacketBuffer() {
-		return getPacketDataSerializer(UnpooledByteBufAllocator.DEFAULT.buffer());
-	}
-
-	private WrappedByteBuf getPacketDataSerializer(ByteBuf buffer) {
+		ByteBuf buffer = UnpooledByteBufAllocator.DEFAULT.buffer();
 		Class<?> packetSerializer = MinecraftReflection.getPacketDataSerializerClass();
 
 		try {
-			return new NettyByteBuf((ByteBuf) packetSerializer.getConstructor(MinecraftReflection.getByteBufClass()).newInstance(buffer));
+			return new NettyByteBuf((ByteBuf) packetSerializer.getConstructor(ByteBuf.class).newInstance(buffer));
 		} catch (Exception e) {
 			throw new RuntimeException("Cannot construct packet serializer.", e);
 		}

ProtocolLib/src/test/java/com/comphenix/protocol/events/PacketContainerTest.java

@@ -371,9 +371,11 @@ public class PacketContainerTest {
 
 			assertEquals(PacketType.Play.Client.CHAT, copy.getType());
 			assertEquals("Test", copy.getStrings().read(0));
-		} catch (NullPointerException ex) {
-			// This occurs intermittently with Java 6, just log it and move on
-			System.err.println("Encountered a NullPointerException with serialization");
+		} catch (Throwable ex) {
+			// This occurs intermittently on Java 6, for the time being just log it and move on
+			// TODO: Possibly find a solution to this
+			System.err.println("Failed to serialize packet:");
+			ex.printStackTrace();
 		}
 	}
 

modules/v1_7_R4/src/main/java/com/comphenix/protocol/compat/netty/shaded/ShadedNetty.java

@@ -45,14 +45,11 @@ public class ShadedNetty implements NettyCompat {
 
 	@Override
 	public WrappedByteBuf createPacketBuffer() {
-		return getPacketDataSerializer(UnpooledByteBufAllocator.DEFAULT.buffer());
-	}
-
-	private WrappedByteBuf getPacketDataSerializer(ByteBuf buffer) {
+		ByteBuf buffer = UnpooledByteBufAllocator.DEFAULT.buffer();
 		Class<?> packetSerializer = MinecraftReflection.getPacketDataSerializerClass();
 
 		try {
-			return new ShadedByteBuf((ByteBuf) packetSerializer.getConstructor(MinecraftReflection.getByteBufClass()).newInstance(buffer));
+			return new ShadedByteBuf((ByteBuf) packetSerializer.getConstructor(ByteBuf.class).newInstance(buffer));
 		} catch (Exception e) {
 			throw new RuntimeException("Cannot construct packet serializer.", e);
 		}

pom.xml

@@ -75,7 +75,7 @@
     </plugins>
   </build>
 
-  <reporting>
+  <!--<reporting>
     <plugins>
       <plugin>
         <artifactId>maven-javadoc-plugin</artifactId>
@@ -89,5 +89,5 @@
         <version>2.4</version>
       </plugin>
     </plugins>
-  </reporting>
+  </reporting>-->
 </project>