----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102668/#review6666 -----------------------------------------------------------
main-widget.cpp <http://git.reviewboard.kde.org/r/102668/#comment5899> What do you mean with that? There is a check below: if (item){ ... } Besides, this is copy-paste 1-1 from the other functions. service-availability-checker.cpp <http://git.reviewboard.kde.org/r/102668/#comment5900> QObject parent-child relationship deletes it, as David said service-availability-checker.cpp <http://git.reviewboard.kde.org/r/102668/#comment5901> Um, ok, I'll rewrite this part. - George On Sept. 19, 2011, 6:07 p.m., George Kiagiadakis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102668/ > ----------------------------------------------------------- > > (Updated Sept. 19, 2011, 6:07 p.m.) > > > Review request for Telepathy and Martin Klapetek. > > > Summary > ------- > > This patch adds a button and a context menu action to every contact in the > contact list that allows users to start desktop sharing with the selected > contact. The patch also includes a class that checks for the availability of > krfb on the system, as there is no way of determining that via telepathy. > > > Diffs > ----- > > CMakeLists.txt 7c463ff50946c4e9b307ed98e41dbf6b4f84a5be > contact-overlays.h 43bcf237397fc2770d834cb29275da17456cf4e2 > contact-overlays.cpp 026e3d34b7ceb92e3415a558d15c73293648a2e6 > main-widget.h 6843af5af4b0b7d32eccb835793f0a0caec15377 > main-widget.cpp 98abf2968bc645f4c78e99806ca264b17e5ccbfc > models/accounts-model.h 40b915b78637dc4d4e2b3d85aa559d4f0b465345 > models/accounts-model.cpp 75d224ff2e332bbd3f96293854396d1a28e23ff9 > models/contact-model-item.h cc2a59f713c9de59ecfe95558c7dbed5acf82570 > models/contact-model-item.cpp c32b4ffe6502af074bddd6687a623c2d5673508d > service-availability-checker.h PRE-CREATION > service-availability-checker.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/102668/diff > > > Testing > ------- > > Successfully started a desktop sharing session between my laptop and my > desktop with krfb. However the client side failed to show the desktop after > approving the channel (this is either a bug in krfb - most likely - or > something in gabble; I'll inspect it) > > > Thanks, > > George > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
