[FFmpeg-devel] Re: [PATCH] avformat/mpegtsenc: Limit TS invalid messages

[Kieran Kunhya via ffmpeg-devel <ffmpeg-devel@ffmpeg.org>]

On Mon, 6 Oct 2025, 04:46 Michael Niedermayer via ffmpeg-devel, <
ffmpeg-devel@ffmpeg.org> wrote:

> Hi
>
> On Thu, Oct 02, 2025 at 11:36:33PM +0200, David Dominguez via ffmpeg-devel
> wrote:
> > mpegts_write_pes() warns when DTS - PCR is negative. This is
> > done to detect frames that are late for decoding, meaning that
> > timestamping is wrong or the muxrate value is insufficient.
> >
> > But this comparison only makes sense on the first packet of
> > the frame. After that the comparison will always be negative
> > as DTS doesn't change until the next frame, resulting in
> > hundreds of useless warnings.
> >
> > Addtionaly, for long key frames that take several frame periods
> > to transmity false warnings may be issued if max_delay
> > is low enough. For example, in low latency CC video.
> >
> > Limiting the warning to the start of frame transmission fixes
> > these issues.
>
> This doesnt feel right, the time a frame is decoded has
> to be after all of its parts are transmitted


In TS, yes. This patch makes no sense.

Kieran

>
>
_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org