On Mon, 2016-08-15 at 10:49 -0700, Daniel Charles wrote:
> after creating underlying surface there's a double check on the
> VAStatus result. Removed assert as it was part of the code when
> the function didn't return vaStatus
> 
> Signed-off-by: Daniel Charles <[email protected]>
> ---
>  src/i965_encoder.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/src/i965_encoder.c b/src/i965_encoder.c
> index beea811..824ed0f 100644
> --- a/src/i965_encoder.c
> +++ b/src/i965_encoder.c
> @@ -153,7 +153,6 @@ intel_encoder_check_yuv_surface(VADriverContextP
> ctx,
>                                   VA_RT_FORMAT_YUV420,
>                                   1,
>                                   &encoder_context-
> >input_yuv_surface);
> -    assert(status == VA_STATUS_SUCCESS);
>  
>      if (status != VA_STATUS_SUCCESS)
>          return status;

Please change these lines to an ASSERT_RET() instead.

Thanks,

Sean

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Libva mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libva

Reply via email to