Skip to content

Conversation

@beppegoodoldrebel
Copy link
Contributor

Here we are in the case where the victim is without an armor/helmet OR the bullet pentrated the armor/helmet. The penetration of the bullet makes no internal damage at all. It took me 5 AK FMJs to get down a unarmored suspect...

//consider adding internal damage
if (!PenetratesVictim)
Damage += Ammo.InternalDamage;

Now I corrected the function like that , so you get half internal damage if a bullet penetrates. On unarmored targets: JHP are still more effective but FMJ/AP/JSP are not that useless anymore....
On armored targets: FMJ/AP/JSP are more effective

//consider adding internal damage
if (!PenetratesVictim)
Damage += Ammo.InternalDamage;

else
{
    if (Victim.isa('SwatPawn') || Victim.isa('SwatPlayer') )
         Damage +=  ( Ammo.InternalDamage /2 );
}

Of course it works BOTH WAYS!

beppegoodoldrebel and others added 2 commits July 14, 2025 22:16
Here we are in the case where the victim is without an armor/helmet OR the bullet pentrated the armor/helmet.
The penetration of the bullet makes no internal damage at all.
It took me 5 AK FMJs to get down a unarmored suspect...

//consider adding internal damage
    if (!PenetratesVictim)
        Damage += Ammo.InternalDamage;

Now I corrected the function like that , so you get half internal damage if a bullet penetrates.
On unarmored targets: JHP are still more effective but FMJ/AP/JSP are not that useless anymore....
On armored targets:   FMJ/AP/JSP are more effective

//consider adding internal damage
    if (!PenetratesVictim)
        Damage += Ammo.InternalDamage;

    else
    {
        if (Victim.isa('SwatPawn') || Victim.isa('SwatPlayer') )
             Damage +=  ( Ammo.InternalDamage /2 );
    }

Of course it works BOTH WAYS!
@beppegoodoldrebel beppegoodoldrebel marked this pull request as ready for review July 14, 2025 20:23
@eezstreet
Copy link
Owner

Won't look at this one until 7.3 or 8.0, whichever we wind up doing next, as it has pretty large and sweeping implications.

There are also other bugs with armor such as #714 that impact this directly and need fixing first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants