SpoutDev/Spoutcraft — pull request #67 by Kasoo (2012-03-14 18:18:23+00:00)
Title: Update src/minecraft/net/minecraft/src/EntityLiving.java

I've made changes to EntityLiving to make gravity, swimspeed, walkingspeed and jump modifiers work correctly again, fixing issues [#1738](https://github.com/SpoutDev/Spout/issues/1738) and [#1729](https://github.com/SpoutDev/Spout/issues/1729)

Additionally I've edited fall() in order to attempt to change the damage taken from a fall in proportion to the gravity, as previously the player would still take damage as if they had been falling at normal gravity speeds.

This gravity damage change is not enough however, the server still sends the client a health-update with the incorrect amount of damage taken. This will need to be fixed within Spout/plugin however I do not have the knowledge to be able to do this, hopefully someone else can take this change.

Finally there are several places (marked by fixmes) where I am unsure whether modifiers need to be applied. I think in some cases the calculations are strictly for AI entities and do not need to be modifiable. However I have left them in because they should not cause any problems even if the entity is an AI.

The original patch that removed the modifiers seems to be authored by Afforess, perhaps consulting them to check the reasoning behind removing these before accepting this pull would be advisable, as there may be reasoning that I do not understand.