Grimoire of Gaia

Grimoire of Gaia

4M Downloads

Siren doesn't implement setSwingingArms (i.e. func_184724_a), causing a crash

bbbbbbbbba opened this issue ยท 9 comments

commented

crash-2018-10-12_12.38.15-server.txt

There are several monsters (e.g. Siren) that implement GaiaIRangedAttackMob, but doesn't implement setSwingingArms, potentially causing a crash.

The root cause of this issue seems to be that the function public void setSwingingArms(boolean swingingArms) in both EntityMobHostileBase and EntityMobPassiveBase are not properly translated into the appropriate Forge name, func_184724_a.

commented

It's as easy as can be. Create a new survival world that allows cheating, and /summon a Siren. I get an instant crash.

commented

All the ranged mobs do have the setSwingingArms (this is the function used by AbstractSkeleton.java and other ranged mobs in vanilla Minecraft).

It is just that they are not properly implemented since I updated the EntityAIGaiaAttackRangedBow.java code to match the EntityAIAttackRangedBow.java code.

Even if that is the case I have yet to encounter an issue with ranged mobs. If you could describe how I can recreate the crash I will get to fixing this issue as soon as possible.

commented

Forge 14.23.5.2769, me too. With minotaurus same crash.

commented

I'm currently working on a possible solution to this issue. It should be up a in few hours after this post.

commented

As far as I know, this crash is fixed in the 1.6.6 release. Closing.

commented

KorDom this is not related to this issue. Yes it also says "is abstract" but is not related to isSwinging.

commented

Please create a new issue for it.

commented

Ok. I think it is connected, as it appeared immediately after a poor-quality repair of this.