diff options
author | Thorbjørn Lindeijer <bjorn@lindeijer.nl> | 2009-07-15 22:37:35 +0200 |
---|---|---|
committer | Thorbjørn Lindeijer <bjorn@lindeijer.nl> | 2009-07-15 22:37:35 +0200 |
commit | fa6d39d9f56e09abfb6b48bacdd66ddbad20c641 (patch) | |
tree | 31a57b7a69c20454e129963e2b2c4327782a2311 /src/net/ea | |
parent | df52e6b900600709ab0069daacf06093c61667b1 (diff) | |
download | mana-client-fa6d39d9f56e09abfb6b48bacdd66ddbad20c641.tar.gz mana-client-fa6d39d9f56e09abfb6b48bacdd66ddbad20c641.tar.bz2 mana-client-fa6d39d9f56e09abfb6b48bacdd66ddbad20c641.tar.xz mana-client-fa6d39d9f56e09abfb6b48bacdd66ddbad20c641.zip |
Fixed compile warnings and crash in skill dialog
The crash was probably due to roaming or uninitialized pointers in the
skill map. Please remember to clear it after deleting its contents, and
also remember that operator[] is non-const and will insert an element
when it's not already there. The proper way to see if a map contains an
element is by using find().
Diffstat (limited to 'src/net/ea')
-rw-r--r-- | src/net/ea/specialhandler.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/net/ea/specialhandler.cpp b/src/net/ea/specialhandler.cpp index 218ff795..528f531e 100644 --- a/src/net/ea/specialhandler.cpp +++ b/src/net/ea/specialhandler.cpp @@ -100,8 +100,8 @@ void SpecialHandler::handleMessage(MessageIn &msg) msg.readInt16(); // target type msg.skip(2); // unused int level = msg.readInt16(); - int sp = msg.readInt16(); - int range = msg.readInt16(); + msg.readInt16(); // sp + msg.readInt16(); // range msg.skip(24); // unused int up = msg.readInt8(); @@ -115,8 +115,8 @@ void SpecialHandler::handleMessage(MessageIn &msg) { skillId = msg.readInt16(); int level = msg.readInt16(); - int sp = msg.readInt16(); - int range = msg.readInt16(); + msg.readInt16(); // sp + msg.readInt16(); // range int up = msg.readInt8(); player_node->setAttributeBase(skillId, level); |