Weird logic in beastmaster.java

[PlNG]

BeastMaster/src/nu/nerd/beastmaster/BeastMaster.java

Line 743 in b3f5d39

if (attackingPlayer != null) {

BeastMaster/src/nu/nerd/beastmaster/BeastMaster.java

Line 745 in b3f5d39

} else if (attackingMob != null) {

If I understand the code correctly, the way the current code is set up, the first will always be false and the second will always be true because attackingplayer or attackingmob gets set, but the whole block is on the premise that attackingmob is set at 729. I think that the "ignore infighting" logic was supposed to be closed with brackets at line 737.

[totemo]

Good catch. I clearly got a bit lost last time I was working on that. I'll fix it up on the next maintenance. Soon.