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

List:       kde-commits
Subject:    Re: KDE/kdebase/workspace/plasma/shells/mid
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2008-08-25 15:42:46
Message-ID: 200808250942.47019.aseigo () kde ! org
[Download RAW message or body]


On Monday 25 August 2008, Kevin Ottens wrote:
> Le Monday 25 August 2008, Aaron J. Seigo a écrit :
> > +    } else {
> > +        QString geom = args->getOption("screen");
>
> Won't it crash if the option isn't set?

no, because it has a default value. i forogot that providing a 3rd argument to 
KCmdLineOptions::add isn't enough and that you actually need to specify a 
<default> in the first item. KCmdLine* is a bit odd in places =)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech


["signature.asc" (application/pgp-signature)]

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