Merged revisions 4207-4209,4221-4222 via svnmerge from

https://themanaworld.svn.sourceforge.net/svnroot/themanaworld/tmw/branches/0.0

........
  r4207 | b_lindeijer | 2008-04-29 11:10:43 +0200 (Tue, 29 Apr 2008) | 6 lines
  
  * Fixed issue with determining whether chat input is focused (method no longer
    virtual).
  * Fixed sometimes rendering with the wrong font.
  * Fixed warnings about hiding virtual method, Window now always deletes its
    children (the option not to do so was never used anyway).
........
  r4208 | b_lindeijer | 2008-04-29 13:25:26 +0200 (Tue, 29 Apr 2008) | 3 lines
  
  Removed unused option from GCContainer to not delete a child, fixing warnings
  about hiding virtual method gcn::Container::add.
........
  r4209 | b_lindeijer | 2008-04-29 14:58:21 +0200 (Tue, 29 Apr 2008) | 3 lines
  
  When compiling with GCC, use the printf format attribute for the log method so
  that the compiler checks the type of its parameters.
........
  r4221 | peaveydk | 2008-05-06 21:23:49 +0200 (Tue, 06 May 2008) | 1 line
  
  Dont allow dragging of empty placeholders.
........
  r4222 | peaveydk | 2008-05-08 00:49:38 +0200 (Thu, 08 May 2008) | 1 line
  
  Make F8 toggle shortcut window as suggested by And1 an fate.
........

1 files changed, 4 insertions, 2 deletions

diff --git a/src/localplayer.cpp b/src/localplayer.cpp
index fe3f1f45..6c802c5c 100644
--- a/src/localplayer.cpp
+++ b/src/localplayer.cpp
@@ -197,8 +197,10 @@ Item* LocalPlayer::searchForItem(int itemId)
 {
     for (int i = 0; i < INVENTORY_SIZE; i++)
     {
-        if (itemId == mInventory->getItem(i)->getId()) {
-            return mInventory->getItem(i);
+        if (Item *item = mInventory->getItem(i)) {
+            if (item->getId() == itemId) {
+                return item;
+            }
         }
     }
     return NULL;