Sorry, commited patch seems to be right. Still it's better to use hr!=S_OK instead of hr itself I think..
On 10/16/08, Nikolay Sivov bunglehead@gmail.com wrote:
It isn't obvious (your patch fot ret value). Why are you hiding all codes except E_FAIL? Tests are needed, but I think you're hiding something by this change.
On 10/16/08, Kirill K. Smirnov lich@math.spbu.ru wrote:
This patch fixes gdiplus test failures revealed by patch: [2/3] oleaut32: return correct error code http://www.winehq.org/pipermail/wine-patches/2008-October/063221.html