XEEN: Fix highlighting of character when Cast Spell dialog is opened

engines/xeen/character.cpp

@@ -37,6 +37,17 @@ void AttributePair::synchronize(Common::Serializer &s) {
 
 /*------------------------------------------------------------------------*/
 
+int CharacterArray::indexOf(const Character &c) {
+	for (uint idx = 0; idx < size(); ++idx) {
+		if ((*this)[idx] == c)
+			return idx;
+	}
+
+	return -1;
+}
+
+/*------------------------------------------------------------------------*/
+
 Character::Character(): _weapons(this), _armor(this), _accessories(this), _misc(this), _items(this) {
 	clear();
 	_faceSprites = nullptr;

engines/xeen/character.h

@@ -165,6 +165,16 @@ class Character {
 	 */
 	Character(const Character &src);
 
+	/**
+	 * Equality operator
+	 */
+	bool operator==(const Character &src) const { return src._rosterId == _rosterId; }
+
+	/**
+	 * Inequality operator
+	 */
+	bool operator!=(const Character &src) const { return src._rosterId != _rosterId; }
+
 	/**
 	 * Clears the data for a character
 	 */
@@ -346,6 +356,14 @@ class Character {
 	void clearConditions();
 };
 
+class CharacterArray : public Common::Array<Character> {
+public:
+	/**
+	 * Returns the index of a given character in the array
+	 */
+	int indexOf(const Character &c);
+};
+
 } // End of namespace Xeen
 
 #endif /* XEEN_CHARACTER_H */

engines/xeen/dialogs/dialogs_spells.cpp

@@ -383,7 +383,6 @@ CastSpell::~CastSpell() {
 
 int CastSpell::show(XeenEngine *vm) {
 	Combat &combat = *vm->_combat;
-	Interface &intf = *vm->_interface;
 	Party &party = *vm->_party;
 	Spells &spells = *vm->_spells;
 	int charNum;
@@ -403,17 +402,19 @@ int CastSpell::show(XeenEngine *vm) {
 	}
 
 	Character *c = &party._activeParty[charNum];
-	intf.highlightChar(charNum);
-
 	return show(vm, c);
 }
 
 int CastSpell::show(XeenEngine *vm, Character *&c) {
+	Interface &intf = *vm->_interface;
 	Spells &spells = *vm->_spells;
 	CastSpell *dlg = new CastSpell(vm);
 	int spellId;
 	int result = -1;
 
+	// Highlight the character
+	intf.highlightChar(c);
+
 	do {
 		spellId = dlg->execute(c);
 

engines/xeen/interface.cpp

@@ -118,6 +118,12 @@ void PartyDrawer::highlightChar(int charId) {
 	}
 }
 
+void PartyDrawer::highlightChar(const Character *c) {
+	int charNum = _vm->_party->_activeParty.indexOf(*c);
+	if (charNum != -1)
+		highlightChar(charNum);
+}
+
 void PartyDrawer::unhighlightChar() {
 	Resources &res = *_vm->_resources;
 	Windows &windows = *_vm->_windows;

engines/xeen/interface.h

@@ -71,8 +71,18 @@ class PartyDrawer {
 
 	void drawParty(bool updateFlag);
 
+	/**
+	 * Highlights the specified character in the party display at the bottom of the screen
+	 * @param charId		Character number
+	 */
 	void highlightChar(int charId);
 
+	/**
+	 * Highlights the specified character in the party display at the bottom of the screen
+	 * @param c		Character to highlight
+	 */
+	void highlightChar(const Character *c);
+
 	void unhighlightChar();
 
 	void resetHighlight();

engines/xeen/party.h

@@ -215,7 +215,7 @@ class Party {
 public:
 	// Other party related runtime data
 	Roster _roster;
-	Common::Array<Character> _activeParty;
+	CharacterArray _activeParty;
 	bool _newDay;
 	bool _isNight;
 	bool _stepped;