<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 12, 2016 at 4:29 AM, Józef Kucia <span dir="ltr"><<a href="mailto:joseph.kucia@gmail.com" target="_blank">joseph.kucia@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 Fri, Aug 12, 2016 at 12:48 AM, Aaryaman Vasishta<br>
<<a href="mailto:jem456.vasishta@gmail.com">jem456.vasishta@gmail.com</a>> wrote:<br>
> v4: Shorter implementation.<br>
> v3: Directly traverse through parent frames instead of using GetParent.<br>
><br>
> Signed-off-by: Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com">jem456.vasishta@gmail.com</a>><br>
> ---<br>
>  dlls/d3drm/frame.c       | 43 ++++++++++++++++++++++++++++++<wbr>+++++++------<br>
>  dlls/d3drm/tests/d3drm.c | 24 ++++++++++++++++++++++++<br>
>  2 files changed, 61 insertions(+), 6 deletions(-)<br>
><br>
> diff --git a/dlls/d3drm/frame.c b/dlls/d3drm/frame.c<br>
> index 2ff0241..4d7ab42 100644<br>
> --- a/dlls/d3drm/frame.c<br>
> +++ b/dlls/d3drm/frame.c<br>
> @@ -1339,23 +1339,54 @@ static HRESULT WINAPI d3drm_frame1_GetRotation(<wbr>IDirect3DRMFrame *iface,<br>
><br>
>  static HRESULT WINAPI d3drm_frame3_GetScene(<wbr>IDirect3DRMFrame3 *iface, IDirect3DRMFrame3 **scene)<br>
>  {<br>
> -    FIXME("iface %p, scene %p stub!\n", iface, scene);<br>
> +    struct d3drm_frame *frame = impl_from_IDirect3DRMFrame3(<wbr>iface);<br>
><br>
> -    return E_NOTIMPL;<br>
> +    TRACE("iface %p, scene %p.\n", iface, scene);<br>
> +<br>
> +    if (!scene)<br>
> +        return D3DRMERR_BADVALUE;<br>
> +<br>
> +    *scene = &frame->IDirect3DRMFrame3_<wbr>iface;<br>
<br>
</span>Do you really need this assignment?<br>
<div class="HOEnZb"><div class="h5"><br>
> +<br>
> +    while (frame->parent)<br>
> +        frame = frame->parent;<br>
> +<br>
> +    IDirect3DRMFrame3_AddRef(*<wbr>scene = &frame->IDirect3DRMFrame3_<wbr>iface);<br>
> +<br>
> +    return D3DRM_OK;<br>
>  }<br>
<br>
<br></div></div></blockquote><div>You're right, it's redundant. Will resend this. Thanks! <br><br></div><div>Cheers,<br></div><div>Aaryaman <br></div></div><br></div></div>