Updated build.gradle

Also added some commenting documentation

Author: ZeroMemes

build.gradle

@@ -9,22 +9,26 @@ buildscript {
             name = 'SpongePowered'
             url = 'http://repo.spongepowered.org/maven'
         }
+        maven {
+            url 'https://plugins.gradle.org/m2/'
+        }
     }
 
     dependencies {
         classpath 'net.minecraftforge.gradle:ForgeGradle:2.2-SNAPSHOT'
         classpath 'org.spongepowered:mixingradle:0.4-SNAPSHOT'
+        classpath 'com.github.jengelman.gradle.plugins:shadow:2.0.0'
     }
 }
 
 apply plugin: 'java'
 apply plugin: 'net.minecraftforge.gradle.tweaker-client'
 apply plugin: 'org.spongepowered.mixin'
+apply plugin: 'com.github.johnrengelman.shadow'
 
 version '1.0'
 group 'me.zero.api'
 
-sourceCompatibility = targetCompatibility = '1.8'
 compileJava {
     sourceCompatibility = targetCompatibility = '1.8'
 }

src/main/java/me/zero/client/api/value/type/resolve/DefaultResolvers.java

@@ -63,9 +63,11 @@ private DefaultResolvers () {}
     public static final TypeResolver<MultiType> MULTI = (parent, field) -> {
         Label label = field.getAnnotation(Label.class);
         MultiValue multi = field.getAnnotation(MultiValue.class);
+
         MultiType type = new MultiType(label.name(), label.id(), label.description(), parent, field, multi.value());
         if (type.getValue() == null)
             type.setValue(multi.value()[0]);
+
         return type;
     };
 
@@ -77,9 +79,8 @@ private DefaultResolvers () {}
         Label label = field.getAnnotation(Label.class);
         NumberValue num = field.getAnnotation(NumberValue.class);
 
-        NumberType type = null;
-
         // This is an absolute mess, still finding some other solution
+        NumberType type = null;
         if (field.getType() == Byte.class || field.getType() == Byte.TYPE) {
             type = new NumberType<Byte>(label.name(), label.id(), label.description(), parent, field, (byte) num.min(), (byte) num.max()) { };
         } else if (field.getType() == Short.class || field.getType() == Short.TYPE) {

src/main/java/me/zero/client/load/ClientTweaker.java

@@ -53,7 +53,11 @@ public void acceptOptions(List<String> args, File gameDir, File assetsDir, Strin
     @Override
     public void injectIntoClassLoader(LaunchClassLoader classLoader) {
         Logger.instance.log(Level.INFO, "Injecting into ClassLoader");
+
+        // Initialize the Mixin Bootstrap
         MixinBootstrap.init();
+
+        // Load the ClientAPI and Wrapper mixins
         Mixins.addConfiguration("mixins.capi.json");
         Mixins.addConfiguration("mixins.wrapper.capi.json");
 
@@ -62,6 +66,7 @@ public void injectIntoClassLoader(LaunchClassLoader classLoader) {
         if (this.getClass().getResourceAsStream("/" + mixin) != null)
             Mixins.addConfiguration(mixin);
 
+        // Ensure that the mixins are only run on client side
         MixinEnvironment.getDefaultEnvironment().setSide(MixinEnvironment.Side.CLIENT);
     }
 

src/main/java/me/zero/client/load/mixin/MixinMinecraft.java

@@ -73,19 +73,20 @@ public void onLoop(CallbackInfo ci) {
 
     @Inject(method = "init", at = @At("RETURN"))
     public void init(CallbackInfo ci) {
+        // Try and find the "client.json" config
         InputStream stream = this.getClass().getResourceAsStream("/client.json");
 
         if (stream == null)
             throw new ClientInitException("Unable to locate the Client.json");
 
-        BufferedReader reader = new BufferedReader(new InputStreamReader(stream));
-        ClientInfo clientInfo = new GsonBuilder().setPrettyPrinting().create().fromJson(reader, ClientInfo.class);
+        // Construct a ClientInfo object from the client json using GSON
+        ClientInfo clientInfo = new GsonBuilder().setPrettyPrinting().create().fromJson(new BufferedReader(new InputStreamReader(stream)), ClientInfo.class);
 
         if (clientInfo == null)
             throw new ClientInitException("Unable to create ClientInfo from Client.json");
 
+        // Attempt to instantiate the specified class from the ClientInfo
         Client client;
-
         try {
             Class<?> clientClass = Class.forName(clientInfo.getMain());
             if (clientClass != null && clientClass.getSuperclass().equals(Client.class)) {
@@ -101,10 +102,15 @@ public void init(CallbackInfo ci) {
             throw new ClientInitException("Unable to find client class");
         }
 
+        // Init GLUtils
         GlUtils.init();
-        client.setInfo(clientInfo);
+
         ClientHandler handler = new ClientHandler();
+
+        // Init client
+        client.setInfo(clientInfo);
         client.onInit(handler);
+
         ClientAPI.EVENT_BUS.subscribe(handler);
     }
 
@@ -132,6 +138,7 @@ public GuiScreen displayGuiScreen(GuiScreen screen) {
 
     @Inject(method = "loadWorld(Lnet/minecraft/client/multiplayer/WorldClient;Ljava/lang/String;)V", at = @At("HEAD"))
     public void loadWorld(@Nullable WorldClient worldClientIn, String loadingMessage, CallbackInfo ci) {
+        // If the world is null, then it must be unloading
         if (worldClientIn != null)
             ClientAPI.EVENT_BUS.post(new WorldEvent.Load(worldClientIn));
         else
@@ -158,6 +165,7 @@ public void setSession(Session session) {
 
     @Override
     public void clickMouse(ClickEvent.MouseButton button) {
+        // IF statements are required because Mixin doesn't support SWITCH
         if (button == LEFT)
             clickMouse();
         if (button == RIGHT)