[opensource-dev] Review Request: STORM-680 (Avaline callers are added to the Recent list)

Seth ProductEngine slitovchuk at productengine.com
Wed Feb 9 06:03:31 PST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/146/#review349
-----------------------------------------------------------

Ship it!


- Seth


On Feb. 8, 2011, 11:26 a.m., Paul ProductEngine wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/146/
> -----------------------------------------------------------
> 
> (Updated Feb. 8, 2011, 11:26 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> -------
> 
> - When accepting an avaline call, add a caller to the recent list as AvalineListItem
> 
> - When adding item to the LLRecentPeople, check whether item with the same phone number exists and delete it if exists. This is need to avoid duplication in the Recent list of the panel People.
> 
> 
> This addresses bug STORM-680.
>     http://jira.secondlife.com/browse/STORM-680
> 
> 
> Diffs
> -----
> 
>   indra/newview/llavatarlist.h 33fc9ed99d29 
>   indra/newview/llavatarlist.cpp 33fc9ed99d29 
>   indra/newview/llrecentpeople.h 33fc9ed99d29 
>   indra/newview/llrecentpeople.cpp 33fc9ed99d29 
>   indra/newview/llvoicechannel.h 33fc9ed99d29 
>   indra/newview/llvoicechannel.cpp 33fc9ed99d29 
> 
> Diff: http://codereview.secondlife.com/r/146/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Paul
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110209/8518ff63/attachment.htm 


More information about the opensource-dev mailing list