Fixed an arithmetic exception in ItemShortcutContainer::draw.

Changed default sitting key back to 's'.
Fixed a problem with all equipment being interpreted as weapon sprite.

4 files changed, 234 insertions, 230 deletions

diff --git a/src/resources/equipmentdb.cpp b/src/resources/equipmentdb.cpp
index 9b963720..55107050 100644
--- a/src/resources/equipmentdb.cpp
+++ b/src/resources/equipmentdb.cpp
@@ -1,176 +1,180 @@
-/*

- *  The Mana World

- *  Copyright 2006 The Mana World Development Team

- *

- *  This file is part of The Mana World.

- *

- *  The Mana World is free software; you can redistribute it and/or modify

- *  it under the terms of the GNU General Public License as published by

- *  the Free Software Foundation; either version 2 of the License, or

- *  any later version.

- *

- *  The Mana World is distributed in the hope that it will be useful,

- *  but WITHOUT ANY WARRANTY; without even the implied warranty of

- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the

- *  GNU General Public License for more details.

- *

- *  You should have received a copy of the GNU General Public License

- *  along with The Mana World; if not, write to the Free Software

- *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA

- *

- *  $Id$

- */

-

-#include "equipmentdb.h"

-

-#include "resourcemanager.h"

-

-#include "../log.h"

-

-#include "../utils/dtor.h"

-#include "../utils/xml.h"

-

-namespace

-{

-    EquipmentDB::EquipmentInfos mEquipmentInfos;

-    EquipmentInfo mUnknown;

-    bool mLoaded = false;

-}

-

-void

-EquipmentDB::load()

-{

-    if (mLoaded)

-        return;

-

-    logger->log("Initializing equipment database...");

-    mUnknown.setSprite("error.xml", 0);

-    mUnknown.setSprite("error.xml", 1);

-

-    ResourceManager *resman = ResourceManager::getInstance();

-    int size;

-    char *data = (char*)resman->loadFile("equipment.xml", size);

-

-    if (!data)

-    {

-        logger->error("Equipment Database: Could not find equipment.xml!");

-    }

-

-    xmlDocPtr doc = xmlParseMemory(data, size);

-    free(data);

-

-    if (!doc)

-    {

-        logger->error("Equipment Database: Error while parsing equipment database (equipment.xml)!");

-    }

-

-    xmlNodePtr rootNode = xmlDocGetRootElement(doc);

-    if (!rootNode || !xmlStrEqual(rootNode->name, BAD_CAST "equipments"))

-    {

-        logger->error("Equipment Database: equipment.xml is not a valid database file!");

-    }

-

-    //iterate <equipment>s

-    for_each_xml_child_node(equipmentNode, rootNode)

-    {

-        if (!xmlStrEqual(equipmentNode->name, BAD_CAST "equipment"))

-        {

-            continue;

-        }

-

-        EquipmentInfo *currentInfo = new EquipmentInfo();

-

-        currentInfo->setSlot (XML::getProperty(equipmentNode, "slot", 0));

-        currentInfo->setAttackType (XML::getProperty(equipmentNode, "attacktype", ""));

-

-        //iterate <sprite>s and <sound>s

-        for_each_xml_child_node(spriteNode, equipmentNode)

-        {

-            if (xmlStrEqual(spriteNode->name, BAD_CAST "sprite"))

-            {

-                std::string gender = XML::getProperty(spriteNode, "gender", "unisex");

-                std::string filename = (const char*) spriteNode->xmlChildrenNode->content;

-

-                if (gender == "male" || gender == "unisex")

-                {

-                    currentInfo->setSprite(filename, 0);

-                }

-

-                if (gender == "female" || gender == "unisex")

-                {

-                    currentInfo->setSprite(filename, 1);

-                }

-            }

-

-            if (xmlStrEqual(spriteNode->name, BAD_CAST "sound"))

-            {

-                std::string event = XML::getProperty(spriteNode, "event", "");

-                const char *filename;

-                filename = (const char*) spriteNode->xmlChildrenNode->content;

-

-                if (event == "hit")

-                {

-                    currentInfo->addSound(EQUIP_EVENT_HIT, filename);

-                }

-                else if (event == "strike")

-                {

-                    currentInfo->addSound(EQUIP_EVENT_STRIKE, filename);

-                }

-                else

-                {

-                    logger->log("EquipmentDB: Warning, sound effect %s for unknown event %s",

-                                filename, event.c_str());

-                }

-            }

-        }

-

-        setEquipment(   XML::getProperty(equipmentNode, "id", 0),

-                        currentInfo);

-    }

-

-    mLoaded = true;

-}

-

-void

-EquipmentDB::unload()

-{

-    // kill EquipmentInfos

-    for_each (  mEquipmentInfos.begin(), mEquipmentInfos.end(),

-                make_dtor(mEquipmentInfos));

-    mEquipmentInfos.clear();

-

-    mLoaded = false;

-}

-

-EquipmentInfo*

