2008/7/17 Vincent Povirk <[EMAIL PROTECTED]>:
> @@ -583,6 +583,9 @@ HRESULT WINAPI OleIsCurrentClipboard(IDataObject 
> *pDataObject)
>    if (!theOleClipboard)
>      return E_OUTOFMEMORY;
>
> +  if (pDataObject == NULL)
> +    return S_FALSE;
> +

Functionally, the patch is good, but one small nit about the style -
the check before uses the !ptr style whereas the check that you
introduce uses the ptr == NULL style. It would be nice if the checks
had a consistent style.

-- 
Rob Shearman

P.S. Congratulations on your new job :-)


Reply via email to