<table cellspacing="0" cellpadding="0" border="0" ><tr><td valign="top" style="font: inherit;">Thanks again for your feedback and assistance.. This is my 4th revision. I've created a static inline is_win9x() as you suggested. I agree that this should be in include/wine/test.h. One minor complication is that some test files define a BOOL or int "is_win9x" local variable, but that is easy to overcome. A few months ago, I wrote an extensive windows version testing patch for wine/test.h, but I appear to have either mis-placed it or accidentally blown it away! =)<br><br>I've added win_skip() messages where tests are skipped on win9x to avoid crashing the and I clarified the FIXME (I hope it makes more sense). I'm trying to say that we should add tests to make sure height & width are checked since CreateCursor currently doesn't validate the size of the cursor you want to create against the values returned by
GetSystemMetrics().<br><br>Finally, by Griswold's suggestion, I renamed test_SetCursor and test_DestroyCursor, adding an underscore prefix and a macro that will fill in the __LINE__ to clean up the code slightly.<br><br>If you can please run this again on a 9x platform and also an NT platform (XP or vista) I would greatly appreciate it! I just want to make sure everything is happy before submitting it and thanks again for all your help!<br><br>Daniel<br><br></td></tr></table><br>