(resend) Scroll window fix

Dmitry Timoshkov dmitry at codeweavers.com
Fri Mar 2 21:19:13 CST 2007


"Aric Stewart" <aric at codeweavers.com> wrote:

> +    SetRect( &rc, 0,40, 100,60);
> +    SetRect( &cliprc, 0,0, 100,100);
> +    ScrollWindowEx( hwnd1, 0, -25, &rc, &cliprc, hrgn, &rcu, SW_INVALIDATE);
> +    SetRectRgn( tmprgn, 0,15,98,35);
> +    CombineRgn( exprgn, exprgn, tmprgn, RGN_OR);
> +    SetRectRgn( tmprgn, 0, 40, 98, 60);
> +    CombineRgn( exprgn, exprgn, tmprgn, RGN_OR);
> +    ok( EqualRgn( exprgn, hrgn), "wrong update region in excessive scroll\n");

Under which version of Windows this was tested? According to test.winehq.org this test
doesn't pass under any of Win98, NT4, Win2000 nor XP invalidating the (now committed) fix.
I'd vote to revert this patch as obviously incorrect.

-- 
Dmitry.



More information about the wine-devel mailing list