<div class="gmail_quote">Attached the new patch<br><br><div class="gmail_quote">2010/3/23 Paul Vriens <span dir="ltr">&lt;<a href="mailto:paul.vriens.wine@gmail.com" target="_blank">paul.vriens.wine@gmail.com</a>&gt;</span><div>
<div></div><div class="h5"><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div>On 03/23/2010 02:08 PM, Sébastien Ramage wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Fixes bug #16595<br>
---<br>
dlls/user32/scroll.c | 5 +----<br>
dlls/user32/tests/scroll.c | 17 ++++++++++++++++-<br>
2 files changed, 17 insertions(+), 5 deletions(-)<br>
</blockquote>
<br></div>
Hi Sébastien<br>
<br>
Can comment too much on the patch but this patch doesn&#39;t apply cleanly to current Git. Please make sure that all patches are based on top of current.<div><br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
diff --git a/dlls/user32/scroll.c b/dlls/user32/scroll.c<br>
index 93f85f2..91adb74 100644<br>
--- a/dlls/user32/scroll.c<br>
+++ b/dlls/user32/scroll.c<br>
@@ -1105,11 +1105,8 @@ void SCROLL_TrackScrollBar( HWND hwnd, INT<br>
scrollbar, POINT pt )<br>
DispatchMessageW(&amp;msg );<br>
}<br>
if (!IsWindow( hwnd ))<br>
- {<br>
ReleaseCapture();<br>
- break;<br>
- }<br>
- } while (msg.message != WM_LBUTTONUP);<br>
+ } while (msg.message != WM_LBUTTONUP&amp;&amp; GetCapture() == hwnd);<br>
}<br>
</blockquote>
<br></div>
While you&#39;re at it please add a space before the &#39;&amp;&amp;&#39;.<br>
<br>
-- <br>
Cheers,<br><font color="#888888">
<br>
Paul.<br>
</font></blockquote></div></div></div><br>
</div><br>