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

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

[Shiqi Zhu]

Signed-off-by: Shiqi Zhu <hiccupzhu@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;
-- 
2.34.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

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

[Andreas Rheinhardt]

Shiqi Zhu:
> Signed-off-by: Shiqi Zhu <hiccupzhu@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

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

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

[Hiccup Zhu]

Andreas:
>
> Shiqi Zhu:
> > Signed-off-by: Shiqi Zhu <hiccupzhu@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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".