----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113798/#review43449 -----------------------------------------------------------
Thanks for implementing that. Indeed the TODO didn't mean adding a method to QApplication, but it was about the member that was there in qt4, and that is now in QXcbConnection. The optional dependency on QtGui breaks the rules for tier1 addons, though. Maybe we need to move KDBusService up to... the KService framework maybe? (Note that there are more todos about startupId stuff, in KDBusService::Activate etc. This would probably be easier to implement in a higher framework too. - David Faure On Nov. 11, 2013, 4:37 p.m., Alex Merry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113798/ > ----------------------------------------------------------- > > (Updated Nov. 11, 2013, 4:37 p.m.) > > > Review request for KDE Frameworks, David Faure and Kevin Ottens. > > > Repository: kdelibs > > > Description > ------- > > KDBusService: pass the desktop startup ID when calling Activate > > We use a bit of private API to ask the xcb platform plugin for it > directly. > > The TODO in the original code suggested getting a method in QApplication (or > one of its ancestors) to get the startup id, but I think that's very unlikely > to be accepted. > > There's still the hooks to make use of the value on the other side to sort > out, though. > > > Diffs > ----- > > tier1/kdbusaddons/CMakeLists.txt 78cc44333574355ff8504481fcb9c88cfc90daf5 > tier1/kdbusaddons/src/CMakeLists.txt > 0509015afd2d24d34f85a7d6fd786092820814bf > tier1/kdbusaddons/src/config-kdbusaddons.h.cmake PRE-CREATION > tier1/kdbusaddons/src/kdbusservice.cpp > b773c80b30c6ee39d6d8b4d8c962b83dbd87f7d4 > > Diff: http://git.reviewboard.kde.org/r/113798/diff/ > > > Testing > ------- > > Builds, tests pass. A quick-hack modification of the autotest, along with > some hacked-in debug statements, show the value is getting passed properly. > > > Thanks, > > Alex Merry > >
_______________________________________________ Kde-frameworks-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
