[Konversation-devel] Review Request 109588: Handle multiple identities with the same name

[Eike Hein]

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



src/application.cpp
<http://git.reviewboard.kde.org/r/109588/#comment22017>

    Itsy-bitsy style issue: No newline before opening brace.



src/application.cpp
<http://git.reviewboard.kde.org/r/109588/#comment22016>

    You're no longer checking for -1 here, which means the at() in the next line is going to crash if IdentityId isn't at least 0. This couldn't happen without someone manually editing the config file, but people sometimes do ...
    
    What you could do though is just change the default to 0 and drop the else case, because if I'm thinking right, the first list entry is always going to be the default identity (it's created by the Preferences constructor and the UI doesn't allow it to be deleted, and since the identities are written out and read in in list order, it'll still be the first list entry after every store+load cycle).


- Eike Hein


On March 19, 2013, 8:15 p.m., José Millán Soto wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109588/
> -----------------------------------------------------------
> 
> (Updated March 19, 2013, 8:15 p.m.)
> 
> 
> Review request for Konversation and Eike Hein.
> 
> 
> Description
> -------
> 
> Correct handling of multiple identities with the same name
> 
> - Identity id is used between ServerGroup dialog and Identity dialog
> - Save order is using to store the Identity of a ServerGroup
> 
> 
> This addresses bug 311300.
>     https://bugs.kde.org/show_bug.cgi?id=311300
> 
> 
> Diffs
> -----
> 
>   src/irc/servergroupdialog.cpp 75655b3 
>   src/application.cpp 71e0f1f 
> 
> Diff: http://git.reviewboard.kde.org/r/109588/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> José Millán Soto
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konversation-devel/attachments/20130319/325315ae/attachment.html>