Re: [FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v4]

2019-08-02 Thread Andreas Håkon
Hi Andriy, ‐‐‐ Original Message ‐‐‐ On Thursday, 1 de August de 2019 16:15, Andriy Gelman wrote: > > > If you are going to add pcr_st to MpegTSService then it would make sense > > > to move the > > > initialization to mpegts_add_service function. > > > > Good idea! Marton recommends

Re: [FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v4]

2019-08-01 Thread Andriy Gelman
Andreas, On Thu, 01. Aug 08:24, Andreas Håkon wrote: > Hi Andriy, > > > ‐‐‐ Original Message ‐‐‐ > On Thursday, 1 de August de 2019 0:23, Andriy Gelman > wrote: > > > > +for (i = 0; i < ts->nb_services; i++) { > > > +service = ts->services[i]; > > > +

Re: [FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v4]

2019-08-01 Thread Andreas Håkon
Hi Andriy, ‐‐‐ Original Message ‐‐‐ On Thursday, 1 de August de 2019 0:23, Andriy Gelman wrote: > > +for (i = 0; i < ts->nb_services; i++) { > > +service = ts->services[i]; > > +service->pcr_st = NULL; > > +} > > + > > If you are going to add pcr_st to

Re: [FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v4]

2019-07-31 Thread Andriy Gelman
Andreas, On Mon, 29. Jul 11:11, Andreas Håkon wrote: > Hi, > > An updated version that fixes all previous bugs: > https://patchwork.ffmpeg.org/patch/14109/ > https://patchwork.ffmpeg.org/patch/14099/ > https://patchwork.ffmpeg.org/patch/14036/ > > It passes the tests pointed by Michael

[FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v4]

2019-07-29 Thread Andreas Håkon
Hi, An updated version that fixes all previous bugs: https://patchwork.ffmpeg.org/patch/14109/ https://patchwork.ffmpeg.org/patch/14099/ https://patchwork.ffmpeg.org/patch/14036/ It passes the tests pointed by Michael Niedermayer (Sample_cut.ts) and Andriy Gelman (day_flight.mpg). I hope this