API: CSPlayer new members (physics related)

[dystopm]

• Added 4 members to CSPlayer class:
  • m_flJumpHeight: player vertical jump force, 0 = default (~268.3)
  • m_flLongJumpHeight: player vertical jump force with longjump enabled, 0 = default (~299.3)
  • m_flLongJumpForce: player horizontal jump force with longjump enabled, 0 = default (560.0)
  • m_flDuckSpeedMultiplier: player duck speed multiplier, 0 = default (0.3333)
• Cached pmove player's CSPlayer pointer in a global variable

[dystopm]

@wopox1337 @s1lentq need a hand here, any clue of why test demos are failing with this?

[s1lentq]

@dystopm

Fields in structs clientdata_t, entity_state_t, etc. are transmitted over the network, so input data of these fields are registered in the demo through sendto.
There is a possibility that differences between double and float with a difference of >0.0001 lead to fail the demo test because output data is not equal to input data.
Thus the demo file was recorded in completely vanilla code and PLAYER_DUCKING MULTIPLIER explicitly uses double-precision floatpoint (without .f), but in this PR use float type.
In practice, this difference is insignificant, but it is important to maintain it for the purpose of the demo test.

[dystopm]

@dystopm

Fields in structs clientdata_t, entity_state_t, etc. are transmitted over the network, so input data of these fields are registered in the demo through sendto. There is a possibility that differences between double and float with a difference of >0.0001 lead to fail the demo test because output data is not equal to input data. Thus the demo file was recorded in completely vanilla code and PLAYER_DUCKING MULTIPLIER explicitly uses double-precision floatpoint (without .f), but in this PR use float type. In practice, this difference is insignificant, but it is important to maintain it for the purpose of the demo test.

That was clever. I thought for a moment of that kind of issue guessing about a precision issue but I just stared at the fact all three memory types were float, I totally dismissed PLAYER_DUCKING_MULTIPLIER is a double for no having .f suffix. Thanks again! 👌🏼