[Mesa-dev] [PATCH] omx/h264: remove stray semicolon after if

Christian König deathsimple at vodafone.de
Tue Sep 2 00:52:55 PDT 2014


Am 02.09.2014 um 01:40 schrieb Dave Airlie:
> From: Dave Airlie <airlied at redhat.com>
>
> Coverity reported this, looks wrong to me.

And is probably one of the reasons why streams with multiple slices in 
one frame didn't worked.

> Signed-off-by: Dave Airlie <airlied at redhat.com>

Reviewed-by: Christian König <christian.koenig at amd.com>

> ---
>   src/gallium/state_trackers/omx/vid_dec_h264.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/state_trackers/omx/vid_dec_h264.c b/src/gallium/state_trackers/omx/vid_dec_h264.c
> index 7f1c2fa..7b57785 100644
> --- a/src/gallium/state_trackers/omx/vid_dec_h264.c
> +++ b/src/gallium/state_trackers/omx/vid_dec_h264.c
> @@ -681,7 +681,7 @@ static void slice_header(vid_dec_PrivateType *priv, struct vl_rbsp *rbsp,
>         if (priv->picture.h264.field_pic_flag) {
>            unsigned bottom_field_flag = vl_rbsp_u(rbsp, 1);
>   
> -         if (bottom_field_flag != priv->picture.h264.bottom_field_flag);
> +         if (bottom_field_flag != priv->picture.h264.bottom_field_flag)
>               vid_dec_h264_EndFrame(priv);
>   
>            priv->picture.h264.bottom_field_flag = bottom_field_flag;



More information about the mesa-dev mailing list