[PATCH] gameux/tests: Avoid an ever changing value in a gamestatistics failure message.

Francois Gouget fgouget at codeweavers.com
Sun Mar 15 17:56:34 CDT 2020


The whole point of printing the IGameStatisticsMgr::GetGameStatistics()
return value was to check whether the pointer had been modified. So
provide this information without including an ever changing value in the
failure message.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 dlls/gameux/tests/gamestatistics.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/dlls/gameux/tests/gamestatistics.c b/dlls/gameux/tests/gamestatistics.c
index d823af1aab2..8d4da4a0f4c 100644
--- a/dlls/gameux/tests/gamestatistics.c
+++ b/dlls/gameux/tests/gamestatistics.c
@@ -181,7 +181,8 @@ static void test_gamestatisticsmgr( void )
     gs = (void *)0xdeadbeef;
     hr = IGameStatisticsMgr_GetGameStatistics(gsm, sExeName, GAMESTATS_OPEN_OPENONLY, &dwOpenResult, &gs);
     ok(hr == HRESULT_FROM_WIN32(ERROR_FILE_NOT_FOUND), "GetGameStatistics returned unexpected value: 0x%08x\n", hr);
-    ok(gs == NULL, "Expected output pointer to be NULL, got %p\n", gs);
+    ok(gs == NULL, "Expected output pointer to be NULL, got %s\n",
+       (gs == (void *)0xdeadbeef ? "deadbeef" : "neither NULL nor deadbeef"));
 
     /* now, allow them to be created */
     hr = IGameStatisticsMgr_GetGameStatistics(gsm, sExeName, GAMESTATS_OPEN_OPENORCREATE, &dwOpenResult, &gs);
-- 
2.20.1




More information about the wine-devel mailing list