On Mon,  9 Dec 2013 11:55:18 +0100, Vittorio Giovara 
<[email protected]> wrote:
> ---
> Resending just because some alignment changed.
> Vittorio
> 
>  libavcodec/mpeg12dec.c |   33 +++++++++++++++++++++++++++++++++
>  1 file changed, 33 insertions(+)
> 
> diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c
> index cc65283..7bb6647 100644
> --- a/libavcodec/mpeg12dec.c
> +++ b/libavcodec/mpeg12dec.c
> @@ -27,6 +27,7 @@
>  
>  #include "libavutil/attributes.h"
>  #include "libavutil/internal.h"
> +#include "libavutil/stereo3d.h"
>  #include "internal.h"
>  #include "avcodec.h"
>  #include "dsputil.h"
> @@ -2122,6 +2123,38 @@ static void mpeg_decode_user_data(AVCodecContext 
> *avctx,
>                  return;
>              avctx->dtg_active_format = p[0] & 0x0f;
>          }
> +    } else if (buf_end - p >= 6 &&
> +               p[0] == 'J' && p[1] == 'P' && p[2] == '3' && p[3] == 'D') {
> +        if (p[4] == 0x03) { // S3D_video_format_length

You can save a level of indentation by merging the two ifs

Otherwise looks fine

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to