Try #3: user32: Windows test request (cursors/icons)

[Daniel Santos]

Ok, I hope this one is stable on win9x.  Please give it a try and let me know.

Thanks
Daniel

[Paul Vriens]

Daniel Santos wrote:
> Ok, I hope this one is stable on win9x.  Please give it a try and let me know.
>
> Thanks
> Daniel

No more crashes on Win95/Win98 and also no failures on these platforms. 
I didn't test the rest as they were OK before.

While you're at it why not do something like:

static inline int is_win9x()
{
     SetLastError(0xdeadbeef);
     lstrcmpW(NULL, NULL);
     return (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED);
}

This avoids the GetVersion() and instead relies more on behaviour. 
(Maybe it would be a good idea to have this in include/wine/test.h in 
the end.)

What you probably also could do now (not before because of the 
GetVersion) is add a win_skip() message in case we run into win9x. So at 
least we know some tests are skipped (on Windows only as win_skip() will 
trigger an error on Wine).

I'm still a bit confused by this comment:

* FIXME:
  * - Add tests for CreateCursor and verify that width & height cannot 
exceed
  *   SM_CXCURSOR & SM_CYCURSOR (because they can).

It has 'verify', 'cannot' and 'because they can' in one sentence. So I 
read this as:

"Add tests for CreateCursor and show that width & height can exceed 
SM_CXCURSOR & SM_CYCURSOR (despite it shouldn't according to ....)".