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