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

List:       kde-panel-devel
Subject:    Re: api review for DataEngineConsumer
From:       Marco Martin <notmart () gmail ! com>
Date:       2012-09-25 9:14:22
Message-ID: CAD6_BotES1Nti+X9=3c0rDB-JVsRH+41XzhypQSfH-f93_o6Vw () mail ! gmail ! com
[Download RAW message or body]

On Tue, Sep 25, 2012 at 9:36 AM, Aaron J. Seigo <aseigo@kde.org> wrote:
> On Monday, September 24, 2012 21:09:57 Marco Martin wrote:
>> rationale to remove remoteDataEngine
>
> the difference between dataEngine and remoteDataEngine was passing a QUrl for
> the remote engine. it seemed natural to merge the two things. not only does
> this give us just one method in the public API, but it also makes the split
> between local/remote less evident / obvious, which is what should be worked
> towards imho.
>
>> and finishedWithEngine?
>
> this was not in DataEngineConsumer (DEC), as DataEngine usage is tied to the
> lifespan of a given DEC. the idea is to just delete the DEC when you are
> finished with the engines loaded using it.

ok, so that pretty much clears it, +1 ;)
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic