<div dir="ltr"><div><div><div>Thanks for the review! Will keep that in mind for the next patches.<br></div><br></div>Cheers,<br></div>Aaryaman<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 12, 2016 at 6:47 PM, Henri Verbeet <span dir="ltr"><<a href="mailto:hverbeet@gmail.com" target="_blank">hverbeet@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 11 April 2016 at 20:04, Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com">jem456.vasishta@gmail.com</a>> wrote:<br>
> +        hr = IDirect3DRM_CreateObject(d3drm1, NULL, NULL, tests[i].iid, (void **)&unknown);<br>
> +        todo_wine ok(hr == D3DRMERR_BADVALUE, "Test %u: expected hr == D3DRMERR_BADVALUE, got %#x.\n", i, hr);<br>
</span>Note that mentioning the expected value is mostly redundant here. It's<br>
obvious as soon as you look at the test source, and test output isn't<br>
generally useful without that.<br>
<span class=""><br>
> +        hr = IDirect3DRM_CreateObject(d3drm1, tests[i].clsid, NULL, tests[i].iid, (void **)&unknown);<br>
> +        todo_wine ok(SUCCEEDED(hr), "Test %u: expected hr == D3DRM_OK, got %#x.\n", i, hr);<br>
> +        if (SUCCEEDED(hr))<br>
> +        {<br>
</span>I'll point out that if you had done a "continue" on failure instead,<br>
you wouldn't need to change the indentation when you remove the<br>
todo_wine.<br>
<br>
<br>
</blockquote></div><br></div>