<div dir="ltr"><span class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">There's no need to add this to d3drm.c and d3drm3.c. The versions in<br>
d3drm1.c, d3drm2.c and d3drm3.c are enough. Ideally d3drm.c will go<br>
away at some point.<span><br></span></blockquote></span><div>I was 
planning in removing them from the start - but I must've forgotten to 
remove them while sending the patch over. I'll do the same and resend 
the patches. <br></div><span class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>
> +    hr = Direct3DRMCreate(&d3drm);<br>
> +    ok(hr == D3DRM_OK, "Cannot get IDirect3DRM interface (hr = %x)\n", hr);<br>
> +<br>
> +    test_qi("d3drm_qi", (IUnknown *)d3drm, &IID_IDirect3DRM, tests, sizeof(tests) / sizeof(*tests));<br>
> +<br>
> +    IDirect3DRM_Release(d3drm);<br>
</span>This test is version independent because there is only one way to<br>
create the d3drm object. You can just keep it in d3drm.c for now, or<br>
move it to d3drm1.c in a separate patch if you want. <br></blockquote></span><div>So I should remove this test from d3drm1.c, d3drm2.c and d3drm3.c aswell? I'll keep this test in d3drm.c as it is.<br></div><div>I agree that we should keep version independent tests in d3drm.c, and the rest in their respective files.<br></div><span class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>
> --- a/dlls/d3drm/tests/Makefile.in<br>
> +++ b/dlls/d3drm/tests/Makefile.in<br>
> @@ -2,4 +2,7 @@ TESTDLL   = d3drm.dll<br>
>  IMPORTS   = d3drm dxguid ddraw user32 uuid<br>
>  C_SRCS = \<br>
>       d3drm.c \<br>
> +     d3drm1.c \<br>
> +     d3drm2.c \<br>
> +     d3drm3.c \<br>
>       vector.c<br>
</span>This doesn't match the Makefile.in file I have. You have a reordered<br>
IMPORTS line. This is why the testbot failed. Do you have other<br>
patches in your tree?<br>
</blockquote></span><div>I might've modified them a couple weeks ago. I'll resend the patch with the proper ordered Makefile.in <br></div><span class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +* Copyright 2012 Andr� Hentschel<br>
This shows up wrong in my mail client. It may be a setup issue on my<br>
side. It shows up OK on <a href="http://source.winehq.org/patches/data/110484" target="_blank">http://source.winehq.org/patches/data/110484</a><br>
when viewed with the browser, but if I download the patch with wget it<br>
is broken as well. Please make sure your text editor is configured to<br>
save the files as UTF8.<br>
<br></blockquote></span><div>I used visual studio to copy these lines from d3drm.c. I'll fix it accordingly and resend the patch. </div></div>