From 72eb9ec9eab5efcb3aa99a962a80423e8c0a3232 Mon Sep 17 00:00:00 2001 From: Torbjörn Andersson Date: Sun, 3 Jan 2010 19:37:43 +0000 Subject: Fixed a bunch of cppcheck warnings. Mostly about checking if a pointer is null before freeing it, which isn't necessary. svn-id: r46941 --- backends/keymapper/remap-dialog.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'backends/keymapper/remap-dialog.cpp') diff --git a/backends/keymapper/remap-dialog.cpp b/backends/keymapper/remap-dialog.cpp index 850f25c4a3..cd6ca1237a 100644 --- a/backends/keymapper/remap-dialog.cpp +++ b/backends/keymapper/remap-dialog.cpp @@ -94,7 +94,7 @@ void RemapDialog::open() { if (_globalKeymaps) { if (divider) _kmPopUp->appendEntry(""); - for (it = _globalKeymaps->begin(); it != _globalKeymaps->end(); it++) { + for (it = _globalKeymaps->begin(); it != _globalKeymaps->end(); ++it) { _kmPopUp->appendEntry(it->_value->getName() + " (Global)", idx); _keymapTable[idx++] = it->_value; } @@ -104,7 +104,7 @@ void RemapDialog::open() { if (_gameKeymaps) { if (divider) _kmPopUp->appendEntry(""); - for (it = _gameKeymaps->begin(); it != _gameKeymaps->end(); it++) { + for (it = _gameKeymaps->begin(); it != _gameKeymaps->end(); ++it) { _kmPopUp->appendEntry(it->_value->getName() + " (Game)", idx); _keymapTable[idx++] = it->_value; } @@ -317,7 +317,7 @@ void RemapDialog::loadKeymap() { List::iterator it; - for (it = km->getActions().begin(); it != km->getActions().end(); it++) { + for (it = km->getActions().begin(); it != km->getActions().end(); ++it) { ActionInfo info = {*it, false, (*it)->description}; _currentActions.push_back(info); -- cgit v1.2.3