----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115233/#review48065 -----------------------------------------------------------
src/plugins/skype/skypedatasource.cpp <https://git.reviewboard.kde.org/r/115233/#comment34024> I don't remember writing this... src/plugins/skype/skypedatasource.cpp <https://git.reviewboard.kde.org/r/115233/#comment34025> obviously we can't merge like this. src/plugins/skype/skypedatasource.cpp <https://git.reviewboard.kde.org/r/115233/#comment34026> This if isn't needed anymore src/plugins/skype/skypedatasource.cpp <https://git.reviewboard.kde.org/r/115233/#comment34027> it's better to use the SQL query API for doing this with .prepare and .bindValue() as that does automatic escaping. - David Edmundson On Jan. 22, 2014, 5:19 p.m., Alexandr Akulich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115233/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2014, 5:19 p.m.) > > > Review request for Telepathy, David Edmundson and Martin Klapetek. > > > Repository: libkpeople > > > Description > ------- > > Initial version of SkypeDataSource plugin. > UI for account selection is not yet implemented, so in current state plugin > doesn't works. (You can edit skypedatasource.cpp:67 and manualy set your > account) > > > Diffs > ----- > > src/plugins/CMakeLists.txt 3633676 > src/plugins/skype/CMakeLists.txt PRE-CREATION > src/plugins/skype/skype_kpeople_plugin.desktop PRE-CREATION > src/plugins/skype/skypedatasource.h PRE-CREATION > src/plugins/skype/skypedatasource.cpp PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/115233/diff/ > > > Testing > ------- > > > Thanks, > > Alexandr Akulich > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
