Slay Undead Fix

[seveianrex]

{attack.cpp}

~2631, right after the definition of Mob::TryCriticalHit, add:


bool slayUndeadCrit = false;


A few lines down, find the following:


if(defender && defender->GetBodyType() == BT_Undead || defender->GetBodyType() == BT_SummonedUndead || defender->GetBodyType() == BT_Vampire){
switch(GetAA(aaSlayUndead)){
case 1:
critMod += 33;
break;
case 2:
critMod += 66;
break;
case 3:
critMod += 100;
break;
}
}


And add the following immediately after the switch close bracket:


slayUndeadCrit = true;


Then, a few more lines down find the following:


if(critChance > 0){
if(MakeRandomFloat(0, 1) <= critChance)
{


And add immediately below:


if (slayUndeadCrit)
{
damage = (damage * (critMod * 1.65)) / 100;
entity_list.MessageClose(this, false, 200, MT_CritMelee, "%s cleanses %s target!(%d)", GetCleanName(), this->GetGender() == 0 ? "his" : "her", damage);
return;
}

[cavedude]

Just to clarify, by "And add the following immediately after the switch close bracket:" I assume you mean in between the two brackets you have in the snippet? The first closing bracket is the switch bracket, the second belongs to the if statement above it.

[seveianrex]

For clarity's sake, the end result should look like this:


if(defender && defender->GetBodyType() == BT_Undead || defender->GetBodyType() == BT_SummonedUndead || defender->GetBodyType() == BT_Vampire){
switch(GetAA(aaSlayUndead)){
case 1:
critMod += 33;
break;
case 2:
critMod += 66;
break;
case 3:
critMod += 100;
break;
}
slayUndeadCrit = true;
}

[cavedude]

Good, we're on the same page :)