----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/159/#review382 -----------------------------------------------------------
Looks good, except for a seemingly redundant code piece. indra/newview/llpanelpeople.cpp <http://codereview.secondlife.com/r/159/#comment270> What do you need the buddies_uuids for? It seems redundant to me. - Vadim On Feb. 21, 2011, 4:57 p.m., Seth ProductEngine wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/159/ > ----------------------------------------------------------- > > (Updated Feb. 21, 2011, 4:57 p.m.) > > > Review request for Viewer. > > > Summary > ------- > > - Added creating own calling card for the user to be able to share it with > other residents. > - Moved calling cards synchronization with friends list to the viewer start > up. Previously synchronized upon opening the Friends tab in People side panel. > - Calling cards for non-friends are not removed upon calling cards > synchronization with friends list. > - Enabled "Share" menu item for calling cards in inventory. > > > This addresses bug STORM-28. > http://jira.secondlife.com/browse/STORM-28 > > > Diffs > ----- > > indra/newview/llfriendcard.h c10d5e37db1e > indra/newview/llfriendcard.cpp c10d5e37db1e > indra/newview/llinventoryfunctions.cpp c10d5e37db1e > indra/newview/llpanelpeople.cpp c10d5e37db1e > > Diff: http://codereview.secondlife.com/r/159/diff > > > Testing > ------- > > > Thanks, > > Seth > >
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
