<div dir="auto"><div>Hi Huw,</div><div dir="auto"><br></div><div dir="auto">yes I saw the failure, though fixing it only requires to remove a todo_wine. My idea was to await, if you have some other points which need to be changed about the patch series, so I can resend with those changes and the fixed test, to not spam the mailing list too much. </div><div dir="auto"><br></div><div dir="auto">Bernhard<br><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">Huw Davies <<a href="mailto:huw@codeweavers.com">huw@codeweavers.com</a>> schrieb am Mo., 24. Jan. 2022, 09:30:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Jan 13, 2022 at 12:34:14AM +0100, Bernhard Kölbl wrote:<br>
> The goal of these tests is to show, that the memory layout<br>
> of hstring_private is different on Windows, than currently<br>
> used in Wine. This creates issues with the WinRT SDK, which<br>
> allocates HSTRINGs without using the functions provided by<br>
> the combase dll.<br>
> <br>
> Signed-off-by: Bernhard Kölbl <<a href="mailto:besentv@gmail.com" target="_blank" rel="noreferrer">besentv@gmail.com</a>><br>
> ---<br>
>  dlls/combase/tests/string.c | 80 +++++++++++++++++++++++++++++++++++++<br>
>  1 file changed, 80 insertions(+)<br>
> <br>
> diff --git a/dlls/combase/tests/string.c b/dlls/combase/tests/string.c<br>
> index 5ebf669a426..713f08d342b 100644<br>
> --- a/dlls/combase/tests/string.c<br>
> +++ b/dlls/combase/tests/string.c<br>
> @@ -479,6 +479,85 @@ static void test_trim(void)<br>
>      ok(WindowsDeleteString(str1) == S_OK, "Failed to delete string\n");<br>
>  }<br>
<br>
Did you see that this had test failures?<br>
<br>
<a href="https://testbot.winehq.org/JobDetails.pl?Key=105204" rel="noreferrer noreferrer" target="_blank">https://testbot.winehq.org/JobDetails.pl?Key=105204</a><br>
<br>
Huw.<br>
</blockquote></div></div></div>