Review Request: Maintain a list of recently used rooms in JoinChatroomDialog

Vishesh Handa handa.vish at gmail.com
Wed Mar 14 11:18:20 UTC 2012


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


Just a cursory glance.


dialogs/join-chat-room-dialog.h
<http://git.reviewboard.kde.org/r/104269/#comment9112>

    Why not use a hash? It's not like you require the string lists to be sorted.



dialogs/join-chat-room-dialog.cpp
<http://git.reviewboard.kde.org/r/104269/#comment9111>

     I know this was like this before, but why have the closing parenthesis on the next line?



dialogs/join-chat-room-dialog.cpp
<http://git.reviewboard.kde.org/r/104269/#comment9114>

    This could potentially insert an <key, QStringList()> in the map.
    
    Maybe you should check is the string list if empty. Unless, that's what you wanted.


- Vishesh Handa


On March 14, 2012, 11 a.m., Dominik Cermak wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104269/
> -----------------------------------------------------------
> 
> (Updated March 14, 2012, 11 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Upon joining a room (more exact upon clicking "Ok") the room is saved in the recently used list for this account.
> ATM it remembers up to 8 rooms and single rooms can be removed as well as the whole list can be cleared.
> 
> The complete list is showed in a new tab and the rooms are used for completion in the lineEdit.
> 
> 
> This addresses bug 291711.
>     http://bugs.kde.org/show_bug.cgi?id=291711
> 
> 
> Diffs
> -----
> 
>   dialogs/join-chat-room-dialog.h 686fe312dc243e49feafa793106f782abb9f9f9b 
>   dialogs/join-chat-room-dialog.cpp 026c5e4f3d1e068739c40a4b014daf84e8a77f8b 
>   dialogs/join-chat-room-dialog.ui 839d644399b5c9031a4c4840c6e8573309c4b859 
> 
> Diff: http://git.reviewboard.kde.org/r/104269/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> recent tab
>   http://git.reviewboard.kde.org/r/104269/s/462/
> recent completion
>   http://git.reviewboard.kde.org/r/104269/s/463/
> 
> 
> Thanks,
> 
> Dominik Cermak
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120314/e27a623d/attachment.html>


More information about the KDE-Telepathy mailing list