From kde-commits  Fri May 23 04:38:32 2008
From: Chani Armitage <chanika () gmail ! com>
Date: Fri, 23 May 2008 04:38:32 +0000
To: kde-commits
Subject: KDE/kdebase/workspace/plasma/plasma
Message-Id: <1211517512.116236.21740.nullmailer () svn ! kde ! org>
X-MARC-Message: https://marc.info/?l=kde-commits&m=121151751930326

SVN commit 811456 by chani:

fun fact: in c++, (-1 % foo) == -1


 M  +8 -2      desktopview.cpp  


--- trunk/KDE/kdebase/workspace/plasma/plasma/desktopview.cpp #811455:811456
@@ -337,14 +337,20 @@
 {
     QList<Plasma::Containment*> containments = containment()->corona()->containments();
     int start = containments.indexOf(containment());
-    int i = (start - 1) % containments.size();
+    //fun fact: in c++, (-1 % foo) == -1
+    int i = start - 1;
+    if (i < 0) {
+        i += containments.size();
+    }
     //FIXME this is a *horrible* way of choosing a "previous" containment.
     while (i != start) {
         if (containments.at(i)->containmentType() != Plasma::Containment::PanelContainment &&
             containments.at(i)->screen() == -1) {
             break;
         }
-        i = (i - 1) % containments.size();
+        if (--i < 0) {
+            i += containments.size();
+        }
     }
 
     Plasma::Containment *c = containments.at(i);