Ok, so for the longest time, one of the parts that has bugged me the most about the MC rendering code is this stupid test to see if each individual side of each individual block rendered in the entire fucking game was the index of the grass texture.
That drove my inner game programmer nuts as it was a shit-ton of useless tests being performed an insane number of times, probably at great performance cost. It's seen in lines like this:
Code: Select all
if (cfgGrassFix && var27 == 3 && this.overrideBlockTexture < 0)
So, I've been hoping that those would finally disappear when all the texturing code got a rewrite, which it just did.
Well, I just got to the block rendering code during my update, and found out instead, that all those tests have been replaced by this:
Code: Select all
if (cfgGrassFix && var22.getIconName().equals("grass_side") && !this.hasOverrideBlockTexture())
Face palms just don't cover it. Remember how I was beginning to have doubts not just about the design, but programming ability at Mojang right now? Yeah...I don't have any doubts any longer.