-EquipmentDB::get(int id)

-{

-    if (!mLoaded) {

-        logger->error("Error: Equipment database used before initialization!");

-    }

-

-    EquipmentInfoIterator i = mEquipmentInfos.find(id);

-

-    if (i == mEquipmentInfos.end() )

-    {

-        logger->log("EquipmentDB: Error, unknown equipment ID# %d", id);

-        return &mUnknown;

-    }

-    else

-    {

-        return i->second;

-    }

-}

-

-void

-EquipmentDB::setEquipment(int id, EquipmentInfo* equipmentInfo)

-{

-    if (mEquipmentInfos.find(id) != mEquipmentInfos.end()) {

-        logger->log("Warning: Equipment Piece with ID %d defined multiple times",

-                    id);

-        delete equipmentInfo;

-    }

-    else {

-        mEquipmentInfos[id] = equipmentInfo;

-    };

-}

+/*
+ *  The Mana World
+ *  Copyright 2006 The Mana World Development Team
+ *
+ *  This file is part of The Mana World.
+ *
+ *  The Mana World is free software; you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation; either version 2 of the License, or
+ *  any later version.
+ *
+ *  The Mana World is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *
+ *  You should have received a copy of the GNU General Public License
+ *  along with The Mana World; if not, write to the Free Software
+ *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ *
+ *  $Id$
+ */
+
+#include <cassert>
+
+#include "equipmentdb.h"
+
+#include "resourcemanager.h"
+
+#include "../log.h"
+
+#include "../utils/dtor.h"
+#include "../utils/xml.h"
+
+namespace
+{
+    EquipmentDB::EquipmentInfos mEquipmentInfos;
+    EquipmentInfo mUnknown;
+    bool mLoaded = false;
+}
+
+void
+EquipmentDB::load()
+{
+    if (mLoaded)
+        return;
+
+    logger->log("Initializing equipment database...");
+    mUnknown.setSprite("error.xml", 0);
+    mUnknown.setSprite("error.xml", 1);
+
+    ResourceManager *resman = ResourceManager::getInstance();
+    int size;
+    char *data = (char*)resman->loadFile("equipment.xml", size);
+
+    if (!data)
+    {
+        logger->error("Equipment Database: Could not find equipment.xml!");
+    }
+
+    xmlDocPtr doc = xmlParseMemory(data, size);
+    free(data);
+
+    if (!doc)
+    {
+        logger->error("Equipment Database: Error while parsing equipment database (equipment.xml)!");
+    }
+
+    xmlNodePtr rootNode = xmlDocGetRootElement(doc);
+    if (!rootNode || !xmlStrEqual(rootNode->name, BAD_CAST "equipments"))
+    {
+        logger->error("Equipment Database: equipment.xml is not a valid database file!");
+    }
+
+    //iterate <equipment>s
+    for_each_xml_child_node(equipmentNode, rootNode)
+    {
+        if (!xmlStrEqual(equipmentNode->name, BAD_CAST "equipment"))
+        {
+            continue;
+        }
+
+        EquipmentInfo *currentInfo = new EquipmentInfo();
+
+        currentInfo->setSlot (XML::getProperty(equipmentNode, "slot", 0));
+        currentInfo->setAttackType (XML::getProperty(equipmentNode, "attacktype", ""));
+
+        //iterate <sprite>s and <sound>s
+        for_each_xml_child_node(spriteNode, equipmentNode)
+        {
+            if (xmlStrEqual(spriteNode->name, BAD_CAST "sprite"))
+            {
+                std::string gender = XML::getProperty(spriteNode, "gender", "unisex");
+                std::string filename = (const char*) spriteNode->xmlChildrenNode->content;
+
+                if (gender == "male" || gender == "unisex")
+                {
+                    currentInfo->setSprite(filename, 0);
+                }
+
+                if (gender == "female" || gender == "unisex")
+                {
+                    currentInfo->setSprite(filename, 1);
+                }
+            }
+
+            if (xmlStrEqual(spriteNode->name, BAD_CAST "sound"))
+            {
+                std::string event = XML::getProperty(spriteNode, "event", "");
+                const char *filename;
+                filename = (const char*) spriteNode->xmlChildrenNode->content;
+
+                if (event == "hit")
+                {
+                    currentInfo->addSound(EQUIP_EVENT_HIT, filename);
+                }
+                else if (event == "strike")
+                {
+                    currentInfo->addSound(EQUIP_EVENT_STRIKE, filename);
+                }
+                else
+                {
+                    logger->log("EquipmentDB: Warning, sound effect %s for unknown event %s",
+                                filename, event.c_str());
+                }
+            }
+        }
+
+        setEquipment(   XML::getProperty(equipmentNode, "id", 0),
+                        currentInfo);
+    }
+
+    mLoaded = true;
+}
+
+void
+EquipmentDB::unload()
+{
+    // kill EquipmentInfos
+    for_each(mEquipmentInfos.begin(), mEquipmentInfos.end(),
+             make_dtor(mEquipmentInfos));
+    mEquipmentInfos.clear();
+
+    mLoaded = false;
+}
+
+EquipmentInfo*
+EquipmentDB::get(int id)
+{
+    if (!mLoaded) {
+        logger->error("Error: Equipment database used before initialization!");
+    }
+
+    EquipmentInfoIterator i = mEquipmentInfos.find(id);
+
+    assert(i != mEquipmentInfos.end());
+
+    if (i == mEquipmentInfos.end())
+    {
+        logger->log("EquipmentDB: Error, unknown equipment ID# %d", id);
+        return &mUnknown;
+    }
+    else
+    {
+        return i->second;
+    }
+}
+
+void
+EquipmentDB::setEquipment(int id, EquipmentInfo* equipmentInfo)
+{
+    if (mEquipmentInfos.find(id) != mEquipmentInfos.end()) {
+        logger->log("Warning: Equipment Piece with ID %d defined multiple times",
+                    id);
+        delete equipmentInfo;
+    }
+    else {
+        mEquipmentInfos[id] = equipmentInfo;
+    };
+}
diff --git a/src/resources/equipmentdb.h b/src/resources/equipmentdb.h
index b8618f5f..76866961 100644
--- a/src/resources/equipmentdb.h
+++ b/src/resources/equipmentdb.h
@@ -1,52 +1,52 @@
-/*

- *  The Mana World

- *  Copyright 2006 The Mana World Development Team

- *

- *  This file is part of The Mana World.

- *

- *  The Mana World is free software; you can redistribute it and/or modify

- *  it under the terms of the GNU General Public License as published by

- *  the Free Software Foundation; either version 2 of the License, or

- *  any later version.

- *

- *  The Mana World is distributed in the hope that it will be useful,

- *  but WITHOUT ANY WARRANTY; without even the implied warranty of

- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the

- *  GNU General Public License for more details.

- *

- *  You should have received a copy of the GNU General Public License

- *  along with The Mana World; if not, write to the Free Software

- *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA

- *

- *  $Id:

- */

