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

Daniel Charles daniel.charles at intel.com
Thu Aug 18 19:43:46 UTC 2016


after creating underlying surface there's a double check on the
VAStatus result. Replace it with ASSERT_RET.

Signed-off-by: Daniel Charles <daniel.charles at intel.com>
---
 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;
-- 
2.5.5



More information about the Libva mailing list