Review Request 119086: Added disabling/enabling logging for specified entities
Dan Vrátil
dvratil at redhat.com
Tue Jul 8 06:14:32 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119086/#review61873
-----------------------------------------------------------
The TPLQT_TPL_IS_VERSION macro is defined in log-manager.h, which is public, so you can safely use it here too. Just do something like
```
#if TPLQT_TPL_IS_VERSION(0, 9, 0)
code
#else
return defualtValue;
#endif
```
In all three TpLoggerPlugin methods implementations.
- Dan Vrátil
On July 7, 2014, 7:37 p.m., Marcin Ziemiński wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119086/
> -----------------------------------------------------------
>
> (Updated July 7, 2014, 7:37 p.m.)
>
>
> Review request for Telepathy.
>
>
> Repository: ktp-common-internals
>
>
> Description
> -------
>
> Added an option to disable or enable logging for entities with an implementation in tp-logger-plugin.
>
>
> Diffs
> -----
>
> KTp/Logger/plugins/tplogger/tp-logger-plugin.cpp 8c0cb16
> KTp/Logger/abstract-logger-plugin.h 6db98d4
> KTp/Logger/abstract-logger-plugin.cpp 6d387bb
> KTp/Logger/log-manager.h c5f92d6
> KTp/Logger/log-manager.cpp 7ce4a12
> KTp/Logger/plugins/tplogger/tp-logger-plugin.h 346c233
>
> Diff: https://git.reviewboard.kde.org/r/119086/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Marcin Ziemiński
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140708/2a354793/attachment.html>
More information about the KDE-Telepathy
mailing list