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

List:       kde-core-devel
Subject:    Re: Patch for kdebase/kstart
From:       Lubos Lunak <l.lunak () suse ! cz>
Date:       2002-11-07 16:33:14
[Download RAW message or body]

On Thursday 07 November 2002 17:25, David Faure wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On Thursday 07 November 2002 16:52, Antonio Larrosa Jiménez wrote:
> > -    if (desktop != 0)
> > +    if (desktop != 0 && desktop <= kwinmodule->numberOfDesktops() )
>
> Should be
>   if (desktop > 0 && desktop <= kwinmodule->numberOfDesktops() )
> then, since desktop is an int :)

 But NetWinInfo::onAllDesktops is -1, so then there should be extra
|| desktop == NetWinInfo::onAllDesktops .


-- 
Lubos Lunak
KDE developer
---------------------------------------------------------------------
SuSE CR, s.r.o.  e-mail: l.lunak@suse.cz , l.lunak@kde.org
Drahobejlova 27  tel: +420 2 9654 2373
190 00 Praha 9   fax: +420 2 9654 2374
Czech Republic   http://www.suse.cz/
[prev in list] [next in list] [prev in thread] [next in thread]