Review Request: Created haze AIM plugin

[Dario Freddi]

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


Looks overall OK to me, I'd like George to set the Ship It! though. I marked some code style issues I'd like to see fixed though.


haze/aim-main-options-widget.h
<http://git.reviewboard.kde.org/r/100689/#comment1910>

    Code style: Q_DISABLE_COPY should go under Q_OBJECT, and the asterisk should be attached to the variable name (Ui::AimMainOptionsWidget *m_ui)



haze/aim-main-options-widget.cpp
<http://git.reviewboard.kde.org/r/100689/#comment1909>

    Code style: don't indent brackets



haze/haze-aim-account.cpp
<http://git.reviewboard.kde.org/r/100689/#comment1911>

    Code style: don't use spaces right after and before parenthesis, and use just a space after a comma, and not before. The correct style should be: 
    
    registerSupportedParameter("server", QVariant::String);
    
    Please fix all occurrences of this


- Dario


On March 12, 2011, 2:10 p.m., Lasath Fernando wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100689/
> -----------------------------------------------------------
> 
> (Updated March 12, 2011, 2:10 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Summary
> -------
> 
> I created a AIM plugin, with basic and advanced options in one go this time.
> 
> It's in my personal clone
> http://quickgit.kde.org/?p=clones/telepathy-accounts-kcm-plugins/fernando/main.git&a=shortlog&h=refs/heads/aol-plugin-build
> 
> (branch called 'aol-plugin-build')
> 
> 
> Diffs
> -----
> 
>   haze/aim-main-options-widget.cpp PRE-CREATION 
>   haze/aim-main-options-widget.h PRE-CREATION 
>   haze/CMakeLists.txt a998a65 
>   haze/aim-main-options-widget.ui PRE-CREATION 
>   haze/aim-server-settings-widget.h PRE-CREATION 
>   haze/aim-server-settings-widget.cpp PRE-CREATION 
>   haze/aim-server-settings-widget.ui PRE-CREATION 
>   haze/haze-account-ui-plugin.cpp 4854636 
>   haze/haze-aim-account.h PRE-CREATION 
>   haze/haze-aim-account.cpp PRE-CREATION 
>   haze/yahoo-main-options-widget.cpp 108a023 
> 
> Diff: http://git.reviewboard.kde.org/r/100689/diff
> 
> 
> Testing
> -------
> 
> Edited options and made sure they were reflected in Empathy.
> 
> 
> Thanks,
> 
> Lasath
> 
>

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