> On June 30, 2011, 11:02 a.m., Vadim ProductEngine wrote: > > indra/newview/llpanelwearing.cpp, lines 297-302 > > <http://codereview.secondlife.com/r/370/diff/3/?file=3241#file3241line297> > > > > Looks a bit confusing. > > Lance Corrimal wrote: > but it's elegant!
Just needs some parens: text += (iter != data.end()) ? item->getName() + "\n" : item->getName(); But otherwise it looks just like a a lot of lines I've run across in the past, and with the comment it's not all that difficult to understand. - Cron ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/370/#review834 ----------------------------------------------------------- On June 30, 2011, 1:27 a.m., Jonathan Yap wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/370/ > ----------------------------------------------------------- > > (Updated June 30, 2011, 1:27 a.m.) > > > Review request for Viewer. > > > Summary > ------- > > Add a feature on the "Wearing TAB" where users could copy to the clipboard > everything you see in the "Wearing TAB". This would make the blogging > communities life soooooo much easier instead of having to type out all that > information. > > The label on this button needs input from someone on the XD team. > > I would like to know if my code for adding a CR to the end of every line but > the last one could be done in a more elegant way. > > > This addresses bug STORM-1459. > http://jira.secondlife.com/browse/STORM-1459 > > > Diffs > ----- > > doc/contributions.txt f9864a43ddf0 > indra/newview/llpanelwearing.h f9864a43ddf0 > indra/newview/llpanelwearing.cpp f9864a43ddf0 > indra/newview/skins/default/xui/en/menu_wearing_gear.xml f9864a43ddf0 > > Diff: http://codereview.secondlife.com/r/370/diff > > > Testing > ------- > > Clicked on Send to Clipboard button and was able to paste results into an > editor. > > > Thanks, > > Jonathan > >
_______________________________________________ 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
