----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/589/#review1245 -----------------------------------------------------------
indra/newview/llavataractions.cpp <http://codereview.secondlife.com/r/589/#comment1138> Shouldn't this check for a null return? - Oz Linden On July 2, 2012, 3:59 p.m., MartinRJ Fayray wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/589/ > ----------------------------------------------------------- > > (Updated July 2, 2012, 3:59 p.m.) > > > Review request for Viewer. > > > Description > ------- > > Repository is here: https://bitbucket.org/MartinRJ/storm-1893 > > I just added two calls to show() and setVisible() in LLAvatarActions::share > to bring the IM window to front even when it's minimized. > > > This addresses bug STORM-1893. > https://jira.secondlife.com/browse/STORM-1893 > > > Diffs > ----- > > indra/newview/llavataractions.cpp 4d9106153407 > > Diff: http://codereview.secondlife.com/r/589/diff/diff > > > Testing > ------- > > Please see the test plan in the Jira. > > > 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