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

List:       kde-panel-devel
Subject:    Re: [Panel-devel] DataEngine and Corona patches
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2007-08-23 23:18:44
Message-ID: 200708231718.44745.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Thursday 23 August 2007, Thomas Georgiou wrote:
> On Wednesday 15 August 2007 02:52:05 pm Aaron J. Seigo wrote:
> > On Wednesday 15 August 2007, Percy Leonhardt wrote:
> > > On Wednesday 08 August 2007, Aaron J. Seigo wrote:
> > > > wrong. sourceRequested may result in the source being created. that's
> > > > the whole point of calling sourceRequested and it returning a bool.
> > > > if the engine can't create the source, then it returns false.
> > > > otherwise, sourceRequested creates the source and returns true.
> > >
> > > BTW: how is the data engine supposed to create the source? Is it okay
> > > to create a new datacontainer and then use addSource()?
> > >
> > > Or should I use setData() like seen in other engines?
> >
> > either way works. whichever is easiest or makes the most sense for you
> > and your particular engine.
> >
> > > Problem is, that I
> > > don't really have something to set in requestSource().
> >
> > at the very least, create something even if it doesn't have any data()
> > yet.
>
> If a DataContainer is created, does that count as being updated?

no. it has to be passed to addSource(DataContainer* source).

> Now, it does not, but emitting updated on creation of the DataContainer
> seems to be a more logical solution.

not really, since it can't know where to send the signal until it is 
associated with an engine.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech

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

_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel


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