Re: [PATCH v4] quartz: Fix crash when pVisible is NULL.

[Jactry Zeng]

Hi Pengpeng,

On 9/9/20 1:07 PM, Pengpeng Dong wrote:
> BaseControlWindowImpl_get_Visible(IVideoWindow *iface, LONG *visi
>  TRACE("window %p, visible %p.\n", window, visible);
>
> - if(!visible)
> -
>    return E_POINTER;

Just a nitpick, please add a space after 'if'.
Also, IMO, a commit comment like "quartz: Handle NULL visible in IVideoWindow::get_Visible()." might be better. At least, we don't use 'pVisible' as its parameter name anymore.

Thanks for your contribution.