diff options
author | Thorbjørn Lindeijer <thorbjorn@lindeijer.nl> | 2012-01-23 00:27:29 +0100 |
---|---|---|
committer | Thorbjørn Lindeijer <thorbjorn@lindeijer.nl> | 2012-01-24 19:25:38 +0100 |
commit | 797e86813686aa0d6cd197cbab79f8ca889c261c (patch) | |
tree | 13cffc47dbb2eb90fc7c7c9c0e8964e2b5b2a2fc /src/resources/beinginfo.cpp | |
parent | b856e8b47ab2dfd393e3c2720c5647eb66393931 (diff) | |
download | mana-client-797e86813686aa0d6cd197cbab79f8ca889c261c.tar.gz mana-client-797e86813686aa0d6cd197cbab79f8ca889c261c.tar.bz2 mana-client-797e86813686aa0d6cd197cbab79f8ca889c261c.tar.xz mana-client-797e86813686aa0d6cd197cbab79f8ca889c261c.zip |
Fixed the crash in setupSpriteDisplay
This happened when an NPC, monster or item couldn't be found and it had
to fall back on Being::Unknown. This instance was bugged since it had a
0 pointer in its sprites list, because when the Being::Unknown was
created, the SpriteDef::Empty was not initialized yet (since both were
global static variables, the initialization order was not well
defined).
Fixed it by removing SpriteDef::Empty and instead creating it in the
BeingInfo constructor. I've also changed the SpriteReference instances
to be inline values rather than instances on the heap, since they're
quite small. That also fixed a leak since those instances were never
getting deleted.
Reviewed-by: Yohann Ferreira
Diffstat (limited to 'src/resources/beinginfo.cpp')
-rw-r--r-- | src/resources/beinginfo.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/resources/beinginfo.cpp b/src/resources/beinginfo.cpp index 62990392..bd7cc337 100644 --- a/src/resources/beinginfo.cpp +++ b/src/resources/beinginfo.cpp @@ -37,7 +37,9 @@ BeingInfo::BeingInfo(): mBlockType(Map::BLOCKTYPE_CHARACTER) { SpriteDisplay display; - display.sprites.push_back(SpriteReference::Empty); + + SpriteReference errorSprite(paths.getStringValue("spriteErrorFile"), 0); + display.sprites.push_back(errorSprite); setDisplay(display); } |