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

List:       kde-core-devel
Subject:    Re: Use of const arg in KConfigGroup constructors doesn't work with
From:       Alex Merry <huntedhacker () tiscali ! co ! uk>
Date:       2008-03-06 10:49:02
Message-ID: 200803061049.08140.huntedhacker () tiscali ! co ! uk
[Download RAW message or body]


On Wednesday 05 March 2008 21:56:54 Richard Dale wrote:
> The attached patch add two extra constructors with a bool 'isConst'
> argument:
>
>     /**
>      * This variant of the constructor is useful for non-c++ languages
>      * without const argument overloading.
>      *
>      * ### KDE 5 make isConst default to true
>      */
>     KConfigGroup(KConfigBase *master, const QString &group, bool isConst);
>     KConfigGroup(KConfigBase *master, const char *group, bool isConst);

Surely we want isConst to default to false (or the equivalent with an enum), s
o that c++ retains its const/non-const selection if the default argument is us
ed (ie: behaviour identical to the existing constructors)?

Alex


-- 
KDE: http://www.kde.org
Ubuntu/Kubuntu: http://www.ubuntu.org http://www.kubuntu.org


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

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