Hi Daniel, On Mon, Aug 15, 2016 at 11:28 AM, Daniel Charles <[email protected]> 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]> > --- > src/i965_encoder.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/i965_encoder.c b/src/i965_encoder.c > index beea811..d909846 100644 > --- a/src/i965_encoder.c > +++ b/src/i965_encoder.c > @@ -153,7 +153,7 @@ intel_encoder_check_yuv_surface(VADriverContextP ctx, > VA_RT_FORMAT_YUV420, > 1, > &encoder_context->input_yuv_surface); > - assert(status == VA_STATUS_SUCCESS); > + ASSERT_RET(status == VA_STATUS_SUCCESS, status); > > if (status != VA_STATUS_SUCCESS) > return status; > You don't need this conditional with the ASSERT_RET(). Remove it too. Thanks, Sean > -- > 2.5.5 > > _______________________________________________ > Libva mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/libva > -- Sean V. Kelley <[email protected]> Open Source Technology Center / SSG Intel Corp.
_______________________________________________ Libva mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libva
