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

Aric Stewart aric at codeweavers.com
Wed Dec 27 08:49:33 CST 2017


Signed-off-by: Aric Stewart <aric at codeweavers.com>

On 12/23/17 4:22 AM, Nikolay Sivov wrote:
> Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
> ---
>   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);
>       }
>   
> 



More information about the wine-devel mailing list