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