KDE/kdelibs/kdeui/windowmanagement

Aaron J. Seigo aseigo at kde.org
Mon Jul 26 19:04:03 CEST 2010


SVN commit 1154998 by aseigo:

since KWindowSystem::currentDesktop() always returns 1 on MS Windows, then it only makes sense for KWindowInfo::desktop() to also return 1 so that things like "windowInfo.isOnDesktop(KWindowSystem::currentDesktop())" actually stand a chance of working. if there's a reason for the mismatch, please let me know.

CCMAIL:kde-windows at kde.org


 M  +2 -4      kwindowinfo_win.cpp  


--- trunk/KDE/kdelibs/kdeui/windowmanagement/kwindowinfo_win.cpp #1154997:1154998
@@ -202,9 +202,7 @@
 
 bool KWindowInfo::isOnDesktop( int desk ) const
 {
-    if(desk == desktop())
-        return true;
-    return false;
+    return desk == desktop();
 }
 
 bool KWindowInfo::onAllDesktops() const
@@ -214,7 +212,7 @@
 
 int KWindowInfo::desktop() const
 {
-    return 0;
+    return 1;
 }
 
 QRect KWindowInfo::geometry() const


More information about the Kde-windows mailing list