> On Jan. 23, 2012, 8:52 a.m., Oz Linden wrote: > > indra/newview/llimview.cpp, lines 2462-2470 > > <http://codereview.secondlife.com/r/536/diff/3/?file=7590#file7590line2462> > > > > 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) > >
There are two issues: 1) Clarity of code for future programmers (I think having the flag variable helps here) 2) Does your folded-in solution work properly for all 4 cases of VoiceCalls true/false and friend/not-friend? Am I having a dumb moment or does the case of VoiceCalls=false and is-friend=false not work properly here (!(false && false)) = false, when it should be true? - Jonathan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/536/#review1151 ----------------------------------------------------------- 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