> On 2011-01-03 18:50:58, Kevin Ottens wrote: > > No go API wise IMO. Only the signal should be in the public API and that's > > it, anything else is showing implementation details to the outside I think.
Well, the main rationale behind it is that the signal can be indeed not be emitted at all - and for the application it might be interested to know it. On a second thought, though, this seem to be useless as the app would simply ignore the signal - so avoiding to emit it would be more than fine. I'll update the patch. Btw, I suppose this is no go for 4.6, isn't it? - Dario ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6119/#review9490 ----------------------------------------------------------- On 2010-12-15 14:05:42, Dario Freddi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/6119/ > ----------------------------------------------------------- > > (Updated 2010-12-15 14:05:42) > > > Review request for Solid and Kevin Ottens. > > > Summary > ------- > > This patch adds a method to connect to the resumingFromSuspend signal > provided by the KDE Power Management system. In addition, I added a method to > detect its availability, to prevent applications from relying on it when it's > not available. > > P.S.: All of the new functions are @since 4.6, but I don't really know if it > is 4.6 material. Kevin, I leave the decision to you. > > > Diffs > ----- > > /trunk/KDE/kdelibs/solid/solid/powermanagement.h 1206691 > /trunk/KDE/kdelibs/solid/solid/powermanagement.cpp 1206691 > /trunk/KDE/kdelibs/solid/solid/powermanagement_p.h 1206691 > > Diff: http://svn.reviewboard.kde.org/r/6119/diff > > > Testing > ------- > > Compiles. > > > Thanks, > > Dario > >
_______________________________________________ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel