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

List:       kde-core-devel
Subject:    Re: Patch for kdebase/kstart
From:       Antonio Larrosa =?iso-8859-1?q?Jim=E9nez?= <larrosa () kde ! org>
Date:       2002-11-07 16:47:52
[Download RAW message or body]

El Jueves, 7 de Noviembre de 2002 17:34, Waldo Bastian escribió:
> >-----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 :)
>
> That depends on whether -1 should work as well. (Isn't that "all
> desktops" ?)
>

Yes ... ouch !

Fixed.

Greetings,

--
Antonio Larrosa Jimenez
KDE core developer - larrosa@kde.org
http://devel-home.kde.org/~larrosa/
Take care of self-appointed experts.
[prev in list] [next in list] [prev in thread] [next in thread]