[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-commits
Subject: Re: KDE/kdebase/workspace/plasma/shells/mid
From: Kevin Ottens <ervin () kde ! org>
Date: 2008-08-25 13:32:55
Message-ID: 200808251533.01538.ervin () kde ! org
[Download RAW message or body]
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? I'm wondering since apparently Laurent
fixed something like that this morning when it was still height/width.
It should probably be:
+ } else if (args->hasOption("screen") {
+ QString geom = args->getOption("screen");
Regards.
--
Kévin 'ervin' Ottens, http://ervin.ipsquad.net
"Ni le maître sans disciple, Ni le disciple sans maître,
Ne font reculer l'ignorance."
["signature.asc" (application/pgp-signature)]
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic