[FFmpeg-devel] [PATCH] libavformar/demux: fix dts being incorrectly set to pts

Hiccup Zhu hiccupzhu at gmail.com
Wed May 15 14:22:19 EEST 2024


Andreas:
>
> Shiqi Zhu:
> > Signed-off-by: Shiqi Zhu <hiccupzhu at gmail.com>
> > ---
> >  libavformat/demux.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavformat/demux.c b/libavformat/demux.c
> > index ecefe7e0a7..e19b8b6473 100644
> > --- a/libavformat/demux.c
> > +++ b/libavformat/demux.c
> > @@ -1112,7 +1112,7 @@ static void compute_pkt_fields(AVFormatContext *s, AVStream *st,
> >              /* presentation is not delayed : PTS and DTS are the same */
> >              if (pkt->pts == AV_NOPTS_VALUE)
> >                  pkt->pts = pkt->dts;
> > -            update_initial_timestamps(s, pkt->stream_index, pkt->pts,
> > +            update_initial_timestamps(s, pkt->stream_index, pkt->dts,
> >                                        pkt->pts, pkt);
> >              if (pkt->pts == AV_NOPTS_VALUE)
> >                  pkt->pts = sti->cur_dts;
>
> Can you provide an example for which this matters?
>
> - Andreas

There is no example yet, I saw this when reviewing the code.
So, is it reasonable to use pts instead of dts settings here?

- Shiqi


More information about the ffmpeg-devel mailing list