> On April 30, 2012, 9:48 p.m., David Edmundson wrote: > > src/kwhiteboard-handler.cpp, line 170 > > <http://git.reviewboard.kde.org/r/104779/diff/2/?file=59737#file59737line170> > > > > I know this isn't your code. > > > > WTF is going on with this variable. > > > > If you could tidy this sort of thing up in a future patch that will > > really help you if you're hacking on it.
Yeah I forgot about that, it was a stupid hack to get 2 different connections, and I forgot to fix it before committing :P I will fix it one of these days ;) - Daniele Elmo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104779/#review13175 ----------------------------------------------------------- On April 30, 2012, 8:44 p.m., Puneet Goyal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104779/ > ----------------------------------------------------------- > > (Updated April 30, 2012, 8:44 p.m.) > > > Review request for Telepathy and Daniele Elmo Domenichelli. > > > Description > ------- > > This patch makes use of Tp-Qt factories to set up the telepathy client > registrar in main.cpp > I have used the codebase at > http://quickgit.kde.org/index.php?p=kwhiteboard.git&a=summary > > > Diffs > ----- > > src/kwhiteboard-handler.cpp d78b09e > src/main.cpp 7e721f3 > > Diff: http://git.reviewboard.kde.org/r/104779/diff/ > > > Testing > ------- > > I have tested it and it runs successfully. > > > Thanks, > > Puneet Goyal > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
