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

2016-09-20 Thread Rodger Combs
> On Sep 20, 2016, at 10:18, Carl Eugen Hoyos wrote: > > 2016-09-20 13:13 GMT+02:00 Rodger Combs : >> Some codecs set this in the parser, but not the decoder > >> +#define FILL(name) if(s->name > 0 && avctx->name <= 0) avctx->name = s->name >> +if (avctx->codec_type == AVMEDIA_TYPE_VIDEO) {

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

2016-09-20 Thread Carl Eugen Hoyos
2016-09-20 13:13 GMT+02:00 Rodger Combs : > Some codecs set this in the parser, but not the decoder > +#define FILL(name) if(s->name > 0 && avctx->name <= 0) avctx->name = s->name > +if (avctx->codec_type == AVMEDIA_TYPE_VIDEO) { > +FILL(field_order); How is this a useful macro? Carl

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

2016-09-20 Thread Rodger Combs
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_pa