On Tue, 5 Jun 2018, Tomas Härdin wrote:

tor 2018-05-31 klockan 02:05 +0200 skrev Marton Balint:
> Signed-off-by: Marton Balint <c...@passwd.hu>
---
 libavformat/mxfdec.c | 8 ++++++++
 1 file changed, 8 insertions(+)

+            type = avcodec_get_type(st->codecpar->codec_id);
+            if (type == AVMEDIA_TYPE_SUBTITLE)
+                st->codecpar->codec_type = type;

This feels like something that belongs in more generic parts of lavf.
Filling codecpar with defaults for some given codec_id that is.

I assumed some codecs can be audio and video at the same time, and this is why it's not automatic. wrapped_avframe (like it or not) is such a codec. In any case, I'd rather not change the way it works here...

Regards,
Marton
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to