From e7ffed76b43471052597bae43afe51c168ded01b Mon Sep 17 00:00:00 2001
From: Paul Gilbert <dreammaster@scummvm.org>
Date: Mon, 16 Feb 2015 11:42:29 -0500
Subject: [PATCH] XEEN: Implemented giveCharDamage

---
 engines/xeen/combat.cpp    | 134 ++++++++++++++++++++++++++++++++++++-
 engines/xeen/combat.h      |   2 +-
 engines/xeen/interface.cpp |   2 +-
 3 files changed, 134 insertions(+), 4 deletions(-)

diff --git a/engines/xeen/combat.cpp b/engines/xeen/combat.cpp
index 3797c181398..9848e8037c3 100644
--- a/engines/xeen/combat.cpp
+++ b/engines/xeen/combat.cpp
@@ -104,8 +104,132 @@ void Combat::doCombat() {
 	error("TODO: doCombat");
 }
 
-void Combat::giveCharDamage(int damage, int v2, int v3) {
-	error("TODO: giveCharDamage");
+void Combat::giveCharDamage(int damage, DamageType attackType, int charIndex) {
+	Party &party = *_vm->_party;
+	Screen &screen = *_vm->_screen;
+	Scripts &scripts = *_vm->_scripts;
+	SoundManager &sound = *_vm->_sound;
+	int charIndex1 = charIndex + 1;
+	int selectedIndex1 = 0;
+	int selectedIndex2 = 0;
+	bool breakFlag = false;
+
+	screen.closeWindows();
+	
+	int idx = (int)party._activeParty.size();
+	if (!scripts._v2) {
+		for (idx = 0; idx < (int)party._activeParty.size(); ++idx) {
+			Character &c = party._activeParty[idx];
+			Condition condition = c.worstCondition();
+
+			if (!(condition >= UNCONSCIOUS && condition <= ERADICATED)) {
+				if (!selectedIndex1) {
+					selectedIndex1 = idx + 1;
+				} else {
+					selectedIndex2 = idx + 1;
+					break;
+				}
+			}
+		}
+	}
+	if (idx == (int)party._activeParty.size()) {
+		selectedIndex1 = scripts._v2 ? charIndex : 0;
+		goto loop;
+	}
+
+	for (;;) {
+		// The if below is to get around errors due to the
+		// goto I was forced to use when reimplementing this method
+		if (true) {
+			Character &c = party._activeParty[selectedIndex1];
+			c._conditions[ASLEEP] = 0;	// Force character to be awake
+
+			int frame = 0, fx = 0;
+			switch (attackType) {
+			case DT_PHYSICAL:
+				fx = 29;
+				break;
+			case DT_MAGICAL:
+				frame = 6;
+				fx = 27;
+				break;
+			case DT_FIRE:
+				damage -= party._fireResistence;
+				frame = 1;
+				fx = 22;
+				break;
+			case DT_ELECTRICAL:
+				damage -= party._electricityResistence;
+				frame = 2;
+				fx = 23;
+				break;
+			case DT_COLD:
+				damage -= party._coldResistence;
+				frame = 3;
+				fx = 24;
+				break;
+			case DT_POISON:
+				damage -= party._poisonResistence;
+				frame = 4;
+				fx = 26;
+				break;
+			case DT_ENERGY:
+				frame = 5;
+				fx = 25;
+				break;
+			case DT_SLEEP:
+				fx = 38;
+				break;
+			default:
+				break;
+			}
+
+			// All attack types other than physical allow for saving
+			// throws to reduce the damage
+			if (attackType != DT_PHYSICAL) {
+				while (c.charSavingThrow(attackType) && damage > 0)
+					damage /= 2;
+			}
+
+			// Draw the attack effect on the character sprite
+			sound.playFX(fx);
+			_powSprites.draw(screen, frame,
+				Common::Point(CHAR_FACES_X[selectedIndex1], 150));
+			screen._windows[33].update();
+
+			// Reduce damage if power shield active, and set it zero
+			// if the damage amount has become negative.. you wouldn't
+			// want attacks healing the characters
+			if (party._powerShield)
+				damage -= party._powerShield;
+			if (damage < 0)
+				damage = 0;
+
+			// TODO: This seems weird.. maybe I've got attack types wrong..
+			// why should attack type 7 (DT_SLEEP) set the dead condition?
+			if (attackType == DT_SLEEP) {
+				damage = c._currentHp;
+				c._conditions[DEAD] = 1;
+			}
+
+			// Subtract the hit points from the character
+			c.subtractHitPoints(damage);
+		}
+
+		if (selectedIndex2) {
+			++selectedIndex1;
+loop:
+			if ((scripts._v2 ? charIndex1 : (int)party._activeParty.size()) > selectedIndex1)
+				break;
+		}
+
+		// Break check and if not, move to other index
+		if (!selectedIndex2 || breakFlag)
+			break;
+
+		selectedIndex1 = selectedIndex2 - 1;
+		breakFlag = true;
+	}
 }
 
 void Combat::moveMonsters() {
@@ -445,6 +569,12 @@ void Combat::attackMonster(int monsterId) {
 }
 
 bool Combat::stopAttack(const Common::Point &diffPt) {
+	Map &map = *_vm->_map;
+
+	if (map._isOutdoors) {
+
+	}
+
 	// TODO
 	return false;
 }
diff --git a/engines/xeen/combat.h b/engines/xeen/combat.h
index 977d5800470..d6ec2333e81 100644
--- a/engines/xeen/combat.h
+++ b/engines/xeen/combat.h
@@ -95,7 +95,7 @@ public:
 
 	void doCombat();
 
-	void giveCharDamage(int damage, int v2, int v3);
+	void giveCharDamage(int damage, DamageType attackType, int charIndex);
 
 	void moveMonsters();
 };
diff --git a/engines/xeen/interface.cpp b/engines/xeen/interface.cpp
index db0bd815aaf..c8324a652a4 100644
--- a/engines/xeen/interface.cpp
+++ b/engines/xeen/interface.cpp
@@ -839,7 +839,7 @@ void Interface::startFalling(bool flag) {
 	draw3d(true);
 	int tempVal = scripts._v2;
 	scripts._v2 = 0;
-	combat.giveCharDamage(party._fallDamage, 0, 0);
+	combat.giveCharDamage(party._fallDamage, DT_PHYSICAL, 0);
 	scripts._v2 = tempVal;
 
 	_flipGround ^= 1;