setTabBoxWindow causes the ItemInfo to be regenerated, so we have to break the loop after finding a window.

Makes more sense anyway as we do not have to go through the hash to find the item we already found.
BUG: 222120

svn path=/trunk/KDE/kdebase/workspace/; revision=1080616

effects/boxswitch/boxswitch.cpp

@@ -257,7 +257,10 @@ void BoxSwitchEffect::windowInputMouseEvent( Window w, QEvent* e )
         for( i = windows.constBegin(); i != windows.constEnd(); ++i )
             {
             if( i.value()->clickable.contains( pos ))
+                {
                 effects->setTabBoxWindow( i.key() );
+                break;
+                }
             }
         // special handling for second half of window in case of animation and even number of windows
         if( mAnimateSwitch && ( windows.size() % 2 == 0 ) )