[PATCH] usp10/tests: Add missing return value tests (Coverity)

[Nikolay Sivov]

Signed-off-by: Nikolay Sivov 
---
 dlls/usp10/tests/usp10.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/dlls/usp10/tests/usp10.c b/dlls/usp10/tests/usp10.c
index 55bec7f74f..35cdec545b 100644
--- a/dlls/usp10/tests/usp10.c
+++ b/dlls/usp10/tests/usp10.c
@@ -3150,6 +3150,7 @@ static void test_ScriptXtoX(void)
         WORD clust = 0;
         INT advance = 16;
         hr = ScriptXtoCP(iX, 1, 1, &clust, psva, &advance, &sa, &piCP, &piTrailing);
+        ok(hr == S_OK, "ScriptXtoCP() failed, hr %#x.\n", hr);
         ok(piCP==0 && piTrailing==1,"%i should return 0(%i) and 1(%i)\n",iX,piCP,piTrailing);
     }
     for (iX = 9; iX < 16; iX++)
@@ -3157,6 +3158,7 @@ static void test_ScriptXtoX(void)
         WORD clust = 0;
         INT advance = 16;
         hr = ScriptXtoCP(iX, 1, 1, &clust, psva, &advance, &sa, &piCP, &piTrailing);
+        ok(hr == S_OK, "ScriptXtoCP() failed, hr %#x.\n", hr);
         ok(piCP==0 && piTrailing==0,"%i should return 0(%i) and 0(%i)\n",iX,piCP,piTrailing);
     }

[Aric Stewart]

Signed-off-by: Aric Stewart

On 12/23/17 4:22 AM, Nikolay Sivov wrote:
> Signed-off-by: Nikolay Sivov
>
> dlls/usp10/tests/usp10.c | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/dlls/usp10/tests/usp10.c b/dlls/usp10/tests/usp10.c
> index 55bec7f74f..35cdec545b 100644
> --- a/dlls/usp10/tests/usp10.c
> +++ b/dlls/usp10/tests/usp10.c
> @@ -3150,6 +3150,7 @@ static void test_ScriptXtoX(void)
>           WORD clust = 0;
>           INT advance = 16;
>           hr = ScriptXtoCP(iX, 1, 1, &clust, psva, &advance, &sa, &piCP, &piTrailing);
> -
>    ok(hr == S_OK, "ScriptXtoCP() failed, hr %#x.\n", hr);
>     ok(piCP==0 && piTrailing==1,"%i should return 0(%i) and 1(%i)\n",iX,piCP,piTrailing);
> }
> for (iX = 9; iX < 16; iX++)
>
> @@ -3157,6 +3158,7 @@ static void test_ScriptXtoX(void)
>           WORD clust = 0;
>           INT advance = 16;
>           hr = ScriptXtoCP(iX, 1, 1, &clust, psva, &advance, &sa, &piCP, &piTrailing);
> -
>    ok(hr == S_OK, "ScriptXtoCP() failed, hr %#x.\n", hr);
>     ok(piCP==0 && piTrailing==0,"%i should return 0(%i) and 0(%i)\n",iX,piCP,piTrailing);
> }