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 :-)