<div dir="ltr">Sorry, I meant 125481 (v4). Should I resend that one?<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 15, 2016 at 3:47 PM, Aaryaman Vasishta <span dir="ltr"><<a href="mailto:jem456.vasishta@gmail.com" target="_blank">jem456.vasishta@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Aug 15, 2016 at 3:37 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>On 12 August 2016 at 01:09, Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com" target="_blank">jem456.vasishta@gmail.com</a>> wrote:<br>
> +    IDirect3DRMFrame3_AddRef(*scen<wbr>e = &frame->IDirect3DRMFrame3_ifac<wbr>e);<br>
</span>Actually, sorry, you probably shouldn't do that in d3drm. The d3drm<br>
headers are written by hand, so IDirect3DRMFrame3_AddRef() is a macro<br>
instead of an inline function. That means that when this expands the<br>
assignment happens twice.<br>
<br>
<br></blockquote></div></div><div>No problem. Thanks for the explanation :)<br></div><div>You can sign-off on v5 of this patch, then? (125544)<br><br></div><div>Cheers,<br></div><div>Aaryaman<br></div></div><br></div></div>
</blockquote></div><br></div>