[Libva] [PATCH][libva-intel-driver] i965_encoder: remove double check for VAStatus result

[Sean V Kelley]

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 <daniel.charles at intel.com>
> ---
>  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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/libva/attachments/20160815/bfe5a990/attachment.sig>