----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/396/#review877 -----------------------------------------------------------
indra/newview/llinventorybridge.cpp <http://codereview.secondlife.com/r/396/#comment917> Why store this in a variable when the result is only used once and the call is this simple? (I assume the result of this call wouldn't change between here where the value is stored and the below where it's used.) indra/newview/llinventorybridge.cpp <http://codereview.secondlife.com/r/396/#comment918> We could just use if (!isItemRemovable()) here, couldn't we? - Boroondas On July 14, 2011, 2:15 a.m., Paul ProductEngine wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/396/ > ----------------------------------------------------------- > > (Updated July 14, 2011, 2:15 a.m.) > > > Review request for Viewer. > > > Summary > ------- > > - Disable "Delete" menu item in case selected folder contains non-removable > items. > > > This addresses bug storm-519. > http://jira.secondlife.com/browse/storm-519 > > > Diffs > ----- > > indra/newview/llinventorybridge.cpp 68ad362920c1 > > Diff: http://codereview.secondlife.com/r/396/diff > > > Testing > ------- > > > Thanks, > > Paul > >
_______________________________________________ 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
