On Wed, Jul 04, 2018 at 03:06:54PM -0700, Baptiste Coudurier wrote: > --- > libavformat/mxf.c | 1 + > libavformat/mxfdec.c | 2 ++ > libavformat/mxfenc.c | 41 +++++++++++++++++++++++++++++++++++++---- > libavformat/utils.c | 6 +++++- > 4 files changed, 45 insertions(+), 5 deletions(-) [...] > diff --git a/libavformat/utils.c b/libavformat/utils.c > index c9cdd2b470..36a32ad9c2 100644 > --- a/libavformat/utils.c > +++ b/libavformat/utils.c > @@ -1003,6 +1003,10 @@ FF_ENABLE_DEPRECATION_WARNINGS > *pnum = frame_size; > *pden = sample_rate; > break; > + case AVMEDIA_TYPE_DATA: > + *pnum = st->time_base.num; > + *pden = st->time_base.den; > + break; > default: > break; > } > @@ -1405,7 +1409,7 @@ static void compute_pkt_fields(AVFormatContext *s, > AVStream *st, > presentation_delayed, delay, av_ts2str(pkt->pts), > av_ts2str(pkt->dts), av_ts2str(st->cur_dts)); > > /* update flags */ > - if (is_intra_only(st->codecpar->codec_id)) > + if (st->codecpar->codec_type == AVMEDIA_TYPE_DATA || > is_intra_only(st->codecpar->codec_id)) > pkt->flags |= AV_PKT_FLAG_KEY; > #if FF_API_CONVERGENCE_DURATION > FF_DISABLE_DEPRECATION_WARNINGS
This should be a seperate patch. Also i think forcing duration=1 is not correct for all AVMEDIA_TYPE_DATA for example there are ID3 and fonts. These would i guess if they have a timebase set apply to more than 1 "unit" [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Let us carefully observe those good qualities wherein our enemies excel us and endeavor to excel them, by avoiding what is faulty, and imitating what is excellent in them. -- Plutarch
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