-

-#ifndef _TMW_EQUIPMENT_DB_H

-#define _TMW_EQUIPMENT_DB_H

-

-#include <map>

-

-#include "equipmentinfo.h"

-

-namespace EquipmentDB

-{

-    /**

-     * Loads the equipment info from Items.xml

-     */

-    void load();

-

-    /**

-     * Frees equipment data

-     */

-    void unload();

-

-    void setEquipment(int id, EquipmentInfo* equipmentInfo);

-

-    EquipmentInfo* get(int id);

-

-    // Equipment database types

-    typedef std::map<int, EquipmentInfo*> EquipmentInfos;

-    typedef EquipmentInfos::iterator EquipmentInfoIterator;

-}

-

-#endif

+/*
+ *  The Mana World
+ *  Copyright 2006 The Mana World Development Team
+ *
+ *  This file is part of The Mana World.
+ *
+ *  The Mana World is free software; you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation; either version 2 of the License, or
+ *  any later version.
+ *
+ *  The Mana World is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *
+ *  You should have received a copy of the GNU General Public License
+ *  along with The Mana World; if not, write to the Free Software
+ *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ *
+ *  $Id$
+ */
+
+#ifndef _TMW_EQUIPMENT_DB_H
+#define _TMW_EQUIPMENT_DB_H
+
+#include <map>
+
+#include "equipmentinfo.h"
+
+namespace EquipmentDB
+{
+    /**
+     * Loads the equipment info from Items.xml
+     */
+    void load();
+
+    /**
+     * Frees equipment data
+     */
+    void unload();
+
+    void setEquipment(int id, EquipmentInfo* equipmentInfo);
+
+    EquipmentInfo* get(int id);
+
+    // Equipment database types
+    typedef std::map<int, EquipmentInfo*> EquipmentInfos;
+    typedef EquipmentInfos::iterator EquipmentInfoIterator;
+}
+
+#endif
diff --git a/src/resources/itemdb.cpp b/src/resources/itemdb.cpp
index 70ead6ab..7f30ebe6 100644
--- a/src/resources/itemdb.cpp
+++ b/src/resources/itemdb.cpp
@@ -18,7 +18,7 @@
  *  along with The Mana World; if not, write to the Free Software

  *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA

  *

- *  $Id:

+ *  $Id$

  */

 

 #include "itemdb.h"

diff --git a/src/resources/itemdb.h b/src/resources/itemdb.h
index c080194b..54a07c46 100644
--- a/src/resources/itemdb.h
+++ b/src/resources/itemdb.h
@@ -18,7 +18,7 @@
  *  along with The Mana World; if not, write to the Free Software

  *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA

  *

- *  $Id: itemdb.h 2650 2006-09-03 15:00:47Z b_lindeijer $

+ *  $Id$

  */

 

 #ifndef _TMW_ITEM_MANAGER_H