Review Request: remove useless private classes, cleanup headers

George Kiagiadakis kiagiadakis.george at gmail.com
Wed Feb 16 16:02:17 CET 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100667/#review1457
-----------------------------------------------------------

Ship it!


Looks good


gabble/main-options-widget.h
<http://git.reviewboard.kde.org/r/100667/#comment1218>

    Accidental trailing space inserted here. Remove it.


- George


On Feb. 16, 2011, 2:55 p.m., Florian Reinhard wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100667/
> -----------------------------------------------------------
> 
> (Updated Feb. 16, 2011, 2:55 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Summary
> -------
> 
> remove useless private classes, cleanup headers
> 
> 
> Diffs
> -----
> 
>   butterfly/butterfly-account-ui-plugin.h 8c027d56d092b907348129a1211a7f7fca3b4436 
>   butterfly/butterfly-account-ui-plugin.cpp 5efa8afc18563697c8b19d51ab352b40289a010b 
>   butterfly/butterfly-account-ui.h 248f9467bc27dd0a277d177f66853419d5328f9a 
>   butterfly/butterfly-account-ui.cpp 67fc5f788bbee68a1c736da834cc9a98e2b13ee8 
>   butterfly/main-options-widget.h ec7ca343f8de5ececba4194138fb05f9040ac4e5 
>   butterfly/main-options-widget.cpp e8c5717545e35489ad37befd78c9772c6ae6b54a 
>   gabble/gabble-account-ui-plugin.h 76de025c15cb82cc98abbb7093fc75fc9b92ee90 
>   gabble/gabble-account-ui-plugin.cpp ae9f79a4f702c7b17ba75b1a09209a2b27783bca 
>   gabble/gabble-account-ui.h 2b828e31cac68444f45feed6ff2a1c20de016a1b 
>   gabble/gabble-account-ui.cpp 4a1ea987115ccfab42927cb1eccd36669b6ad658 
>   gabble/main-options-widget.h 7b04473b00c87370082f86e90c673207bf0b8fe2 
>   gabble/main-options-widget.cpp 333e0b49f0043491115b3adf6a25fd7ce22c8585 
>   gabble/proxy-settings-widget.h 55777a76f80c9f1475888b24be6ef85bb58b3b63 
>   gabble/proxy-settings-widget.cpp f68fe36877e8c684435c8d106177343804fe237e 
>   gabble/server-settings-widget.h 504797eb8820a0da8d7c34f13494d94950aa83e2 
>   gabble/server-settings-widget.cpp 9bc2fe285a6eceaa27d023d983c2cfce429339d4 
>   haze/haze-account-ui-plugin.h 28d7b430bff59bc28d22021fa44c8f213fb70469 
>   haze/haze-account-ui-plugin.cpp 2e5a386da637ac2fb390d33cbbd15137e7e44742 
>   haze/haze-icq-account.h d90c7898751237d9f60a74530f4b40922c995c18 
>   haze/haze-icq-account.cpp 5bc662a44f06a195929b4a2eee78a7dc667290ad 
>   haze/haze-yahoo-account.h 1f94f44e0a4fd2d689e0f4f9d80ba7a8b2ffd20b 
>   haze/haze-yahoo-account.cpp 96fda32f1390f2e1c91bafbfbdf187f426d7edcb 
>   haze/icq-main-options-widget.h ffe340b9d2e838138756519de1f0c83b8e3a7e0f 
>   haze/icq-main-options-widget.cpp 3faa1af9aec25ce5b4cc9ecb0465b033aa16f3a0 
>   haze/icq-server-settings-widget.h 6353269c36a338477c3cf8799b2efbe89cbebbf9 
>   haze/icq-server-settings-widget.cpp 3d08d50d471c0bda0b39c71ca2d780578466a490 
>   haze/yahoo-main-options-widget.h a694145ec6d26930c80aaf737d84f8f16569a624 
>   haze/yahoo-main-options-widget.cpp 10501eb6301a470a6ddac7b06f816334d226b0a0 
>   idle/advanced-options-widget.h 4ff2cc0e8e9e1b3c51b15e11d9385ca376f2364c 
>   idle/advanced-options-widget.cpp 295d8c12ec26c83841e9967b0dcc0a8fce603125 
>   idle/idle-account-ui-plugin.h 98eaf4e732791ae144a7b55801a88c6c7ddcc58f 
>   idle/idle-account-ui-plugin.cpp 87aca1715ceb0636d955e0f2e26bf70f36e55be1 
>   idle/idle-account-ui.h fa6516777c6cd564fbdeffd97a3e99bea0fc56cf 
>   idle/idle-account-ui.cpp 7db617649534be1b496e81bdcede28752056ebdc 
>   idle/main-options-widget.h 513d37243e0e5bc904a0f4f467d4408f8599a584 
>   idle/main-options-widget.cpp 2bcc600cd837d5b59ed72966406e28f32500c06e 
>   rakia/rakia-account-ui-plugin.cpp 9a09052ca1ed71d7a25c38999ec7dd1cdd68ab4a 
>   rakia/rakia-account-ui.cpp 97c78422506a6ee56a72b9537168d48f53c082bb 
>   rakia/rakia-advanced-options-widget.h 0b619d400da7ea4168077dd41c15936c6988d0c7 
>   rakia/rakia-advanced-options-widget.cpp c6fa94ab360c06ec4f50e6740c9941f62b112a6a 
>   rakia/rakia-main-options-widget.h efcc07a719a3a7e90f137c194a047b229f83eadd 
> 
> Diff: http://git.reviewboard.kde.org/r/100667/diff
> 
> 
> Testing
> -------
> 
> compiles, dialogs open.
> 
> 
> Thanks,
> 
> Florian
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110216/b09292df/attachment.htm 


More information about the KDE-Telepathy mailing list