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

List:       kde-commits
Subject:    Re: KDE/kdebase/workspace/kcontrol/access
From:       Benjamin Meyer <benjamin.meyer () trolltech ! com>
Date:       2005-11-03 8:56:43
Message-ID: 200511030956.43926.benjamin.meyer () trolltech ! com
[Download RAW message or body]


On Wednesday 02 November 2005 04:00 pm, Stephan Kulow wrote:
> Am Mittwoch, 2. November 2005 12:10 schrieb Laurent Montel:
> > SVN commit 476899 by mlaurent:
> >
> > Fix ?
> >
> >
> >  M  +4 -3      kcmaccess.cpp
> >  M  +1 -1      kcmaccess.h
> >
> >
> > --- trunk/KDE/kdebase/workspace/kcontrol/access/kcmaccess.cpp
> > #476898:476899 @@ -226,8 +226,8 @@
> >    return result.arg(keyname);
> >  }
> >
> > -KAccessConfig::KAccessConfig(QWidget *parent, const char *)
> > -  : KCModule(parent)
> > +KAccessConfig::KAccessConfig(KInstance *inst, QWidget *parent)
> > +  : KCModule(inst, parent)
> >  {
> >
> >    KAboutData *about =
> > @@ -822,7 +822,8 @@
> >  {
> >    KDE_EXPORT KCModule *create_access(QWidget *parent, const char *name)
> >    {
> > -    return new KAccessConfig(parent, name);
> > +	KInstance *inst = new KInstance(name);
> > +    return new KAccessConfig(inst,parent);
>
> You better hardcode the name here to "kcmaccess" as I'd rather remove the
> name argument from the callback. Not sure how this can be checked though -
> it's just a symbol name ;(
>
> Greetings, Stephan

Does the name even matter?  There are no docs so I have been filling it with 
"".

-Benjamin Meyer

[Attachment #3 (application/pgp-signature)]

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