On 25/10/2007, James Hawkins wrote: > Hi, > > Changelog: > * Fix a test that now passes in Windows.
Looking at the code, it looks as if they are now passing in Wine :). > dlls/user32/tests/dde.c | 13 ++++++++++--- > 1 files changed, 10 insertions(+), 3 deletions(-) > + str = (LPSTR)DdeAccessData(hdata, &size); > + ok(!lstrcmpA(str, "requested data\r\n"), "Expected 'requested > data\\r\\n', got %s\n", str); Shouldn't there be quote marks around the %s (i.e. '%s', to match the expected value)? > + ok(size == 19, "Expected 19, got %d\n", size); > + > + ret = DdeUnaccessData(hdata); > + ok(ret == TRUE, "Expected TRUE, got %d\n", ret); These are new test cases. Since these are not fixes to existing test cases, they should really be in a separate patch. - Reece