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]
