Review Request: Add spell dictionary combobox to text-ui

David Edmundson kde at davidedmundson.co.uk
Wed Nov 30 19:33:06 UTC 2011


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

Ship it!



app/chat-window.h
<http://git.reviewboard.kde.org/r/103296/#comment7286>

    Indent please.



app/chat-window.h
<http://git.reviewboard.kde.org/r/103296/#comment7287>

    What on earth is going on with the documentation here.
    
    /** Description of the method */
    void theMethod();
    
    not below the line.
    
    If we ran this through doxygen it would make no sense.
    
    (tbh, this looks like the fault of an earlier person, I blame whoever made the text-ui)


- David Edmundson


On Nov. 30, 2011, 2:13 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103296/
> -----------------------------------------------------------
> 
> (Updated Nov. 30, 2011, 2:13 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> This is a finished patch of Alin's addition of combobox to the text-ui. All credit goes to him.
> 
> 
> Diffs
> -----
> 
>   app/chat-window.h 6cbc425 
>   app/chat-window.cpp d423965 
>   app/chatwindow.rc c24ddcc 
>   lib/chat-widget.h 31f13d6 
>   lib/chat-widget.cpp 2971b67 
> 
> Diff: http://git.reviewboard.kde.org/r/103296/diff/diff
> 
> 
> Testing
> -------
> 
> Works.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20111130/7fd02e08/attachment-0001.html>


More information about the KDE-Telepathy mailing list