OLEAUT32: Use the FAILED macro to check for failure of interface methods.

Mike McCormack mike at codeweavers.com
Fri Dec 30 20:12:33 CST 2005

Robert Shearman wrote:

> These changes are wrong. IStream_Read can return S_FALSE, which means 
> that the call was successful, but didn't do what was intended. From MSDN:
>    The data cannot be read from the stream object. Depending on the
>    implementation, either S_FALSE or an error code could be returned
>    when reading past the end of the stream.

Fair enough.   Then perhaps the ole storage code is broken, because it 
returns S_FALSE when it correctly read the requested amount of data, and 
IOlePicture_Load was failing to read a bitmap from an MSI package.


More information about the wine-devel mailing list