Review Request 115618: KTp-KDED-Module: Ported to KTp::methods. Performed some cleanup.

Commit Hook null at kde.org
Tue Feb 11 12:03:23 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115618/#review49542
-----------------------------------------------------------


This review has been submitted with commit 645d85f52d69ffc22529eb0853ebf061e5f4e4fd by Alexandr Akulich to branch master.

- Commit Hook


On Feb. 10, 2014, 10:19 a.m., Alexandr Akulich wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115618/
> -----------------------------------------------------------
> 
> (Updated Feb. 10, 2014, 10:19 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Repository: ktp-kded-module
> 
> 
> Description
> -------
> 
> Manually constructed factories and managers replaced by KTp ones.
> Such change let us to cleanup code and let freely use KTp methods without dbus traffic duplication (which is occurs in last commit).
> 
> As according things:
>   Reworked includes.
>   Fixed a lot of coding style issues.
>   ContactNotify: Removed "using namespace KTp" from header.
> 
> 
> Diffs
> -----
> 
>   error-handler.cpp 2b84784 
>   screensaveraway.cpp 6c47cfb 
>   telepathy-kded-module-plugin.cpp 43cb383 
>   telepathy-module.h ec08c6e 
>   telepathy-module.cpp bf154c6 
>   telepathy-mpris.cpp 1c7b98c 
>   autoaway.h ac42718 
>   autoaway.cpp 79f62f0 
>   autoconnect.h aeb4b54 
>   autoconnect.cpp b779426 
>   contact-request-handler.h 69d2a1e 
>   contact-request-handler.cpp 1b13194 
>   contactnotify.h 4dc71e4 
>   contactnotify.cpp 9f001f4 
>   error-handler.h dd62822 
> 
> Diff: https://git.reviewboard.kde.org/r/115618/diff/
> 
> 
> Testing
> -------
> 
> Compiled successfully. Seems to work, but not tested a lot.
> 
> 
> Thanks,
> 
> Alexandr Akulich
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140211/cdc222ab/attachment.html>


More information about the KDE-Telepathy mailing list