[PATCH] New configDialogSize() and saveDialogSize() methods

Christian Loose christian.loose at hamburg.de
Wed Jan 22 22:51:43 GMT 2003


Am Mittwoch, 22. Januar 2003 00:25 schrieb Simon Hausmann:
> On Tue, Jan 21, 2003 at 11:49:01PM +0100, Christian Loose wrote:
> > > Any opinion on my thoughts regarding passing a KConfig reference
> > > instead of a pointer?
> > >
> > > Simon
> >
> > You are right. I forgot to answer your question. Sorry!
> >
> > I don't know. Most methods return a pointer to KConfig like
> > KGlobal::config() etc. So if you changed it to passing a KConfig
> > reference instead, you would have to call it like this
> >
> > configDialogSize( *config, "Dialog" );
> >
> > instead of
> >
> > configDialogSize( config, "Dialog" );
> >
> > Also what happens if config is null?
>
> I don't think it makes sense to call these methods with a null
> pointer, and KGlobal::config() never returns a null pointer either.
>
> Maybe it's a matter of taste though.
>
> Simon

The promised patch. Now with KConfig references. :-)
What do you think?


-------------- next part --------------
A non-text attachment was scrubbed...
Name: pass-kconfig-03.diff
Type: text/x-diff
Size: 4221 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20030122/1a73fd6f/attachment.diff>


More information about the kde-core-devel mailing list