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

List:       kde-core-devel
Subject:    Re: [PATCH] KConfig code cleanups
From:       Oswald Buddenhagen <ossi () kde ! org>
Date:       2007-10-29 7:57:15
Message-ID: 20071029075715.GA4160 () ugly ! local
[Download RAW message or body]

On Sun, Oct 28, 2007 at 09:44:40PM -0500, Thomas Braxton wrote:
> On 10/28/07, Oswald Buddenhagen <ossi@kde.org> wrote:
> > On Sun, Oct 28, 2007 at 01:39:17PM -0500, Thomas Braxton wrote:
> > > what should be returned as the name of the <default> group?
> > > QByteArray()?
> > >
> > yes
> 
> not a problem, but, since we're not supporting "[]", what would be the
> problem with using "" as the name of the default group? I was thinking
> that we could use this to verify that we have a valid key by testing
> mGroup.isNull() ==> invalid key, something wierd happened.
> 
gah, always the same with you. ;-)
nobody cares what you do internally, but in the api QByteArray() and
QByteArray("") are the same. for return values, null seems more
appropriate, as it better reflects the "no name" instead of "empty name"
idea, but well ...

-- 
Hi! I'm a .signature virus! Copy me into your ~/.signature, please!
--
Chaos, panic, and disorder - my work here is done.
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic