<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Mar 22, 2016 at 2:09 AM, Huw Davies <span dir="ltr"><<a href="mailto:huw@codeweavers.com" target="_blank">huw@codeweavers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">On Fri, Mar 18, 2016 at 11:59:52AM -0700, Adam Buchbinder wrote:<br>
> diff --git a/dlls/riched20/tests/richole.c b/dlls/riched20/tests/richole.c<br>
> index 7c6c5d5..de3c7a0 100644<br>
> --- a/dlls/riched20/tests/richole.c<br>
> +++ b/dlls/riched20/tests/richole.c<br>
> @@ -53,6 +53,7 @@ static HWND new_window(LPCSTR lpClassName, DWORD dwStyle, HWND parent)<br>
>    HWND hwnd = CreateWindowA(lpClassName, NULL,<br>
>                              dwStyle | WS_POPUP | WS_HSCROLL | WS_VSCROLL | WS_VISIBLE,<br>
>                              0, 0, 200, 60, parent, NULL, hmoduleRichEdit, NULL);<br>
> +  ok(hwnd != NULL, "class: %s, error: %d\n", lpClassName, (int) GetLastError());<br>
>    return hwnd;<br>
>  }<br>
><br>
> @@ -761,6 +762,66 @@ static void test_ITextRange_GetChar(void)<br>
>    ITextRange_Release(txtRge);<br>
>  }<br>
><br>
> +/* Helper function for testing ITextRange_ScrollIntoView */<br>
> +static void check_range(HWND w, ITextDocument* doc, int first, int lim,<br>
> +                        LONG bStart, int expected_nonzero) {<br>
<br>
</span>Please put the opening brace on a new line.<br>
<br></blockquote><div>Will do, but I'm a bit confused about this:</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
More importantly, I get a test failure:<br>
richole.c:785: Test failed: Moved scrollbar is at 60, should be 0. (Range 0-160.)<br></blockquote><div><br></div><div>Where? The TestBot completed okay in a variety of environments:</div><div><br></div><div><a href="https://testbot.winehq.org/JobDetails.pl?Key=21524">https://testbot.winehq.org/JobDetails.pl?Key=21524</a></div><div><br></div><div>and it works on my local workstation. The trace points to a helper function, which is being called with expected_nonzero = 0. There are four calls in test_ITextRange_ScrollIntoView which could be behind this; without logging information or a stack trace, I can't tell what's breaking.</div><div><br></div><div>How were you able to get the test to fail? I'd like to try and reproduce the issue myself. Thanks for the feedback, and sorry about the hassle--if there's some kind of local testing I should have done before sending it off, please let me know.</div><div><br></div><div>Adam Buchbinder</div></div></div></div>