[FFmpeg-devel] [PATCH 1/4] lavc/parser: export field order if not already set

Michael Niedermayer michael at niedermayer.cc
Wed Oct 5 14:19:23 EEST 2016


On Mon, Oct 03, 2016 at 11:49:36PM -0500, Rodger Combs wrote:
> Some codecs set this in the parser, but not the decoder
> ---
>  libavcodec/parser.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/libavcodec/parser.c b/libavcodec/parser.c
> index 2c8fc69..30cfc55 100644
> --- a/libavcodec/parser.c
> +++ b/libavcodec/parser.c
> @@ -182,6 +182,11 @@ int av_parser_parse2(AVCodecParserContext *s, AVCodecContext *avctx,
>      index = s->parser->parser_parse(s, avctx, (const uint8_t **) poutbuf,
>                                      poutbuf_size, buf, buf_size);
>      av_assert0(index > -0x20000000); // The API does not allow returning AVERROR codes
> +#define FILL(name) if(s->name > 0 && avctx->name <= 0) avctx->name = s->name
> +    if (avctx->codec_type == AVMEDIA_TYPE_VIDEO) {
> +        FILL(field_order);
> +    }

LGTM

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Does the universe only have a finite lifespan? No, its going to go on
forever, its just that you wont like living in it. -- Hiranya Peiri
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161005/f424f58d/attachment.sig>


More information about the ffmpeg-devel mailing list