From 59544993bd54839d40b4d771d787c55398d0f1de Mon Sep 17 00:00:00 2001 From: Sebastian Hartte Date: Wed, 26 Oct 2016 00:37:33 +0200 Subject: [PATCH] This reverts back to the old way of extending the vertex format with the necessary vertex element for lightmap data, if fullbright quads are enabled. Instead of then enabling the extended vertex format for Optifine, it is disabled if Optifine is detected. Since the root cause is actually that the Vanilla lighting pipeline doesn't support such Vertex Formats, we disable it also if the Forge lighting pipeline is disabled. This also relates to #2489. --- src/main/java/appeng/client/ClientHelper.java | 8 ++- .../appeng/client/render/VertexFormats.java | 68 +++++++++++++++++++ .../client/render/cablebus/CubeBuilder.java | 4 +- .../client/render/model/UVLModelLoader.java | 6 +- 4 files changed, 82 insertions(+), 4 deletions(-) create mode 100644 src/main/java/appeng/client/render/VertexFormats.java diff --git a/src/main/java/appeng/client/ClientHelper.java b/src/main/java/appeng/client/ClientHelper.java index 490cf135..d90bfe21 100644 --- a/src/main/java/appeng/client/ClientHelper.java +++ b/src/main/java/appeng/client/ClientHelper.java @@ -39,7 +39,9 @@ import net.minecraftforge.client.event.MouseEvent; import net.minecraftforge.client.event.RenderLivingEvent; import net.minecraftforge.client.event.TextureStitchEvent; import net.minecraftforge.client.model.ModelLoaderRegistry; +import net.minecraftforge.common.ForgeModContainer; import net.minecraftforge.common.MinecraftForge; +import net.minecraftforge.fml.client.FMLClientHandler; import net.minecraftforge.fml.common.eventhandler.SubscribeEvent; import appeng.api.parts.CableRenderMode; @@ -79,7 +81,11 @@ public class ClientHelper extends ServerHelper public void preinit() { MinecraftForge.EVENT_BUS.register( this ); - ModelLoaderRegistry.registerLoader( UVLModelLoader.INSTANCE ); + // Do not register the Fullbright hacks if Optifine is present or if the Forge lighting is disabled + if( !FMLClientHandler.instance().hasOptifine() && ForgeModContainer.forgeLightPipelineEnabled ) + { + ModelLoaderRegistry.registerLoader( UVLModelLoader.INSTANCE ); + } } @Override diff --git a/src/main/java/appeng/client/render/VertexFormats.java b/src/main/java/appeng/client/render/VertexFormats.java new file mode 100644 index 00000000..36334da1 --- /dev/null +++ b/src/main/java/appeng/client/render/VertexFormats.java @@ -0,0 +1,68 @@ +/* + * This file is part of Applied Energistics 2. + * Copyright (c) 2013 - 2015, AlgorithmX2, All rights reserved. + * + * Applied Energistics 2 is free software: you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * Applied Energistics 2 is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public License + * along with Applied Energistics 2. If not, see . + */ + +package appeng.client.render; + + +import net.minecraft.client.renderer.vertex.DefaultVertexFormats; +import net.minecraft.client.renderer.vertex.VertexFormat; +import net.minecraftforge.common.ForgeModContainer; +import net.minecraftforge.fml.client.FMLClientHandler; + + +/** + * Utility for managing extended Vertex Formats without having to re-clone existing vertex formats over and over again. + */ +public final class VertexFormats +{ + + // Standard item format extended with lightmap coordinates + private static final VertexFormat itemFormatWithLightMap = new VertexFormat( DefaultVertexFormats.ITEM ).addElement( DefaultVertexFormats.TEX_2S ); + + private VertexFormats() { + } + + public static VertexFormat getFormatWithLightMap( VertexFormat format ) + { + // Do not use this when Optifine is present or if the vanilla lighting pipeline is used + if( FMLClientHandler.instance().hasOptifine() || !ForgeModContainer.forgeLightPipelineEnabled ) + { + return format; + } + + VertexFormat result; + if( format == DefaultVertexFormats.BLOCK ) + { + result = DefaultVertexFormats.BLOCK; + } + else if( format == DefaultVertexFormats.ITEM ) + { + result = itemFormatWithLightMap; + } + else if( !format.hasUvOffset( 1 ) ) + { + result = new VertexFormat( format ); + result.addElement( DefaultVertexFormats.TEX_2S ); + } + else + { + result = format; // Already has the needed UV, so keep it + } + return result; + } +} diff --git a/src/main/java/appeng/client/render/cablebus/CubeBuilder.java b/src/main/java/appeng/client/render/cablebus/CubeBuilder.java index d161e4cd..3ec6bcd1 100644 --- a/src/main/java/appeng/client/render/cablebus/CubeBuilder.java +++ b/src/main/java/appeng/client/render/cablebus/CubeBuilder.java @@ -35,6 +35,8 @@ import net.minecraft.client.renderer.vertex.VertexFormatElement; import net.minecraft.util.EnumFacing; import net.minecraftforge.client.model.pipeline.UnpackedBakedQuad; +import appeng.client.render.VertexFormats; + /** * Builds the quads for a cube. @@ -84,7 +86,7 @@ public class CubeBuilder if( renderFullBright ) { savedFormat = format; - format = DefaultVertexFormats.BLOCK; + format = VertexFormats.getFormatWithLightMap( format ); } for( EnumFacing face : drawFaces ) diff --git a/src/main/java/appeng/client/render/model/UVLModelLoader.java b/src/main/java/appeng/client/render/model/UVLModelLoader.java index 131c53f9..d456dde0 100644 --- a/src/main/java/appeng/client/render/model/UVLModelLoader.java +++ b/src/main/java/appeng/client/render/model/UVLModelLoader.java @@ -64,7 +64,6 @@ import net.minecraft.client.renderer.block.model.ItemTransformVec3f; import net.minecraft.client.renderer.block.model.ModelBakery; import net.minecraft.client.renderer.block.model.ModelBlock; import net.minecraft.client.renderer.texture.TextureAtlasSprite; -import net.minecraft.client.renderer.vertex.DefaultVertexFormats; import net.minecraft.client.renderer.vertex.VertexFormat; import net.minecraft.client.resources.IResource; import net.minecraft.client.resources.IResourceManager; @@ -81,6 +80,8 @@ import net.minecraftforge.common.model.IModelState; import net.minecraftforge.common.model.ITransformation; import net.minecraftforge.fml.relauncher.ReflectionHelper; +import appeng.client.render.VertexFormats; + public enum UVLModelLoader implements ICustomModelLoader { @@ -335,7 +336,8 @@ public enum UVLModelLoader implements ICustomModelLoader Pair brightness = uvlightmap.get( face ); if( brightness != null ) { - UnpackedBakedQuad.Builder builder = new UnpackedBakedQuad.Builder( DefaultVertexFormats.BLOCK ); + VertexFormat newFormat = VertexFormats.getFormatWithLightMap( quad.getFormat() ); + UnpackedBakedQuad.Builder builder = new UnpackedBakedQuad.Builder( newFormat ); VertexLighterFlat trans = new VertexLighterFlat( Minecraft.getMinecraft().getBlockColors() ){ @Override