BlockRenderer

BlockRenderer

32.3k Downloads

KeyBinding registered at wrong time - kills dev workspaces

Tslat opened this issue · 2 comments

commented

Hi there - just a heads up;
You're currently registering your keybinds in the constructor of ClientRenderHandler, as seen here:

This is somewhat ok, but it breaks development environments which contain your mod, and it shouldn't be done there anyway.
This happens because some dev runtimes don't actually let Minecraft assign an instance (such as datagen), which causes this line to NPE because it's being called when it shouldn't be.

Can you move this to FMLClientSetupEvent, where it should be?

I'd do a PR but I'm not able to do so currently due to being away from my main dev setup

commented

You should really be using the fork of this mod.

Will fix this Soon™. Learning the fork exists and how much more advanced it is than this version very quickly killed any interest I had in reviving it. (This is not a bad thing, but I merged the 1.16 port PR with the expectation of starting a serious dev push on this again, but it's basically already been done.)

commented

Thanks