prevent peekmessage from crashing when msg == NULL

[Juan Lang]

--- Eric Frias <efrias at syncad.com> wrote:
> I must admit I've never looked at the test suite.  What kinds of things
> should have tests?  Are they intended to verify implementations of the
> undocumented or poorly-documented parts of the windows API?

That, as well as correct and documented behavior.  The more the merrier.

> I guess by that standard, the other patch I posted which fixed the
> behavior of two consecutive calls to OpenClipboard() would be another
> good candidate for a test case.

I think so.

The test code is pretty straightforward:  you use the ok macro to check a
condition.  Something like, ok(GetLastError() == ERROR_NO_MORE_DATA,
"Expected ERROR_NO_MORE_DATA, got %d\n", GetLastError());

If you write a test that fails under wine, and you don't have a chance to
fix it, you can still submit the test with a todo_wine, like:
todo_wine ok(...)

You can run the tests with make test.  I try to do a make test before
sending in a patch to make sure I didn't regress anything (fortunately
Alexandre seems to as well).

In some areas, particularly new areas of code, having no tests is
acceptable.  In existing code, having a test case will go a long way to
getting a patch accepted.

> It might take me a while to get around to it, right now I'm under
> pressure to get winelib working on HPUX.

Good luck!

--Juan


		
__________________________________ 
Do you Yahoo!? 
Meet the all-new My Yahoo! - Try it today! 
http://my.yahoo.com