Re: [PATCH] user32: end scrollbar tracking if the mouse capture is lost (retry)

2010-03-25 Thread Sébastien Ramage
0 Subject: [PATCH] user32: end scrollbar tracking if the mouse capture is lost MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="1.6.3.3" This is a multi-part message in MIME format. --1.6.3.3 Content-Type: text/plain; charset=UTF-8; format=fixed Content-Tran

Re: [PATCH] user32: end scrollbar tracking if the mouse capture is lost (retry)

2010-03-25 Thread Paul Vriens
On 03/25/2010 09:56 AM, Sébastien Ramage wrote: You have DOS line endings in your patch. A simple dos2unix fixes this. Thank you Attached the patch with unix line endings Seb Patch should go to wine-patches. Please mark as [Try x] and tell what the difference to the previous version is.

Re: [PATCH] user32: end scrollbar tracking if the mouse capture is lost (retry)

2010-03-25 Thread Paul Vriens
On 03/25/2010 09:15 AM, Sébastien Ramage wrote: --- dlls/user32/scroll.c | 5 + dlls/user32/tests/scroll.c | 16 +++- 2 files changed, 16 insertions(+), 5 deletions(-) Hi You can see at http://source.winehq.org/patches/ that your patch '59740' had an apply failure. And fair eno

Re: [PATCH] user32: end scrollbar tracking if the mouse capture is lost

2010-03-23 Thread Paul Vriens
On 03/23/2010 02:08 PM, Sébastien Ramage wrote: Fixes bug #16595 --- dlls/user32/scroll.c | 5 + dlls/user32/tests/scroll.c | 17 - 2 files changed, 17 insertions(+), 5 deletions(-) Hi Sébastien Can comment too much on the patch but this patch doesn't apply cleanly to curren

Re: [PATCH] user32: end scrollbar tracking if the mouse capture is lost

2010-03-23 Thread Dmitry Timoshkov
Sébastien Ramage wrote: > -} while (msg.message != WM_LBUTTONUP); > +} while (msg.message != WM_LBUTTONUP&& GetCapture() == hwnd); Please fix the spacing above. > +SendMessage( hScroll, WM_LBUTTONDOWN, 0, 0x10001); 0 for lParam would work just fine. A magic value needs either expl