[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: KDE/kdebase/workspace/libs/plasma
From: "Aaron J. Seigo" <aseigo () kde ! org>
Date: 2008-07-28 22:23:22
Message-ID: 200807281623.22417.aseigo () kde ! org
[Download RAW message or body]
[Attachment #2 (multipart/signed)]
On Monday 28 July 2008, Alex Merry wrote:
> On Monday 28 July 2008 22:43:06 Aaron J. Seigo wrote:
> > hm... good point, at least for the name based approach. i wonder if this
> > shouldn't happen whenever the name changes, however, not only when it's
> > only been run once.
> >
> > see the attached patch; it also fixes the fact that we aren't setting the
> > name on plugin creation (though the apidox clearly says otherwise)
> >
> > thoughts?
>
> Hah! That's exactly what you objected to in the last patch I submitted for
> Plasma::Service, so I didn't put it in when I committed.
i had forgotten about the link between the name of the service and the
automated loading of the operation definition. my objection was to not having a
reason to set it .. well, we do evidently =)
and to make my defeat complete: we can't make it use the storageId either, as
i suggested in that email (since we use the name to create another file name;
the storageId woudl be a bit too unpredictable for the developer for that
purpose)
;)
--
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
["signature.asc" (application/pgp-signature)]
_______________________________________________
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