19 Jul
2008
19 Jul
'08
4:27 p.m.
2008/7/17 Vincent Povirk <vincent(a)codeweavers.com>:
@@ -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 :-)