[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Kwin grave problem
From:       Antonio Larrosa <antlarr () arrakis ! es>
Date:       2000-10-17 0:03:25
[Download RAW message or body]

Antonio Larrosa wrote:
> 
> Hello Matthias and list,
> 
> Press Ctrl and then press alternatively, randomly and very fast F1, F2,
> F3 and F4.
> 
> Kwin will stop managing some windows in those desktops (first visual
> effect is that the windows
> lose their borders althought there are more important ones).
> 
> I've tried to debug it but I've not found any solution nor know why is
> this happening.
> 
> Note also that changing the style does _NOT_ fix the problem. The
> windows that you loose this
> way seem to be doomed to remain on your desktop forever.
> 

Ok, Carsten Pfeiffer and me worked on this at IRC.

The problem seems to be a fall back in a switch statement.
The attached patch "fixes" this, but it seems to break other
things: While trying to attach the patch to this mail, netscape's
file dialog is not closed after selecting the file.
The same happens by substituting the break; line for an
if (mapped) return TRUE;

Please Matthias (Ettrich) comment on it asap.

(I won't read any mail until tuesday 18:00 so don't expect a fast answer
from me. Sorry)

Greetings,

--
Antonio Larrosa Jimenez
KDE core developer
antlarr@arrakis.es        larrosa@kde.org
http://www.arrakis.es/~rlarrosa
KDE - The development framework of the future, today.
["client.cpp.diff" (text/plain)]

Index: client.cpp
===================================================================
RCS file: /home/kde/kdebase/kwin/client.cpp,v
retrieving revision 1.182
diff -u -r1.182 client.cpp
--- client.cpp	2000/10/16 22:01:16	1.182
+++ client.cpp	2000/10/16 23:48:25
@@ -870,6 +870,7 @@
 	    invalidateWindow();
 	}
 	// fall through
+        break;
     case WithdrawnState: // however that has been possible....
 	withdraw();
 	break;


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic