Fixed a bunch of cppcheck warnings

2 files changed, 6 insertions, 8 deletions

diff --git a/src/scripting/luascript.cpp b/src/scripting/luascript.cpp
index 2b9644d..1617f74 100644
--- a/src/scripting/luascript.cpp
+++ b/src/scripting/luascript.cpp
@@ -101,14 +101,12 @@ void LuaScript::push(const std::list<InventoryItem> &itemList)
     lua_createtable(mCurrentState, itemList.size(), 0);
     int itemTable = lua_gettop(mCurrentState);
 
-    for (std::list<InventoryItem>::const_iterator i = itemList.begin();
-         i != itemList.end();
-         i++)
+    for (const InventoryItem &inventoryItem : itemList)
     {
         // create the item structure
         std::map<std::string, int> item;
-        item["id"] = i->itemId;
-        item["amount"] = i->amount;
+        item["id"] = inventoryItem.itemId;
+        item["amount"] = inventoryItem.amount;
         pushSTLContainer<std::string, int>(mCurrentState, item);
         lua_rawseti(mCurrentState, itemTable, ++position);
     }
diff --git a/src/scripting/luautil.h b/src/scripting/luautil.h
index 8bcc2a3..26ed33c 100644
--- a/src/scripting/luautil.h
+++ b/src/scripting/luautil.h
@@ -230,7 +230,7 @@ void pushSTLContainer(lua_State *s, const std::list<T> &container)
     {
         push(s, *i);
         lua_rawseti(s, table, key);
-        i++;
+        ++i;
     }
 }
 
@@ -264,7 +264,7 @@ void pushSTLContainer(lua_State *s, const std::map<Tkey, Tval> &container)
         push(s, i->first);
         push(s, i->second);
         lua_settable(s, table);
-        i++;
+        ++i;
     }
 }
 
@@ -282,7 +282,7 @@ void pushSTLContainer(lua_State *s, const std::set<T> &container)
     {
         push(s, *i);
         lua_rawseti(s, table, key);
-        i++;
+        ++i;
     }
 }