----------------------------------------------------------- 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
