> On July 14, 2011, 9:03 a.m., Boroondas Gupte wrote: > > indra/newview/llinventorybridge.cpp, line 2502 > > <http://codereview.secondlife.com/r/396/diff/1/?file=6693#file6693line2502> > > > > We could just use > > if (!isItemRemovable()) > > here, couldn't we?
You are right. Changed according to your commentaries. - Paul ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/396/#review877 ----------------------------------------------------------- On July 19, 2011, 11:11 a.m., Paul ProductEngine wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/396/ > ----------------------------------------------------------- > > (Updated July 19, 2011, 11:11 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 7c7d01be2310 > > 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
