Fixed: P_PlayerThink was called after the previous players had already moved online potentially voiding hitscan attacks (fixes 3315).

#59 Merged at c66dd9b
Repository
Deleted repository
Branch
default (c66dd9b9ec47)
Repository
zandronum-stable
Branch
default
Author
  1. Leonard74123
Reviewers
Description
No description

Comments (2)

  1. torr.samaho repo owner

    The code looks good (I just left a cosmetic comment), but how thoroughly did you test this? This looks like something that needs to be tested very thoroughly.

    1. Leonard74123 author

      I followed exactly the reproducing steps in the ticket and as reported I'm forced to aim ahead of the player without the fix applied.

      With the fix everything works as expected, I also tested with some latency to see if nothing broke with unlagged hitting and everything works just fine.