Changes between Initial Version and Version 1 of Ticket #13359, comment 2


Ignore:
Timestamp:
03/20/22 07:43:48 (9 months ago)
Author:
mduggan

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #13359, comment 2

    initial v1  
    44* The `BOUNCBOX::gotHit` event should change a `BLASERNS` to a `BLASEREW`, and vice-versa
    55* The `BLASER` repeatedly uses `I_legalMoveToPoint` without the `move_if_blocked` flag set.  When the move fails, it reverses direction
    6 * This suggests that even when `move_if_blocked` is *not* set, we should still call the hit / gotHit events - otherwise there is no way for `BOUNCBOX::gotHit` to ever get called in this case.
     6* This suggests that even when `move_if_blocked` is *not* set, we should still call the hit / gotHit events - otherwise there is no way for `BOUNCBOX::gotHit` to ever get called in this case (the laser's move just fails and it reverses direction)
    77
    88I can put calling the events in before the early return in `Item::I_legalMoveToPoint` and it should fix this problem.  I hope it doesn't cause any other problems?  The current behaviour was set in revision cbcebce55ddf to make spiderbombs work right.  I need to go back and compare to the behaviour before that too to be sure.