On Thu, 2016-08-18 at 12:43 -0700, Daniel Charles wrote:
> after creating underlying surface there's a double check on the
> VAStatus result. Replace it with ASSERT_RET.
> 
> Signed-off-by: Daniel Charles <[email protected]>

lgtm, applied.

Thanks,

Sean

> ---
>  src/i965_encoder.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/src/i965_encoder.c b/src/i965_encoder.c
> index beea811..47368fb 100644
> --- a/src/i965_encoder.c
> +++ b/src/i965_encoder.c
> @@ -153,10 +153,7 @@ 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;
> +    ASSERT_RET(status == VA_STATUS_SUCCESS, status);
>  
>      obj_surface = SURFACE(encoder_context->input_yuv_surface);
>      encode_state->input_yuv_object = obj_surface;

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