----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/579/#review1219 -----------------------------------------------------------
indra/llui/lltexteditor.h <http://codereview.secondlife.com/r/579/#comment1131> 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); - Oz Linden 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