From: Mircea Kitsune <sonichedgehog_hyperblast00@yahoo.com>
Date: Sat, 14 Jan 2012 12:17:14 +0000 (+0200)
Subject: Use a safer formula to skip empty bones
X-Git-Tag: xonotic-v0.6.0~110^2^2~14
X-Git-Url: https://git.rm.cloudns.org/?a=commitdiff_plain;h=aaa9171d2bf8aa8cdc1c2fe4564474f805e11774;p=xonotic%2Fxonotic-data.pk3dir.git

Use a safer formula to skip empty bones
---

diff --git a/qcsrc/client/damage.qc b/qcsrc/client/damage.qc
index a0682eef1..703344734 100644
--- a/qcsrc/client/damage.qc
+++ b/qcsrc/client/damage.qc
@@ -49,12 +49,12 @@ void DamageEffect(vector hitorg, float dmg, float type, float specnum)
 	// if there's no skeleton, object origin will automatically be selected
 	FOR_EACH_TAG(self)
 	{
+		if(!tagnum)
+			continue; // skip empty bones
 		// blacklist bones positioned outside the mesh, or the effect will be floating
 		// TODO: Do we have to do it this way? Why do these bones exist at all?
 		if(gettaginfo_name == "master" || gettaginfo_name == "knee_L" || gettaginfo_name == "knee_R" || gettaginfo_name == "leg_L" || gettaginfo_name == "leg_R")
 			continue; // player model bone blacklist
-		if(gettaginfo_name == "")
-			continue; // skip empty bones
 
 		// now choose the bone closest to impact origin
 		if(vlen(hitorg - gettaginfo(self, tagnum)) <= vlen(hitorg - gettaginfo(self, nearestbone)))