Trying to emulate call lists, in the meantime i made some other optimizations

Author: DeexCoding

README.md

@@ -7,3 +7,6 @@ SaltLib aims to be an alternative of the Sodium mod for Minecraft, but for legac
 
 **Performance:**
   idk try it out
+
+**Requirements:**
+  A gpu (can be integrated) made after 2008

build.gradle

@@ -33,7 +33,7 @@ loom
 	setIntermediaryUrl('https://maven.legacyfabric.net/net/legacyfabric/intermediary/%1$s/intermediary-%1$s-v2.jar');
 
 	// Only needed for versions not available from vanilla launcher by default.
-	customMinecraftManifest.set("https://meta.legacyfabric.net/v2/manifest/${minecraft_version}")
+	// customMinecraftManifest.set("https://meta.legacyfabric.net/v2/manifest/${minecraft_version}")
 
 	// Required by 1.7.x
 	// runs 

gradle.properties

@@ -6,7 +6,7 @@ org.gradle.jvmargs=-Xmx1G
 # Fabric Properties
 	minecraft_version = 1.8.9
 	yarn_mappings = 1.8.9+build.382
-	loader_version = 0.14.7
+	loader_version = 0.14.9
 
 # Legacy Fabric API
 	fabric_version = 1.7.0+1.8.9

settings.gradle

@@ -13,7 +13,5 @@ pluginManagement
 			name = 'Jitpack'
 			url = 'https://jitpack.io'
 		}
-		
-        gradlePluginPortal()
 	}
 }

src/main/java/me/Deex/SaltLib/Mixin/BufferBuilderMixin.java

@@ -11,14 +11,15 @@
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Overwrite;
 import org.spongepowered.asm.mixin.Shadow;
+import org.spongepowered.asm.mixin.gen.Accessor;
 
 import me.Deex.SaltLib.Renderer.MatrixStack;
 import net.minecraft.client.render.BufferBuilder;
 import net.minecraft.client.render.VertexFormat;
 import net.minecraft.client.render.VertexFormatElement;
 
 @Mixin(BufferBuilder.class)
-public class BufferBuilderMixin 
+public abstract class BufferBuilderMixin 
 {
     @Shadow
     private ByteBuffer buffer;
@@ -44,13 +45,21 @@ public class BufferBuilderMixin
     @Shadow
     private double offsetZ;
 
+    @Shadow
+    private boolean building;
+
+    private int method_9757() 
+    {
+        return this.vertexCount * this.format.getVertexSizeInteger();
+    }
+
     /**
      * what is javadoc
      * @author me
      * @reason mod needs functionality
      */
     @Overwrite
-    private void nextElement() //Basically gets the next element in the buffer
+    private void nextElement()
     {
         List<VertexFormatElement> elements = format.getElements();
 
@@ -67,11 +76,50 @@ private void nextElement() //Basically gets the next element in the buffer
         } while (currentElement.getType() == VertexFormatElement.Type.PADDING);
     }
 
-    
+    @Shadow
+    private int drawMode;
+
+    @Shadow
+    private boolean textured;
+
+    @Shadow
+    public abstract void reset();
+
+    @Overwrite
+    public void begin(int drawMode, VertexFormat format) 
+    {
+        if (this.building) 
+        {
+            return; //Don't throw an error idiot
+        }
+
+        this.building = true;
+        this.reset();
+        this.drawMode = drawMode;
+        this.format = format;
+        this.currentElement = format.get(this.currentElementId);
+        this.textured = false;
+        this.buffer.limit(this.buffer.capacity());
+    }
+
+    @Overwrite
+    public void end() 
+    {
+        if (!this.building) 
+        {
+            return; //Have you heard of assertions?
+        }
+        
+        this.building = false;
+        this.buffer.position(0);
+        this.buffer.limit(this.method_9757() * 4);
+    }
+
+
     //The matrix tranforms are not exact functionaltiy, since OpenGL transforms the verticies with the matrix stack once 
     //it is requested to be drawn or glBegin has been called
 
-    @Overwrite
+    /*@Overwrite
     public BufferBuilder vertex(double x, double y, double z) 
     {
         int i = this.vertexCount * this.format.getVertexSize() + this.format.getIndex(this.currentElementId);
@@ -170,6 +218,7 @@ public BufferBuilder texture(double u, double v)
         return (BufferBuilder)(Object)this;
     }
 
+    @Overwrite
     public BufferBuilder texture2(int u, int v) 
     {
         //TODO: Call to texture instead
@@ -215,4 +264,5 @@ public BufferBuilder texture2(int u, int v)
         this.nextElement();
         return (BufferBuilder)(Object)this;
     }
+    /**/
 }

src/main/java/me/Deex/SaltLib/Mixin/BufferRendererMixin.java

@@ -1,20 +1,12 @@
 package me.Deex.SaltLib.Mixin;
 
-import java.nio.ByteBuffer;
-import java.nio.FloatBuffer;
-import java.util.List;
+import me.Deex.SaltLib.Renderer.CustomBufferRenderer;
 
-import com.mojang.blaze3d.platform.GLX;
-import com.mojang.blaze3d.platform.GlStateManager;
-
-import org.lwjgl.opengl.GL11;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Overwrite;
 
 import net.minecraft.client.render.BufferBuilder;
 import net.minecraft.client.render.BufferRenderer;
-import net.minecraft.client.render.VertexFormat;
-import net.minecraft.client.render.VertexFormatElement;
 
 @Mixin(BufferRenderer.class)
 public class BufferRendererMixin 
