[PATCH v3 0/2] avformat/mpjpegdec: add support for X-Timestamp and X-Framerate headers

Vladimir Sobolev via ffmpeg-devel <[email protected]>
Newsgroups gmane.comp.video.ffmpeg.devel
Message-ID <[email protected]>
Hi,

v2 of this patch was sent on 2025-11-12 but received no further review.
It addresses Michael's comments from v1:

- use isfinite() instead of !isnan() && isfinite()
- remove redundant if(log_ctx) checks when mpjpeg != NULL
- set timebase once in read_header using AV_TIME_BASE_Q
- set timestamps directly without rescaling

This version is rebased on current master and adds an ffprobe_demux FATE
test with a minimal multipart MJPEG sample exercising X-Timestamp and
X-Framerate header parsing.

Thanks,
Vladimir

Vladimir Sobolev (2):
  avformat/mpjpegdec: add support for X-Timestamp and X-Framerate
    headers
  fate: add mpjpeg X-Timestamp and X-Framerate demux test

 libavformat/mpjpegdec.c                 |  67 ++++++++++++++++++++++--
 tests/fate/demux.mak                    |   3 ++
 tests/mpjpeg-x-timestamp.mjpeg          | Bin 0 -> 613 bytes
 tests/ref/fate/mpjpeg-x-timestamp-demux |   4 ++
 4 files changed, 69 insertions(+), 5 deletions(-)
 create mode 100644 tests/mpjpeg-x-timestamp.mjpeg
 create mode 100644 tests/ref/fate/mpjpeg-x-timestamp-demux

-- 
2.50.1 (Apple Git-155)
_______________________________________________
ffmpeg-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.