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

List:       kwin
Subject:    Re: clientArea crash
From:       "Guillaume Pothier" <gpothier () gmail ! com>
Date:       2008-11-23 19:15:28
Message-ID: 8caa8ded0811231115ge4b05f3vd24d5aa81f1f6a96 () mail ! gmail ! com
[Download RAW message or body]

On Wed, Nov 19, 2008 at 11:07 AM, Lubos Lunak <l.lunak@suse.cz> wrote:
>  Screen being -1 means 'active screen', but IIRC it's not an allowed value for
> active screen itself. The bug is that client's screen() should never return
> this value. Looking at Toplevel::screen(), I guess there needs to be an
> additional check for the return value (also, specifically for #173017, I
> think windows should be moved when they were on a screen that has
> disappeared).

Ok, I added the check and committed. Unfortunately I don't have an
external monitor today so I can't check whether this is enough to fix
the crash. I'll see that tomorrow.
Regarding moving the windows that are on a removed screen, it seems to
already work that way because when I remove a screen all the windows
it contains are moved to the remaining one.
Cheers,
g
_______________________________________________
kwin mailing list
kwin@kde.org
https://mail.kde.org/mailman/listinfo/kwin
[prev in list] [next in list] [prev in thread] [next in thread]