[TRY 4] [PATCH] user32: horizontal and vertical scrolling of selection in edit control

Dmitry Timoshkov dmitry at codeweavers.com
Mon Mar 29 05:58:49 CDT 2010


Henry Kroll III <henry at comptune.com> wrote:

> +    if (es->region_posx < 0)EDIT_EM_LineScroll(es,-10,0);
> +    else if (es->region_posx > 0)EDIT_EM_LineScroll(es,10,0);
> +    if (es->region_posy < 0)EDIT_EM_LineScroll(es,0,-1);
> +    else if (es->region_posy > 0)EDIT_EM_LineScroll(es,0,1);

Where do -/+10,-/+1 come from? A test case wouldn't hurt. Also please add
some spacing and try to follow the indentation style of existing code.

-- 
Dmitry.



More information about the wine-devel mailing list