[PATCH] user32/tests: Avoid an unneeded lstrlenA() call.

[Francois Gouget]

Note that ptr may be NULL (see ok() check a few lines up).

Signed-off-by: Francois Gouget <fgouget(a)free.fr>
---
 dlls/user32/tests/dde.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/user32/tests/dde.c b/dlls/user32/tests/dde.c
index eed2ac437e1..d370df181a3 100644
--- a/dlls/user32/tests/dde.c
+++ b/dlls/user32/tests/dde.c
@@ -1977,7 +1977,7 @@ static void test_DdeCreateDataHandle(void)
     ok(size == 262, "Expected 262, got %d\n", size);
     todo_wine
     {
-        ok(lstrlenA((LPSTR)ptr) == 0, "Expected 0, got %d\n", lstrlenA((LPSTR)ptr));
+        ok(ptr && !*ptr, "Expected 0, got %d\n", lstrlenA((LPSTR)ptr));
     }
 
     ret = DdeUnaccessData(hdata);
-- 
2.20.1

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=61629

Your paranoid android.


=== w2003std (task log) ===

Task errors:
The task timed out

=== w1064v1507 (32 bit report) ===

user32:
dde.c:1012: Test failed: Expected item atom, got 00000081
dde.c:852: Test failed: client failed with 1 error(s)