Review Request 119086: Added disabling/enabling logging for specified entities

[Dan Vrátil]

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



KTp/Logger/log-manager.cpp
<https://git.reviewboard.kde.org/r/119086/#comment42798>

    const bool



KTp/Logger/log-manager.cpp
<https://git.reviewboard.kde.org/r/119086/#comment42799>

    coding style:
    
    if (!result) {
        return false;
    }



KTp/Logger/log-manager.cpp
<https://git.reviewboard.kde.org/r/119086/#comment42796>

    Remove the whitespace at the end



KTp/Logger/log-manager.cpp
<https://git.reviewboard.kde.org/r/119086/#comment42795>

    const bool



KTp/Logger/log-manager.cpp
<https://git.reviewboard.kde.org/r/119086/#comment42797>

    Coding style



KTp/Logger/log-manager.cpp
<https://git.reviewboard.kde.org/r/119086/#comment42805>

    Coding style



KTp/Logger/plugins/tplogger/tp-logger-plugin.cpp
<https://git.reviewboard.kde.org/r/119086/#comment42800>

    Remove whitespace at the end


- Dan Vrátil


On July 2, 2014, 2:05 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 2, 2014, 2:05 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/abstract-logger-plugin.h 6db98d4 
>   KTp/Logger/log-manager.h c5f92d6 
>   KTp/Logger/log-manager.cpp 7ce4a12 
>   KTp/Logger/plugins/tplogger/tp-logger-plugin.h 346c233 
>   KTp/Logger/plugins/tplogger/tp-logger-plugin.cpp 8c0cb16 
> 
> 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/20140702/03878114/attachment-0001.html>