[FFmpeg-devel] [PATCH] mov: Make sure PTS are both monotonically increasing, and unique
Sasi Inguva
isasi at google.com
Tue May 29 19:04:32 EEST 2018
Hi. sorry for the late reply. I sent a patch similar to this a while back
https://patchwork.ffmpeg.org/patch/8227/ but it got lost in the sea. You
also want to do,
@@ -3579,7 +3579,8 @@ static void mov_fix_index(MOVContext *mov, AVStream
*st)
frame_duration_buffer[num_discarded_begin - 1] =
frame_duration;
- if (first_non_zero_audio_edit > 0 &&
st->codecpar->codec_id != AV_CODEC_ID_VORBIS) {
+ if (st->codecpar->codec_type == AVMEDIA_TYPE_AUDIO
&&
+ first_non_zero_audio_edit > 0 &&
st->codecpar->codec_id != AV_CODEC_ID_VORBIS) {
st->skip_samples += frame_duration;
}
so that we only increment skip samples for audio streams. Otherwise patch
looks good to me.
On Thu, May 17, 2018 at 7:03 AM Derek Buitenhuis <derek.buitenhuis at gmail.com>
wrote:
> On Tue, May 15, 2018 at 8:44 PM, Derek Buitenhuis
> <derek.buitenhuis at gmail.com> wrote:
> > We already did this for audio, but it should be done for video too.
> > If we don't, seeking back to the start of the file, for example, can
> > become quite broken, since the first N packets will have repeating
> > and nonmonotonic PTS, yet they need to be decoded even if they are
> > to be discarded.
> >
> > Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
> > ---
> > libavformat/mov.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Ping.
>
> Is nobody outside Sasi able to review code in this part of
> mov.c? That is slightly worrying to me.
>
> - Derek
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
More information about the ffmpeg-devel
mailing list