ieframe/tests: Skip rest of tests if CoCreateInstance failed.

Jacek Caban jacek at
Thu Aug 25 07:31:51 CDT 2011

On 08/25/11 14:09, Octavian Voicu wrote:
> On Thu, Aug 25, 2011 at 2:59 PM, Jacek Caban<jacek at>  wrote:
>> Those are failures on Wine, so tests are fine, it's Wine that needs fixing.
> Yes, the first test usually fails on Wine. Nonetheless, if
> CoCreateInstance fails for whatever reason (like it happens on that
> NT4 machine, for example), running the rest of the tests is pointless
> and will definitely crash.
> It's annoying to get crashes while running tests. The skip is not
> really needed I guess, but the return should be there to stop the
> crashing.

As long as there is a failure (like in your patch), I'm fine with 
return. Just let's not pretend it's somehow valid run with skip() call. 
Also testing hres is cleaner than unk, IMO.


More information about the wine-devel mailing list