Re: [PATCH i-g-t] tests/gem_concurrent_all: Please the compiler regarding fscanfs unused result.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 05, 2016 at 08:13:23PM +0300, Marius Vlad wrote:
> Signed-off-by: Marius Vlad <marius.c.vlad@xxxxxxxxx>
> ---
>  tests/gem_concurrent_all.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/gem_concurrent_all.c b/tests/gem_concurrent_all.c
> index 10e5357..e84ded3 100644
> --- a/tests/gem_concurrent_all.c
> +++ b/tests/gem_concurrent_all.c
> @@ -650,7 +650,7 @@ static int read_sysctl(const char *path)
>  	FILE *file = fopen(path, "r");
>  	int max = 0;
>  	if (file) {
> -		fscanf(file, "%d", &max);
> +		igt_assert(fscanf(file, "%d", &max) == 1);

But the code is written in case fscanf() actually fails...
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux