> On May 11, 2012, 6:20 a.m., Oz Linden wrote: > > indra/llui/lltexteditor.h, line 242 > > <http://codereview.secondlife.com/r/579/diff/1/?file=7859#file7859line242> > > > > It would be much clearer when reading calls to this method if the > > parameter were an enum rather than a boolean. Consider the readability > > difference between: > > addLineBreakChar(FALSE); > > and > > addLineBreakChar(DoNotGroupLines); > >
This would require additional, and IMO, inelegant code to set an additional boolean variable in the function. A comment would make it equally, if not more, clear and not require unnecessary code or an enum. - Ima ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/579/#review1219 ----------------------------------------------------------- On May 5, 2012, 8:06 p.m., Ima Mechanique wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/579/ > ----------------------------------------------------------- > > (Updated May 5, 2012, 8:06 p.m.) > > > Review request for Viewer. > > > Description > ------- > > Fixes paste action in editor windows (notecards, scripts, etc.) so that > pasting text is a single action as far as Undo is concerned. > This is an updating of Satomi Ahn's patch to work with v3.3.1. > > > This addresses bug VWR-20553. > > > Diffs > ----- > > doc/contributions.txt 3142ebd0bc5a > indra/llui/lltexteditor.h 3142ebd0bc5a > indra/llui/lltexteditor.cpp 3142ebd0bc5a > > Diff: http://codereview.secondlife.com/r/579/diff/diff > > > Testing > ------- > > Tried pasting/undoing. > > > Thanks, > > Ima Mechanique > >
_______________________________________________ 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