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



indra/newview/llimview.cpp
<http://codereview.secondlife.com/r/536/#comment1106>

    Couldn't this all be folded into the 'if' without the flag variable as:
    
    if (  !LLMuteList::getInstance()->isMuted(other_participant_id, 
LLMute::flagTextChat)
        && ! (gSavedSettings.getBOOL("VoiceCallsFriendsOnly") && 
(LLAvatarTracker::instance().getBuddyInfo(other_participant_id) == NULL))
    
    (that probably won't format well)
    


- Oz Linden


On Jan. 20, 2012, 11:29 a.m., Jonathan Yap wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/536/
> -----------------------------------------------------------
> 
> (Updated Jan. 20, 2012, 11:29 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Description
> -------
> 
> With avatar A: check on Preferences->Privacy->Only friends and groups can 
> call or IM me
> 
> Get together with:
> Avatar B (not a friend of A)
> Avatar C (a friend of A)
> 
> Using avatar B establish an ad-hoc chat with A and C and write a message
> 
> Observed result: Avatar A and C receive the message
> Expected result: Only avatar C receives the message
> 
> 
> This addresses bug STORM-1795.
>     http://jira.secondlife.com/browse/STORM-1795
> 
> 
> Diffs
> -----
> 
>   doc/contributions.txt 4982ab91ef6a 
>   indra/newview/llimview.cpp 4982ab91ef6a 
> 
> Diff: http://codereview.secondlife.com/r/536/diff/diff
> 
> 
> Testing
> -------
> 
> See test plan in jira
> 
> 
> Thanks,
> 
> Jonathan Yap
> 
>

_______________________________________________
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

Reply via email to