[opensource-dev] Review Request: Fixes paste action in editor windows.
Ima Mechanique
Ima.Mechanique at blueyonder.co.uk
Sun May 13 22:13:30 PDT 2012
> 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20120514/e8ef1cd4/attachment-0001.htm
More information about the opensource-dev
mailing list