Storage Drawers: Biomes O' Plenty Pack

Storage Drawers: Biomes O' Plenty Pack

6M Downloads

Show quantity (Quantity Key) on drawer causing issue

Fireztonez opened this issue ยท 7 comments

commented

Hello,
I have the latest stable release from Storage Drawer for 1.10.2 (v.3.5.16), and when I show quantity of item with the Quantity Key, I have a strange rendering issue. Is look like I can see through some blocks.
The best way for help to understanding this issue is with screenshot I think, so:

With the quantity bug:
drawer_issue

Without quantity:
drawer_issue_noquatity

On the same arround, you can see through a door:
drawer_issue_door

commented

This shouldn't be a big deal to fix. I'm currently traveling though, so I can't look at it until early next week.

commented

Yep. Looks like glDepthMask is turned off and then never turned back on.

And it looks to me like there's no way to turn it off and then set it back to what it was before? This "GlStateManager" class doesn't seem to do a good job of actually managing state!

commented

GlStateMapper was an ill-considered addition, and there's several GL aspects that can't be properly restored through it.

Historically in 1.7, many entities (including vanilla entities) were really sloppy about restoring their state, so you had to be defensive about setting up your state regardless. Though it's polite to clean up after yourself.

commented

To clean up the state I usually use glPushAttrib (which the GlStateManager definitely does not handle correctly), but in modern OGL that's gone and you're expected to maintain your own state stack in client space.

Oh well. Quick-fix incoming.

commented

thank's for that reply, I'm not to sure if I realy understand wath is the GlStateMapper, but I gessing is a technology on OpenGL??

If you said you make a fix for that is perfect, thank's.

And, for sure, Happy new year for you 2!!

commented

So that's what was causing it. I have been wondering what was happening for a long time. Glad to see you guys are already on it.

commented

Fixed in 3.5.17