Use of const arg in KConfigGroup constructors doesn't work with language bindings

Alex Merry huntedhacker at tiscali.co.uk
Thu Mar 6 10:49:02 GMT 2008


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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080306/2db65230/attachment.sig>


More information about the kde-core-devel mailing list