MrNerdy42 / keywizard-legacy

A simple Minecraft mod that makes finding and fixing key conflicts easier.
https://www.curseforge.com/minecraft/mc-mods/keyboard-wizard
Other
5 stars 3 forks source link

[1.12.1] Crash opening the Controls menu #3

Closed MokahTGS closed 5 years ago

MokahTGS commented 6 years ago

Getting an odd crash when trying to open the controls menu on 1.12.1

https://pastebin.com/q1impHDg

mcenderdragon commented 6 years ago

https://github.com/mcenderdragon/Futurepack-API/issues/357 Smae issue. If only keyWiz and FUutrepack are put together it crashes with a IndexOutOfBound exception. Why is that so? I register the keybindings the same way JEI does, do I have to do it at a certain point in time ?

P3rf3ctXZer0 commented 6 years ago

@Thutmose didn't you fix a bug like this for me back in 1.7.10?

MrNerdy42 commented 5 years ago

I picked the wrong time to disappear for a while. It seems like it was confirmed that there is definitely a conflict between KeyboardWizard and the Futurepack Mod. @mcenderdragon could you point me to your key binding code? It would help me get an idea of how our mods interact.

mcenderdragon commented 5 years ago

I think by now this issue should be resolved. So a test with the latest version for 1.12.2 is needed.

MrNerdy42 commented 5 years ago

Great! I wasn't aware this was already addressed. I'll test it out this evening.

MrNerdy42 commented 5 years ago

Okay so I've tested keyWiz and futurepack together and there was no smoke. I'm going to check this one off my list.