cleanup; remove Block##getRenderType

This commit is contained in:
Sen 2025-07-25 10:32:13 +02:00
parent d56f28e6d2
commit af68ad3f30
Signed by: sen
GPG key ID: 3AC50A6F47D1B722
50 changed files with 35 additions and 120 deletions

View file

@ -20,6 +20,7 @@ import common.block.liquid.BlockLiquid;
import common.block.liquid.BlockStaticLiquid;
import common.collect.Maps;
import common.init.BlockRegistry;
import common.init.Blocks;
import common.util.BlockPos;
import common.util.ExtMath;
import common.util.Facing;
@ -50,7 +51,7 @@ public class BlockRenderer
public void renderBlockDamage(State state, BlockPos pos, Sprite texture, IWorldAccess blockAccess)
{
Block block = state.getBlock();
if (block.getRenderType() == 3)
if (block != Blocks.air && !block.getMaterial().isLiquid())
{
state = block.getState(state, blockAccess, pos);
IBakedModel ibakedmodel = this.manager.getModelForState(state);
@ -60,30 +61,17 @@ public class BlockRenderer
}
}
public boolean renderBlock(State state, BlockPos pos, IWorldAccess blockAccess, RenderBuffer worldRendererIn)
{
if(this.gm.xrayActive && !state.getBlock().isXrayVisible())
return false;
int i = state.getBlock().getRenderType();
if (i == -1)
{
return false;
}
else
{
switch (i)
{
case 1:
return this.renderFluid(blockAccess, state, pos, worldRendererIn);
case 3:
IBakedModel ibakedmodel = this.getModelFromBlockState(state, blockAccess, pos);
return this.renderBase(blockAccess, ibakedmodel, state, pos, worldRendererIn, !this.gm.xrayActive);
default:
return false;
}
}
}
public boolean renderBlock(State state, BlockPos pos, IWorldAccess world, RenderBuffer rb) {
Block block = state.getBlock();
if(this.gm.xrayActive && !block.isXrayVisible())
return false;
if(block == Blocks.air)
return false;
else if(block.getMaterial().isLiquid())
return this.renderFluid(world, state, pos, rb);
IBakedModel model = this.getModelFromBlockState(state, world, pos);
return this.renderBase(world, model, state, pos, rb, !this.gm.xrayActive);
}
public IBakedModel getModelFromBlockState(State state, IWorldAccess worldIn, BlockPos pos)
{
@ -113,7 +101,7 @@ public class BlockRenderer
public void renderBlockEntity(State state, float brightness)
{
if (state.getBlock().getRenderType() == 3)
if (state.getBlock() != Blocks.air && !state.getBlock().getMaterial().isLiquid())
{
IBakedModel model = this.manager.getModelForState(state);
this.renderModelBrightness(model, state, brightness, true);

View file

@ -1100,7 +1100,7 @@ public class EffectRenderer {
State iblockstate = this.world.getState(pos);
Block block = iblockstate.getBlock();
if(block.getRenderType() != -1) {
if(block != Blocks.air) {
int i = pos.getX();
int j = pos.getY();
int k = pos.getZ();

View file

@ -17,11 +17,11 @@ import client.renderer.texture.TextureMap;
import common.block.Block;
import common.entity.npc.EntityNPC;
import common.entity.types.EntityLiving;
import common.init.Blocks;
import common.item.Item;
import common.item.ItemAction;
import common.item.ItemStack;
import common.model.BlockLayer;
import common.model.GuiPosition;
import common.util.BlockPos;
import common.util.ExtMath;
import common.util.Vec3;
@ -429,7 +429,7 @@ public class ItemRenderer
}
}
if (iblockstate.getBlock().getRenderType() != -1)
if (iblockstate.getBlock() != Blocks.air)
{
this.renderBlockInside(partialTicks, this.gm.getBlockRendererDispatcher().getModelManager().getTexture(iblockstate));
}

View file

@ -19,7 +19,6 @@ import common.collect.Sets;
import common.init.BlockRegistry;
import common.init.ItemRegistry;
import common.item.Item;
import common.model.Model;
import common.model.ModelProvider;
import common.model.ModelRotation;
import common.model.GuiPosition;

View file

@ -9,9 +9,9 @@ import client.renderer.texture.Sprite;
import client.renderer.texture.TextureMap;
import common.block.Block;
import common.block.liquid.BlockDynamicLiquid;
import common.block.liquid.BlockLiquid;
import common.collect.Maps;
import common.init.BlockRegistry;
import common.init.Blocks;
import common.properties.Property;
import common.world.State;
@ -29,9 +29,8 @@ public class ModelManager
{
this.texMap = textures;
for(Block block : BlockRegistry.blocks()) {
if(block.getRenderType() != 3) {
if(block == Blocks.air || block.getMaterial().isLiquid()) {
this.builtin.add(block);
// Log.info("Builtin: " + BlockRegistry.getNameFromBlock(block));
}
else {
Property<?>[] ignored = block.getIgnoredProperties();

View file

@ -23,6 +23,7 @@ import common.block.Block;
import common.block.ITileEntityProvider;
import common.collect.Maps;
import common.collect.Sets;
import common.init.Blocks;
import common.model.BlockLayer;
import common.tileentity.TileEntity;
import common.util.BlockPos;
@ -172,7 +173,7 @@ public class RenderChunk
BlockLayer enumworldblocklayer1 = block.getRenderLayer();
int j = enumworldblocklayer1.ordinal();
if (block.getRenderType() != -1)
if (block != Blocks.air)
{
RenderBuffer worldrenderer = generator.getRegionRenderCacheBuilder().getWorldRendererByLayerId(j);

View file

@ -12,6 +12,7 @@ import client.renderer.blockmodel.IBakedModel;
import client.renderer.texture.TextureMap;
import common.block.Block;
import common.entity.item.EntityFalling;
import common.init.Blocks;
import common.util.BlockPos;
import common.world.State;
import common.world.World;
@ -37,7 +38,7 @@ public class RenderFallingBlock extends Render<EntityFalling>
BlockPos blockpos = new BlockPos(entity);
World world = entity.getWorldObj();
if (iblockstate != world.getState(blockpos) && block.getRenderType() == 3)
if (iblockstate != world.getState(blockpos) && block != Blocks.air && !block.getMaterial().isLiquid())
{
GL11.glPushMatrix();
GL11.glTranslatef((float)x, (float)y, (float)z);

View file

@ -7,7 +7,6 @@ import client.renderer.texture.TextureMap;
import common.entity.Entity;
import common.item.Item;
import common.item.ItemStack;
import common.model.GuiPosition;
public class RenderItemEntity<T extends Entity> extends Render<T>

View file

@ -8,6 +8,7 @@ import client.renderer.model.ModelBase;
import client.renderer.model.ModelMinecart;
import client.renderer.texture.TextureMap;
import common.entity.item.EntityCart;
import common.init.Blocks;
import common.util.ExtMath;
import common.util.Vec3;
import common.world.State;
@ -93,7 +94,7 @@ public class RenderMinecart<T extends EntityCart> extends Render<T>
int j = entity.getDisplayTileOffset();
State iblockstate = entity.getDisplayTile();
if (iblockstate.getBlock().getRenderType() == 3)
if (iblockstate.getBlock() != Blocks.air && !iblockstate.getBlock().getMaterial().isLiquid())
{
GL11.glPushMatrix();
this.bindTexture(TextureMap.BLOCKS);

View file

@ -10,7 +10,6 @@ import common.entity.types.EntityLiving;
import common.init.Items;
import common.item.Item;
import common.item.ItemStack;
import common.model.GuiPosition;
public class LayerHeldItem implements LayerRenderer<EntityLiving>
{

View file

@ -4,8 +4,8 @@ import org.lwjgl.opengl.GL11;
import client.renderer.Drawing;
import client.renderer.GlState;
import common.block.tile.BlockStandingSign;
import common.block.tile.BlockWallSign;
import common.block.tech.BlockStandingSign;
import common.block.tech.BlockWallSign;
import common.tileentity.TileEntitySign;
import common.world.State;

View file

@ -1069,10 +1069,6 @@ public class Block {
return light;
}
public int getRenderType() {
return 3;
}
@Clientside
public BlockLayer getRenderLayer() {
return BlockLayer.SOLID;

View file

@ -1,6 +1,5 @@
package common.block;
import common.item.Item;
import common.util.BlockPos;
import common.util.BoundingBox;
import common.world.State;
@ -11,10 +10,6 @@ public final class BlockAir extends Block {
super(Material.NONE);
}
public int getRenderType() {
return -1;
}
public BoundingBox getCollisionBox(World world, BlockPos pos, State state) {
return null;
}

View file

@ -6,7 +6,6 @@ import common.color.DyeColor;
import common.item.CheatTab;
import common.model.Model;
import common.model.ModelProvider;
import common.model.GuiPosition;
import common.util.BlockPos;
import common.util.Facing;
import common.world.AWorldServer;

View file

@ -8,7 +8,6 @@ import common.collect.Lists;
import common.entity.Entity;
import common.entity.npc.EntityNPC;
import common.item.CheatTab;
import common.item.Item;
import common.item.tool.ItemLead;
import common.model.Model;
import common.model.ModelProvider;

View file

@ -11,7 +11,6 @@ import common.item.CheatTab;
import common.model.Model;
import common.model.ModelProvider;
import common.model.ModelRotation;
import common.model.GuiPosition;
import common.properties.Property;
import common.properties.PropertyBool;
import common.util.BlockPos;

View file

@ -8,7 +8,6 @@ import common.block.Block;
import common.block.Material;
import common.collect.Sets;
import common.entity.Entity;
import common.item.Item;
import common.model.Model;
import common.model.ModelProvider;
import common.rng.Random;

View file

@ -10,7 +10,6 @@ import common.entity.npc.EntityNPC;
import common.init.BlockRegistry;
import common.init.Blocks;
import common.init.Items;
import common.item.CheatTab;
import common.item.Item;
import common.item.ItemStack;
import common.model.BlockLayer;

View file

@ -4,7 +4,6 @@ import common.block.BlockRotatedPillar;
import common.block.Material;
import common.entity.types.EntityLiving;
import common.item.CheatTab;
import common.item.ItemStack;
import common.properties.Property;
import common.util.BlockPos;
import common.util.Facing;

View file

@ -11,7 +11,6 @@ import common.block.SoundType;
import common.entity.npc.EntityNPC;
import common.init.MetalType;
import common.item.CheatTab;
import common.item.Item;
import common.item.ItemStack;
public class BlockMetalBlock extends Block {

View file

@ -1,21 +1,17 @@
package common.block.artificial;
import java.util.List;
import java.util.Map;
import common.block.Block;
import common.block.Material;
import common.block.natural.BlockFire;
import common.collect.Lists;
import common.entity.Entity;
import common.init.Blocks;
import common.item.Item;
import common.model.BlockLayer;
import common.model.Model;
import common.model.ModelProvider;
import common.properties.Property;
import common.properties.PropertyEnum;
import common.properties.PropertyInteger;
import common.rng.Random;
import common.util.BlockPos;
import common.util.BoundingBox;

View file

@ -11,7 +11,6 @@ import common.entity.npc.EntityNPC;
import common.entity.types.EntityLiving;
import common.init.BlockRegistry;
import common.item.CheatTab;
import common.item.Item;
import common.item.ItemStack;
import common.model.Model;
import common.model.ModelProvider;

View file

@ -3,7 +3,6 @@ package common.block.artificial;
import common.block.Material;
import common.color.DyeColor;
import common.item.CheatTab;
import common.item.Item;
import common.model.BlockLayer;
import common.properties.Property;

View file

@ -11,7 +11,6 @@ import common.model.BlockLayer;
import common.model.Model;
import common.model.ModelProvider;
import common.model.ModelRotation;
import common.model.GuiPosition;
import common.properties.Property;
import common.properties.PropertyBool;
import common.properties.PropertyEnum;

View file

@ -7,7 +7,6 @@ import common.block.Material;
import common.collect.Lists;
import common.init.Blocks;
import common.item.CheatTab;
import common.item.Item;
import common.model.Model;
import common.model.ModelProvider;
import common.model.ModelRotation;

View file

@ -1,10 +1,7 @@
package common.block.foliage;
import common.init.Blocks;
import common.init.Items;
import common.item.Item;
import common.item.StackSize;
import common.item.consumable.ItemSeedFood;
import common.model.Model;
import common.model.ModelProvider;
import common.world.State;

View file

@ -7,8 +7,6 @@ import common.init.Items;
import common.item.CheatTab;
import common.item.Item;
import common.item.ItemStack;
import common.item.StackSize;
import common.item.material.ItemSeeds;
import common.model.Model;
import common.model.ModelProvider;
import common.properties.Property;

View file

@ -23,7 +23,6 @@ import common.util.BlockPos;
import common.util.BoundingBox;
import common.util.Facing;
import common.util.HitPosition;
import common.world.IWorldAccess;
import common.world.State;
import common.world.World;

View file

@ -1,11 +1,8 @@
package common.block.foliage;
import common.init.Blocks;
import common.init.Items;
import common.item.Item;
import common.item.ItemStack;
import common.item.StackSize;
import common.item.consumable.ItemSeedFood;
import common.model.Model;
import common.model.ModelProvider;
import common.util.BlockPos;

View file

@ -3,7 +3,6 @@ package common.block.foliage;
import common.block.Block;
import common.block.Material;
import common.init.Blocks;
import common.item.Item;
import common.item.StackSize;
import common.model.BlockLayer;
import common.model.Model;
@ -16,7 +15,6 @@ import common.util.BlockPos;
import common.util.BoundingBox;
import common.util.Facing;
import common.vars.Vars;
import common.world.IWorldAccess;
import common.world.State;
import common.world.World;
import common.world.AWorldServer;

View file

@ -4,10 +4,7 @@ import common.block.Block;
import common.init.Blocks;
import common.init.Items;
import common.item.CheatTab;
import common.item.Item;
import common.item.ItemStack;
import common.item.StackSize;
import common.item.material.ItemSeeds;
import common.model.Model;
import common.model.ModelProvider;
import common.properties.Property;

View file

@ -5,10 +5,7 @@ import common.block.Material;
import common.init.Blocks;
import common.init.Items;
import common.item.CheatTab;
import common.item.Item;
import common.item.ItemStack;
import common.item.StackSize;
import common.item.material.ItemSeeds;
import common.model.Model;
import common.model.ModelProvider;
import common.properties.Property;

View file

@ -136,14 +136,6 @@ public abstract class BlockLiquid extends Block
return null;
}
/**
* The type of render function called. 3 for standard block models, 2 for TESR's, 1 for liquids, -1 is no render
*/
public int getRenderType()
{
return 1;
}
/**
* Get the Item that this Block should drop when harvested.
*/

View file

@ -5,9 +5,7 @@ import java.util.Map;
import common.block.Block;
import common.block.Material;
import common.block.artificial.BlockPortal;
import common.init.Blocks;
import common.init.SoundEvent;
import common.item.Item;
import common.model.BlockLayer;
import common.model.Model;
import common.model.ModelProvider;

View file

@ -8,7 +8,6 @@ import common.attributes.UsageSlot;
import common.block.SoundType;
import common.entity.npc.EntityNPC;
import common.init.MetalType;
import common.item.Item;
import common.item.ItemStack;
public class BlockMetalOre extends BlockOre {

View file

@ -10,7 +10,6 @@ import common.item.Item;
import common.item.ItemStack;
import common.model.Model;
import common.model.ModelProvider;
import common.model.GuiPosition;
import common.properties.Property;
import common.properties.PropertyInteger;
import common.rng.Random;

View file

@ -1,4 +1,4 @@
package common.block.tile;
package common.block.tech;
import common.block.Block;
import common.block.ITileEntityProvider;

View file

@ -1,20 +1,17 @@
package common.block.tile;
package common.block.tech;
import common.block.Block;
import common.block.Rotatable;
import common.entity.npc.EntityNPC;
import common.init.Blocks;
import common.item.CheatTab;
import common.item.ItemStack;
import common.model.Model;
import common.model.ModelProvider;
import common.model.ModelRotation;
import common.properties.PropertyInteger;
import common.tileentity.TileEntity;
import common.tileentity.TileEntitySign;
import common.util.BlockPos;
import common.util.Clientside;
import common.util.ExtMath;
import common.util.Facing;
import common.world.AWorldServer;
import common.world.IWorldAccess;

View file

@ -2,7 +2,6 @@ package common.block.tech;
import common.block.Block;
import common.block.Material;
import common.color.DyeColor;
import common.color.TextColor;
import common.entity.Entity;
import common.entity.item.EntityTnt;

View file

@ -1,4 +1,4 @@
package common.block.tile;
package common.block.tech;
import common.block.Block;
import common.block.Rotatable;

View file

@ -1,6 +1,5 @@
package common.dimension;
import common.block.Block;
import common.world.State;
public final class Semi extends Dimension {

View file

@ -10,7 +10,6 @@ import common.block.artificial.BlockFenceGate;
import common.block.artificial.BlockWall;
import common.block.liquid.BlockLiquid;
import common.color.TextColor;
import common.dimension.DimType;
import common.dimension.Dimension;
import common.dimension.Space;
import common.enchantment.Enchantment;
@ -1042,7 +1041,7 @@ public abstract class Entity
State iblockstate = this.worldObj.getState(blockpos);
Block block = iblockstate.getBlock();
if (block.getRenderType() != -1)
if (block != Blocks.air)
{
this.worldObj.clientParticle(ParticleType.BLOCK_CRACK, this.posX + ((double)this.rand.floatv() - 0.5D) * (double)this.width, this.getEntityBoundingBox().minY + 0.1D, this.posZ + ((double)this.rand.floatv() - 0.5D) * (double)this.width, BlockRegistry.getId(iblockstate));
}

View file

@ -113,6 +113,7 @@ import common.block.tech.BlockPistonHead;
import common.block.tech.BlockPressurePlate;
import common.block.tech.BlockPressurePlateWeighted;
import common.block.tech.BlockRail;
import common.block.tech.BlockStandingSign;
import common.block.tech.BlockToggleableLight;
import common.block.tech.BlockLitTorch;
import common.block.tech.BlockWire;
@ -121,10 +122,9 @@ import common.block.tech.BlockTianReactor;
import common.block.tech.BlockTripWire;
import common.block.tech.BlockTripWireHook;
import common.block.tech.BlockUnlitTorch;
import common.block.tech.BlockWallSign;
import common.block.tech.BlockWarpChest;
import common.block.tech.BlockWorkbench;
import common.block.tile.BlockStandingSign;
import common.block.tile.BlockWallSign;
import common.collect.BiMap;
import common.collect.HashBiMap;
import common.collect.Lists;

View file

@ -13,7 +13,6 @@ import common.block.foliage.*;
import common.block.liquid.*;
import common.block.natural.*;
import common.block.tech.*;
import common.block.tile.*;
import common.collect.Lists;
import common.collect.Sets;
import common.util.Util;

View file

@ -3,7 +3,6 @@ package common.item.consumable;
import common.block.Block;
import common.block.foliage.BlockCrops;
import common.entity.npc.EntityNPC;
import common.item.CheatTab;
import common.item.ItemStack;
import common.util.BlockPos;
import common.util.Facing;

View file

@ -71,14 +71,14 @@ import common.block.tech.BlockPistonHead;
import common.block.tech.BlockPressurePlate;
import common.block.tech.BlockPressurePlateWeighted;
import common.block.tech.BlockRail;
import common.block.tech.BlockStandingSign;
import common.block.tech.BlockRail.EnumRailDirection;
import common.block.tech.BlockTorch;
import common.block.tech.BlockTNT;
import common.block.tech.BlockTripWire;
import common.block.tech.BlockTripWireHook;
import common.block.tech.BlockWallSign;
import common.block.tech.BlockWarpChest;
import common.block.tile.BlockStandingSign;
import common.block.tile.BlockWallSign;
import common.collect.Lists;
import common.collect.Maps;
import common.color.DyeColor;

View file

@ -2,7 +2,6 @@ package server.worldgen;
import java.util.Arrays;
import common.block.Material;
import common.block.natural.BlockColoredClay;
import common.dimension.Dimension;
import common.init.Blocks;

View file

@ -1,12 +1,8 @@
package server.worldgen.structure;
import java.util.Arrays;
import java.util.List;
import common.rng.Random;
import common.util.BlockPos;
import server.world.WorldServer;
import server.worldgen.Biome;
public class MapGenScatteredFeature extends MapGenStructure
{

View file

@ -1,13 +1,9 @@
package server.worldgen.structure;
import java.util.List;
import java.util.Set;
import common.collect.Sets;
import common.rng.Random;
import common.tags.TagObject;
import server.world.WorldServer;
import server.worldgen.Biome;
public class MapGenVillage extends MapGenStructure
{

View file

@ -21,8 +21,6 @@ import common.vars.Vars;
import common.world.State;
import server.vars.SVars;
import server.world.WorldServer;
import server.worldgen.Biome;
import server.worldgen.BiomeGenerator;
import server.worldgen.LootConstants;