Refactor, remove unused class, add API docs

core/src/main/java/com/seibel/distanthorizons/core/api/internal/ClientApi.java

@@ -272,7 +272,7 @@ public class ClientApi
 		}
 		
 		short commandLength = byteBuf.readShort();
-		if (commandLength > 32)
+		if (commandLength > 32) // TODO 32 should be put into a constant somewhere
 		{
 			LOGGER.error("Server sent command > 32");
 			ClientApi.INSTANCE.serverIsMalformed = true;
@@ -295,7 +295,7 @@ public class ClientApi
 				
 			case "WorldChanged":
 				short worldKeyLength = byteBuf.readShort();
-				if (worldKeyLength > 128)
+				if (worldKeyLength > 128) // TODO 128 should be put into a constant somewhere
 				{
 					LOGGER.error("Server sent worldKey > 128");
 					this.serverIsMalformed = true;

core/src/main/java/com/seibel/distanthorizons/core/wrapperInterfaces/world/IDimensionTypeWrapper.java

@@ -28,11 +28,13 @@ import com.seibel.distanthorizons.coreapi.interfaces.dependencyInjection.IBindab
  */
 public interface IDimensionTypeWrapper extends IDhApiDimensionTypeWrapper, IBindable
 {
-	
+	@Override
 	String getDimensionName();
 	
+	@Override
 	boolean hasCeiling();
-
+	
+	@Override
 	boolean hasSkyLight();
 	
 }