From 4cd4481663455aa10ce2c525beda570dd146c8ab Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Thomas=20L=C3=BCbking?= Date: Tue, 1 Nov 2011 21:20:31 +0100 Subject: [PATCH] don't use const_iterator's when calling non const functions BUG: 282933 FIXED-IN: 4.7.4 --- workspace.cpp | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/workspace.cpp b/workspace.cpp index 836d44c695..70b2df889c 100644 --- a/workspace.cpp +++ b/workspace.cpp @@ -520,9 +520,7 @@ Workspace::~Workspace() // TODO: grabXServer(); // Use stacking_order, so that kwin --replace keeps stacking order - for (ClientList::ConstIterator it = stacking_order.constBegin(); - it != stacking_order.constEnd(); - ++it) { + for (ClientList::iterator it = stacking_order.begin(), end = stacking_order.end(); it != end; ++it) { // Only release the window (*it)->releaseWindow(true); // No removeClient() is called, it does more than just removing. @@ -531,9 +529,7 @@ Workspace::~Workspace() clients.removeAll(*it); desktops.removeAll(*it); } - for (UnmanagedList::ConstIterator it = unmanaged.constBegin(); - it != unmanaged.constEnd(); - ++it) + for (UnmanagedList::iterator it = unmanaged.begin(), end = unmanaged.end(); it != end; ++it) (*it)->release(); #ifdef KWIN_BUILD_DESKTOPCHANGEOSD delete desktop_change_osd;