<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 17, 2016 at 7:23 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>On 17 June 2016 at 15:40, Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com" target="_blank">jem456.vasishta@gmail.com</a>> wrote:<br>
> On Fri, Jun 17, 2016 at 6:44 PM, Henri Verbeet <<a href="mailto:hverbeet@gmail.com" target="_blank">hverbeet@gmail.com</a>> wrote:<br>
</span><span>>> This may have come up before, but any reason you can't call<br>
>> d3drm_frame3_QueryInterface() directly here?<br>
><br>
> I could probably put these stubs after version 3, but wouldn't that make the<br>
> version order in the file 2->3->1? Maybe a separate patch could rearrange it<br>
> to 3->2->1. Maybe we could do that after these stubs are added.<br>
><br>
</span>I'd make the order<br>
<br>
    d3drm_frame3_QueryInterface()<br>
    d3drm_frame2_QueryInterface()<br>
    d3drm_frame1_QueryInterface()<br>
    d3drm_frame3_AddRef()<br>
    d3drm_frame2_AddRef()<br>
    d3drm_frame1_AddRef()<br>
    etc.<br>
<br>
I'd don't care so much if you move the version 2 functions in the same<br>
patch that introduces the version 1 functions, although I think it<br>
makes sense to do that in a separate patch.<br></blockquote><div>Right, will get right on it. I'll first write a patch to rearrange it and rebase these patches on top of it. Please review the other patches in the mean time if possible. Thanks for the review!<br><br></div><div>Cheers,<br></div><div>Aaryaman <br></div></div><br></div></div>