[PATCH v2] msvcrt/tests: Be more explicit about sscanf() with %d/%hd/%hhd.

[Serge Gautherie]

Signed-off-by: Serge Gautherie 
---
Removed comments, added a cast.
---
 dlls/msvcrt/tests/scanf.c | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/dlls/msvcrt/tests/scanf.c b/dlls/msvcrt/tests/scanf.c
index 57d9cc0..60e1f6c 100644
--- a/dlls/msvcrt/tests/scanf.c
+++ b/dlls/msvcrt/tests/scanf.c
@@ -207,9 +207,14 @@ static void test_sscanf( void )

result = 0xdeadbeef;
     strcpy(buffer,"12345678");
-    ret = p_sscanf(buffer, "%hd", &result);
+    ret = p_sscanf(buffer, "%d", &result);
     ok(ret == 1, "Wrong number of arguments read: %d\n", ret);
-    ok(result == 0xdead614e, "Wrong number read (%x)\n", result);
+    ok(result == 0x00bc614e, "Wrong number read (%08x)\n", result);
+
+    result = 0xdeadbeef;
+    ret = p_sscanf(buffer, "%hd", (short *)&result);
+    ok(ret == 1, "Wrong number of arguments read: %d\n", ret);
+    ok(result == 0xdead614e, "Wrong number read (%08x)\n", result);

result = 0xdeadbeef;
     strcpy(buffer,"12345678");
@@ -238,7 +243,7 @@ static void test_sscanf( void )
     result = 0xdeadbeef;
     ret = p_sscanf(buffer, "%hhd", &result);
     ok(ret == 1, "Wrong number of arguments read: %d\n", ret);
-    ok(result == 0xbc614e, "Wrong number read (%x)\n", result);
+    ok(result == 0x00bc614e, "Wrong number read (%08x)\n", result);

strcpy(buffer,"12345678901234");
     ret = p_sscanf(buffer, "%lld", &result64);

[Piotr Caban]

Hi Serge,

I think it's better to leave this test as is. I don't think that this 
patch makes the tests easier to read.

Thanks,
Piotr

On 1/29/20 6:37 PM, Serge Gautherie wrote:
> Signed-off-by: Serge Gautherie
> Removed comments, added a cast.
>
> dlls/msvcrt/tests/scanf.c | 11 ++++++++---
>   1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/dlls/msvcrt/tests/scanf.c b/dlls/msvcrt/tests/scanf.c
> index 57d9cc0..60e1f6c 100644
> --- a/dlls/msvcrt/tests/scanf.c
> +++ b/dlls/msvcrt/tests/scanf.c
> @@ -207,9 +207,14 @@ static void test_sscanf( void )
>   result = 0xdeadbeef;
>   strcpy(buffer,"12345678");
> - ret = p_sscanf(buffer, "%hd", &result);
> - ret = p_sscanf(buffer, "%d", &result);
>  ok(ret == 1, "Wrong number of arguments read: %d\n", ret);
> - ok(result == 0xdead614e, "Wrong number read (%x)\n", result);
> -
>
> ok(result == 0x00bc614e, "Wrong number read (%08x)\n", result);
> -
> -
>
> result = 0xdeadbeef;
> -
>
> ret = p_sscanf(buffer, "%hd", (short *)&result);
> -
>
> ok(ret == 1, "Wrong number of arguments read: %d\n", ret);
> -
>
> ok(result == 0xdead614e, "Wrong number read (%08x)\n", result);
>
> result = 0xdeadbeef;
>  strcpy(buffer,"12345678");
>
> @@ -238,7 +243,7 @@ static void test_sscanf( void )
>       result = 0xdeadbeef;
>       ret = p_sscanf(buffer, "%hhd", &result);
>       ok(ret == 1, "Wrong number of arguments read: %d\n", ret);
> - ok(result == 0xbc614e, "Wrong number read (%x)\n", result);
> -
>
> ok(result == 0x00bc614e, "Wrong number read (%08x)\n", result);
>
> strcpy(buffer,"12345678901234");
>  ret = p_sscanf(buffer, "%lld", &result64);