[Mesa-dev] [PATCH 8/8] st/va: fix indentation

Julien Isorce julien.isorce at gmail.com
Mon Nov 23 09:46:02 PST 2015


Nice cleanups. I also tested these changes on "nouveau" driver with one
video per format (except HEVC) and works fine with mpv and gstreamer.

Patches "[PATCH 2/8] st/va: move post processing function into own file" to
"[PATCH 8/8] st/va: fix indentation"
are:
Reviewed-by: Julien Isorce <j.isorce at samsung.com>

On 23 November 2015 at 14:37, Christian König <deathsimple at vodafone.de>
wrote:

> From: Christian König <christian.koenig at amd.com>
>
> Signed-off-by: Christian König <christian.koenig at amd.com>
> ---
>  src/gallium/state_trackers/va/picture.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/src/gallium/state_trackers/va/picture.c
> b/src/gallium/state_trackers/va/picture.c
> index ce75e2d..25d2940 100644
> --- a/src/gallium/state_trackers/va/picture.c
> +++ b/src/gallium/state_trackers/va/picture.c
> @@ -192,19 +192,19 @@ handleVASliceDataBufferType(vlVaContext *context,
> vlVaBuffer *buf)
>     format = u_reduce_video_profile(context->decoder->profile);
>     switch (format) {
>     case PIPE_VIDEO_FORMAT_MPEG4_AVC:
> -         if (bufHasStartcode(buf, 0x000001, 24))
> -            break;
> +      if (bufHasStartcode(buf, 0x000001, 24))
> +         break;
>
> -         buffers[num_buffers] = (void *const)&start_code_h264;
> -         sizes[num_buffers++] = sizeof(start_code_h264);
> +      buffers[num_buffers] = (void *const)&start_code_h264;
> +      sizes[num_buffers++] = sizeof(start_code_h264);
>        break;
>     case PIPE_VIDEO_FORMAT_HEVC:
>        if (bufHasStartcode(buf, 0x000001, 24))
>           break;
>
> -         buffers[num_buffers] = (void *const)&start_code_h265;
> -         sizes[num_buffers++] = sizeof(start_code_h265);
> -         break;
> +      buffers[num_buffers] = (void *const)&start_code_h265;
> +      sizes[num_buffers++] = sizeof(start_code_h265);
> +      break;
>     case PIPE_VIDEO_FORMAT_VC1:
>        if (bufHasStartcode(buf, 0x0000010d, 32) ||
>            bufHasStartcode(buf, 0x0000010c, 32) ||
> --
> 2.5.0
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20151123/bae82d7b/attachment-0001.html>


More information about the mesa-dev mailing list