Fix vehicle crashes

[PabstMirror]

Fix #3306
Fix #3269

Enemy RPG to MRAP (driver):

HD: [carrd,"",1,<NULL-object>,"",-1]
HD: [carrd,"",1,<NULL-object>,"",-1]
HD: [carrd,"",1,O Alpha 1-1:1,"R_PG32V_F",-1]

Enemy RPG to MRAP (gunner):

HD: [carrg,"",1,O Alpha 1-1:1 REMOTE,"R_PG32V_F",-1]

35m fall veh explodes (driver):

HD: [carrd,"",1,<NULL-object>,"",-1]
HD: [carrd,"",1,<NULL-object>,"",-1]
HD: [carrd,"",1,carrd,"",-1]

35m fall veh explodes (gunner):

HD: [carrg,"",1,carrd,"",-1]

So in crashes, the shooter/source can be the pilot for non-drivers.
Switching the order in determine if fatal will handle RPG shots that blow up vehicles as there doesn't seem to be any hit point damage.

[SzwedzikPL]

Is this also fixes #1811 ?

[PabstMirror]

@SzwedzikPL I believe so, I couldn't reproduce that issue directly, but now in-general a vehicle explosion should almost always be fatal.

[thojkooi]

Why are we doing this? Isn't selection "" here?

[PabstMirror]

On crash, handle damage caching will convert the "" into a random selection,
I think we want the same value for passing to FUNC(determineIfFatal)
But we don't want to overwrite _selection for the _selection in ["", "head", "body"] checks on instantDeath/revive.

[thojkooi]

👍 Just my two comments. Nice work @PabstMirror

[ghost]

(RHS) something strange become with AGS damage to troops and hellfire from apache to t72 tank with this change. Cant say in detail. We will test this issue in the next few days on the new version

[SzwedzikPL]

This problem is not fixed. Today i had a situation, my helicopter was shot down. After helicopter explosion i was still alive in vehicle. As soon as the fire was over I got out of the vehicle without any wounds.

[ghost]

@SzwedzikPL
did you test it on 3.5.0 version ?

[SzwedzikPL]

@ivankrouglyi Yes

[jonpas]

Create a new issue with app the required information please, don't reopen old closed PRs that were already released, thank you.