@@ -27,87 +19,6 @@ public class BufferRendererMixin
     @Overwrite
     public void draw(BufferBuilder builder) 
     {
-        if (builder.getVertexCount() <= 0) 
-        {
-            return;
-        }
-
-        int textureUnitOffset;
-        int j;
-        VertexFormat vertexFormat = builder.getFormat();
-        int stride = vertexFormat.getVertexSize();
-        ByteBuffer byteBuffer = builder.getByteBuffer();
-        List<VertexFormatElement> list = vertexFormat.getElements();
-        block12: for (j = 0; j < list.size(); ++j) 
-        {
-            VertexFormatElement vertexFormatElement = list.get(j);
-            VertexFormatElement.Type type = vertexFormatElement.getType();
-            int vertexTypeID = vertexFormatElement.getFormat().getGlId();
-            textureUnitOffset = vertexFormatElement.getIndex();
-            byteBuffer.position(vertexFormat.getIndex(j));
-            switch (type) 
-            {
-                case POSITION: 
-                {
-                    GL11.glVertexPointer(vertexFormatElement.getCount(), vertexTypeID, stride, byteBuffer);
-                    GL11.glEnableClientState(32884);
-                    continue block12;
-                }
-                case UV: 
-                {
-                    GLX.gl13ClientActiveTexture(GLX.textureUnit + textureUnitOffset);
-                    GL11.glTexCoordPointer(vertexFormatElement.getCount(), vertexTypeID, stride, byteBuffer);
-                    GL11.glEnableClientState(32888);
-                    GLX.gl13ClientActiveTexture(GLX.textureUnit);
-                    continue block12;
-                }
-                case COLOR: 
-                {
-                    GL11.glColorPointer(vertexFormatElement.getCount(), vertexTypeID, stride, byteBuffer);
-                    GL11.glEnableClientState(32886);
-                    continue block12;
-                }
-                case NORMAL: 
-                {
-                    GL11.glNormalPointer(vertexTypeID, stride, byteBuffer);
-                    GL11.glEnableClientState(32885);
-                }
-            }
-        }
-        GL11.glDrawArrays(builder.getDrawMode(), 0, builder.getVertexCount());
-        int m = list.size();
-        block13: for (j = 0; j < m; ++j) 
-        {
-            VertexFormatElement vertexFormatElement2 = list.get(j);
-            VertexFormatElement.Type type2 = vertexFormatElement2.getType();
-            textureUnitOffset = vertexFormatElement2.getIndex();
-            switch (type2) 
-            {
-                case POSITION: 
-                {
-                    GL11.glDisableClientState(32884);
-                    continue block13;
-                }
-                case UV: 
-                {
-                    GLX.gl13ClientActiveTexture(GLX.textureUnit + textureUnitOffset);
-                    GL11.glDisableClientState(32888);
-                    GLX.gl13ClientActiveTexture(GLX.textureUnit);
-                    continue block13;
-                }
-                case COLOR: 
-                {
-                    GL11.glDisableClientState(32886);
-                    GlStateManager.clearColor();
-                    continue block13;
-                }
-                case NORMAL: 
-                {
-                    GL11.glDisableClientState(32885);
-                }
-            }
-        }
-
-        builder.reset();
+        CustomBufferRenderer.DrawAndReset(builder);
     }
 }

src/main/java/me/Deex/SaltLib/Mixin/ChunkBuilderMixin.java

@@ -1,31 +1,20 @@
 package me.Deex.SaltLib.Mixin;
 
-import org.spongepowered.asm.mixin.Final;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Overwrite;
-import org.spongepowered.asm.mixin.Shadow;
 
-import com.mojang.blaze3d.platform.GlStateManager;
 
 import net.minecraft.client.render.BufferBuilder;
-import net.minecraft.client.render.BufferRenderer;
 import net.minecraft.client.render.chunk.ChunkBuilder;
 import net.minecraft.client.world.BuiltChunk;
 
 @Mixin(ChunkBuilder.class)
 public class ChunkBuilderMixin 
 {
-    @Shadow
-    @Final
-    private BufferRenderer field_11040;
-
-    /*@Overwrite
+    @Overwrite
     private void method_10125(BufferBuilder bufferBuilder, int i, BuiltChunk builtChunk)
     {
-        GlStateManager.pushMatrix();
-        builtChunk.method_10169();
-        this.field_11040.draw(bufferBuilder);
-        GlStateManager.popMatrix();
+        //NOTE: To my knowledge, this function doesn't get called when VBOs are turned on.
+        return;
     }
-    /* */
 }

src/main/java/me/Deex/SaltLib/Mixin/ChunkRenderHelperImplMixin.java

@@ -0,0 +1,27 @@
+package me.Deex.SaltLib.Mixin;
+
+import org.spongepowered.asm.mixin.Mixin;
+
+import net.minecraft.client.render.BlockRenderLayer;
+import net.minecraft.client.render.WorldRenderer;
+import net.minecraft.client.render.world.ChunkRenderHelperImpl;
+import net.minecraft.client.util.GlAllocationUtils;
+import net.minecraft.client.world.BuiltChunk;
+import net.minecraft.util.math.BlockPos;
+import net.minecraft.world.World;
+
+@Mixin(ChunkRenderHelperImpl.class)
+public abstract class ChunkRenderHelperImplMixin extends BuiltChunk
+{
+    //TODO: Ask someone smart what to do with the constructor when mixing in with an extends class
+    public ChunkRenderHelperImplMixin(World world, WorldRenderer worldRenderer, BlockPos blockPos, int i) 
+    {
+        super(world, worldRenderer, blockPos, i);
+    }
+
+    @Override
+    public void delete() 
+    {
+        return;
+    }
+}