----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/583/#review1230 -----------------------------------------------------------
No issues with the code so far, but from inspection I don't think this code will block lights for an avatar that is already muted -- it only blocks lights at the moment you mute another avatar. You'll need to add a check to LLPipeline::setLight that verifies the object is not on the mute list before inserting the object onto gPipeline.mLights. - David Parks On June 18, 2012, 11:04 a.m., MartinRJ Fayray wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/583/ > ----------------------------------------------------------- > > (Updated June 18, 2012, 11:04 a.m.) > > > Review request for Viewer and David Parks. > > > Description > ------- > > An avatar currently gets derendered/invisible when the user blocks it. > The local lights attached to that avatar have to become erased too. > Repository: https://bitbucket.org/MartinRJ/storm-1878 > > > This addresses bug STORM-1878. > https://jira.secondlife.com/browse/STORM-1878 > > > Diffs > ----- > > indra/newview/llmutelist.cpp 29143d1fc6fa > indra/newview/pipeline.h 29143d1fc6fa > indra/newview/pipeline.cpp 29143d1fc6fa > > Diff: http://codereview.secondlife.com/r/583/diff/diff > > > Testing > ------- > > Tested with 1-6 local attached lights on the target avatar. > Blocking/muting the avatar also disabled the attached lights in all tests > successfully. > > > Thanks, > > MartinRJ Fayray > >
_______________________________________________ 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