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

List:       kde-commits
Subject:    Re: KDE/kdebase/workspace/plasma/applets/pager
From:       Marco Martin <notmart () gmail ! com>
Date:       2008-12-02 10:29:10
Message-ID: 200812021129.10696.notmart () gmail ! com
[Download RAW message or body]

On Tuesday 02 December 2008, Lubos Lunak wrote:
> On Monday 01 of December 2008, Marco Martin wrote:
> > SVN commit 891404 by mart:
> >
> > be paranoid about what the number of the current desktop is
> > it turns out that KWindowsysten returns numbers dependent from the
> > window manager, starting either from 0 or 1, kwin seems to start by 1,
> > compiz 0, try to find out the answer
>
>  That rather doesn't make sense. The spec numbers desktops as 0-(max-1),
> but the KDE API maps them to 1-max, for whatever historical reason. Which
> means that Compiz would have to claim the current desktop is -1.

hmm, yes, should have double checked that :/
now i just was able to reproduce the crash a single time yesterday, so that 
behaviour in compiz shoud be quite rare
reverted the thing and put just an index checking

>  Also, would it be possible to fix this in the right place, instead of
> patching whichever place the problem affects? This is either a Compiz bug
> or kdelibs/kdeui bug, more probably the sooner.


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