Re: [PATCH 3/5] qedit: Implement IAMTimelineObj_GetTimelineType and add tests.

[Nikolay Sivov]

On 26.04.2016 6:09, Alex Henrie wrote:
> static HRESULT WINAPI TimelineObj_GetTimelineType(IAMTimelineObj *iface, TIMELINE_MAJOR_TYPE *type)
>  {
>      TimelineObjImpl *This = impl_from_IAMTimelineObj(iface);
> -    FIXME("(%p)->(%p): not implemented!\n", This, type);
> -    return E_NOTIMPL;
> +    TRACE("(%p)->(%p)\n", This, type);
> +    *type = This->timeline_type;
> +    return S_OK;
>  }

Is it supposed to handle NULL input?