<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 17, 2016 at 6:44 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 17 June 2016 at 09:38, Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com">jem456.vasishta@gmail.com</a>> wrote:<br>
> v4: Replaced GetVisuals, GetParent and GetTransform with thunks to version 2.<br>
><br>
> v3: Added version 1->3 thunks for only those frame1 methods that have tests (see test_Frame). The rest will be stubs returning E_NOTIMPL. Also avoid the use of QI in Add/DeleteChild.<br>
><br>
</span>Doesn't that potentially break currently working applications? And<br>
while having tests is great, I'm not sure I agree with the premise<br>
that it's better to return E_NOTIMPL than to forward to an<br>
implementation that may have subtle differences but probably doesn't<br>
in most cases.<br></blockquote><div>I don't disagree with you. I just want an acceptable patch for this as I'll need it for implementing viewports. Also a lot of version 2 stuff uses version 1 frames as well, which makes it more important. I've CC'd Stefan in this mail, hopefully we can agree on something which works for all. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> +static HRESULT WINAPI d3drm_frame1_QueryInterface(IDirect3DRMFrame *iface, REFIID riid, void **out)<br>
> +{<br>
> +    struct d3drm_frame *frame = impl_from_IDirect3DRMFrame(iface);<br>
> +<br>
> +    TRACE("iface %p, riid %s, out %p.\n", iface, debugstr_guid(riid), out);<br>
> +<br>
> +    return IDirect3DRMFrame3_QueryInterface(&frame->IDirect3DRMFrame3_iface, riid, out);<br>
> +}<br>
</span>This may have come up before, but any reason you can't call<br>
d3drm_frame3_QueryInterface() directly here?<br></blockquote><div>I could probably put these stubs after version 3, but wouldn't that make the version order in the file 2->3->1? Maybe a separate patch could rearrange it to 3->2->1. Maybe we could do that after these stubs are added.<br><br></div><div>Cheers,<br></div><div>Aaryaman<br></div></div><br></div></div>