[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: signal messageButtonPressed(const Plasma::MessageButton button);
From: "Aaron J. Seigo" <aseigo () kde ! org>
Date: 2011-02-12 18:40:10
Message-ID: 201102121040.10784.aseigo () kde ! org
[Download RAW message or body]
[Attachment #2 (multipart/signed)]
On Saturday, February 12, 2011, Marco Martin wrote:
> On Friday 11 February 2011, Aaron J. Seigo wrote:
> > On Tuesday, February 8, 2011, Marco Martin wrote:
> > > Hi all,
> > > I see in Plasma::Applet that
> > > void messageButtonPressed(const MessageButton button);
> > > was changed in
> > > void messageButtonPressed(const Plasma::MessageButton button);
> >
> > that sucks; specifying Plasma:: in the signature is absolutely required
> > for it to be expected to work consistently. the only way for
> > non-namespaced params to work in signals like this is if plasmoid
> > writers include a "using Plasma" statement in their code which does a
> > lot to invalidate the point of having a namespace in the first place.
> >
> > i suppose we can move back in this case, but it's really, really, really
> > something we need to make consistent in libplasma2 as it is a source of
> > ongoing annoyances.
>
> to me honestly if it's ok for the only real user that i know of that we
> have right now of this particular feature, we can really leave it changed
> with the proper namespace ;)
ok, that's good then :)
so it's just the lesson gong forward that new signals must use namespaces in
their params and we need to flag old signals that fail this for fixing in
libplasma2.
(and i bear some responsibility here as i "fixed" it to use the namespace so
this signal could more easily be used elsewhere)
--
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 Qt Development Frameworks
["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