diff options
| author | Torbjörn Andersson | 2016-09-24 09:34:28 +0200 |
|---|---|---|
| committer | Torbjörn Andersson | 2016-09-24 09:34:28 +0200 |
| commit | 31ffa223876130532351a4394e32b2f311b38104 (patch) | |
| tree | ce729fda65d31e31ec449a5fca1dca8ae18788d5 /engines/macventure | |
| parent | 42d8a82c437ef041dd436768ffc058fe6b8e1894 (diff) | |
| download | scummvm-rg350-31ffa223876130532351a4394e32b2f311b38104.tar.gz scummvm-rg350-31ffa223876130532351a4394e32b2f311b38104.tar.bz2 scummvm-rg350-31ffa223876130532351a4394e32b2f311b38104.zip | |
MACVENTURE: Silence Cppcheck warning
I think it was harmless, but don't use 'it' after erasing it.
Diffstat (limited to 'engines/macventure')
| -rw-r--r-- | engines/macventure/gui.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/engines/macventure/gui.cpp b/engines/macventure/gui.cpp index 14c99f6cd7..9e0a6e9f00 100644 --- a/engines/macventure/gui.cpp +++ b/engines/macventure/gui.cpp @@ -1129,12 +1129,11 @@ Common::Point Gui::localizeTravelledDistance(Common::Point point, WindowReferenc void Gui::removeInventoryWindow(WindowReference ref) { _inventoryWindows.remove_at(ref - kInventoryStart); - bool found = false; Common::List<WindowData>::iterator it; - for (it = _windowData->begin(); it != _windowData->end() && !found; it++) { + for (it = _windowData->begin(); it != _windowData->end(); it++) { if (it->refcon == ref) { _windowData->erase(it); - found = true; + break; } } } |