[FFmpeg-devel] [PATCH v1 2/2] Add stream dump test with test for ogg/flac.
Romain Beauxis
romain.beauxis at gmail.com
Wed Jan 29 18:05:59 EET 2025
Le mer. 29 janv. 2025 à 15:43, Romain Beauxis
<romain.beauxis at gmail.com> a écrit :
>
> This patch adds a fate test for ogg/flac chained streams.
>
> First, a new api-dump-stream-meta-test utility is introduced then it is
> used to dump the metadata of a ogg/flac chained sample.
>
> Sample is available here:
https://www.dropbox.com/scl/fo/fxt2edwkyj2mjc9qubku5/AICHxJyxMMAK8MIJqWLcvk4?rlkey=mlt12lsu741ejukz0p5qtn9rq&dl=0
>
> Sample is very short so the dump output is:
>
> Stream ID: 0, PTS: 0, DTS: 0, metadata: encoder=Lavc61.19.100
flac:title=First Stream
> Stream ID: 0, PTS: 4608, DTS: 4608, metadata: encoder=Lavc61.19.100
flac:title=First Stream
> Stream ID: 0, PTS: 0, DTS: 0, metadata: encoder=Lavc61.19.100
flac:title=Second Stream
> Stream ID: 0, PTS: 4608, DTS: 4608, metadata: encoder=Lavc61.19.100
flac:title=Second Stream
I forgot to add the output of the stream meta dump prior to the changes:
Stream ID: 0, PTS: 0, DTS: 0, metadata: encoder=Lavc61.19.100
flac:title=First Stream
Stream ID: 0, PTS: 4608, DTS: 4608, metadata: encoder=Lavc61.19.100
flac:title=First Stream
Stream ID: 0, PTS: 0, DTS: 0, metadata: encoder=Lavc61.19.100
flac:title=First Stream
Stream ID: 0, PTS: 0, DTS: 0, metadata: encoder=Lavc61.19.100
flac:title=First Stream
Stream ID: 0, PTS: 0, DTS: 0, metadata: encoder=Lavc61.19.100
flac:title=First Stream
Stream ID: 0, PTS: 4608, DTS: 4608, metadata: encoder=Lavc61.19.100
flac:title=First Stream
The extra two packets are the header packets.
The PTS/DTS discontinuity is already present, the decoder seems to
have no problem with them.
I'd be happy to do a follow-up to tentatively adjust PTS/DTS if that'd
make sense/is possible.
Le mer. 29 janv. 2025 à 15:43, Romain Beauxis <romain.beauxis at gmail.com> a
écrit :
> This patch adds a fate test for ogg/flac chained streams.
>
> First, a new api-dump-stream-meta-test utility is introduced then it is
> used to dump the metadata of a ogg/flac chained sample.
>
> Sample is available here:
> https://www.dropbox.com/scl/fo/fxt2edwkyj2mjc9qubku5/AICHxJyxMMAK8MIJqWLcvk4?rlkey=mlt12lsu741ejukz0p5qtn9rq&dl=0
>
> Sample is very short so the dump output is:
>
> Stream ID: 0, PTS: 0, DTS: 0, metadata: encoder=Lavc61.19.100
> flac:title=First Stream
> Stream ID: 0, PTS: 4608, DTS: 4608, metadata: encoder=Lavc61.19.100
> flac:title=First Stream
> Stream ID: 0, PTS: 0, DTS: 0, metadata: encoder=Lavc61.19.100
> flac:title=Second Stream
> Stream ID: 0, PTS: 4608, DTS: 4608, metadata: encoder=Lavc61.19.100
> flac:title=Second Stream
>
> ---
> tests/Makefile | 2 +
> tests/api/Makefile | 2 +-
> tests/api/api-dump-stream-meta-test.c | 88 +++++++++++++++++++++++++++
> tests/fate/api.mak | 5 ++
> tests/fate/ogg-flac.mak | 11 ++++
> 5 files changed, 107 insertions(+), 1 deletion(-)
> create mode 100644 tests/api/api-dump-stream-meta-test.c
> create mode 100644 tests/fate/ogg-flac.mak
>
> diff --git a/tests/Makefile b/tests/Makefile
> index f9f5fc07f3..66e5189e6d 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -219,6 +219,7 @@ include $(SRC_PATH)/tests/fate/mpeg4.mak
> include $(SRC_PATH)/tests/fate/mpegps.mak
> include $(SRC_PATH)/tests/fate/mpegts.mak
> include $(SRC_PATH)/tests/fate/mxf.mak
> +include $(SRC_PATH)/tests/fate/ogg-flac.mak
> include $(SRC_PATH)/tests/fate/oma.mak
> include $(SRC_PATH)/tests/fate/opus.mak
> include $(SRC_PATH)/tests/fate/pcm.mak
> @@ -277,6 +278,7 @@ $(FATE_FFPROBE) $(FATE_FFMPEG_FFPROBE)
> $(FATE_SAMPLES_FFPROBE) $(FATE_SAMPLES_FF
> $(FATE_SAMPLES_FASTSTART): tools/qt-faststart$(EXESUF)
> $(FATE_SAMPLES_DUMP_DATA) $(FATE_SAMPLES_DUMP_DATA-yes):
> tools/venc_data_dump$(EXESUF)
> $(FATE_SAMPLES_SCALE_SLICE): tools/scale_slice_test$(EXESUF)
> +$(FATE_SAMPLES_DUMP_STREAM_META):
> tests/api/api-dump-stream-meta-test$(EXESUF)
>
> ifdef SAMPLES
> FATE += $(FATE_EXTERN)
> diff --git a/tests/api/Makefile b/tests/api/Makefile
> index c96e636756..a2cb06a729 100644
> --- a/tests/api/Makefile
> +++ b/tests/api/Makefile
> @@ -1,7 +1,7 @@
> APITESTPROGS-$(call ENCDEC, FLAC, FLAC) += api-flac
> APITESTPROGS-$(call DEMDEC, H264, H264) += api-h264
> APITESTPROGS-$(call DEMDEC, H264, H264) += api-h264-slice
> -APITESTPROGS-yes += api-seek
> +APITESTPROGS-yes += api-seek api-dump-stream-meta
> APITESTPROGS-$(call DEMDEC, H263, H263) += api-band
> APITESTPROGS-$(HAVE_THREADS) += api-threadmessage
> APITESTPROGS += $(APITESTPROGS-yes)
> diff --git a/tests/api/api-dump-stream-meta-test.c
> b/tests/api/api-dump-stream-meta-test.c
> new file mode 100644
> index 0000000000..d750e0aecb
> --- /dev/null
> +++ b/tests/api/api-dump-stream-meta-test.c
> @@ -0,0 +1,88 @@
> +/*
> + * Copyright (c) 2025 Romain Beauxis
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining
> a copy
> + * of this software and associated documentation files (the "Software"),
> to deal
> + * in the Software without restriction, including without limitation the
> rights
> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or
> sell
> + * copies of the Software, and to permit persons to whom the Software is
> + * furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be
> included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
> OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> ARISING FROM,
> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> IN
> + * THE SOFTWARE.
> + */
> +
> +/**
> + * Dump stream metadata
> + */
> +
> +#include "libavformat/avformat.h"
> +
> +static int dump_stream_meta(const char *input_filename)
> +{
> + AVPacket *pkt = NULL;
> + AVFormatContext *fmt_ctx = NULL;
> + int result;
> + char *metadata;
> +
> + result = avformat_open_input(&fmt_ctx, input_filename, NULL, NULL);
> + if (result < 0) {
> + av_log(NULL, AV_LOG_ERROR, "Can't open file\n");
> + return result;
> + }
> +
> + result = avformat_find_stream_info(fmt_ctx, NULL);
> + if (result < 0) {
> + av_log(NULL, AV_LOG_ERROR, "Can't get stream info\n");
> + goto end;
> + }
> +
> + pkt = av_packet_alloc();
> + if (!pkt) {
> + av_log(NULL, AV_LOG_ERROR, "Cannot allocate packet\n");
> + result = AVERROR(ENOMEM);
> + goto end;
> + }
> +
> + for (;;) {
> + result = av_read_frame(fmt_ctx, pkt);
> + if (result)
> + goto end;
> +
> + result = av_dict_get_string(
> + fmt_ctx->streams[pkt->stream_index]->metadata, &metadata,
> '=', ':');
> + if (result < 0)
> + goto end;
> +
> + printf("Stream ID: %d, PTS: %lld, DTS: %lld, metadata: %s\n",
> + pkt->stream_index, pkt->pts, pkt->dts, metadata);
> + av_free(metadata);
> + av_packet_unref(pkt);
> + }
> +
> +end:
> + av_packet_free(&pkt);
> + avformat_close_input(&fmt_ctx);
> + return result;
> +}
> +
> +int main(int argc, char **argv)
> +{
> + if (argc < 2) {
> + av_log(NULL, AV_LOG_ERROR, "Incorrect input\n");
> + return 1;
> + }
> +
> + if (dump_stream_meta(argv[1]) != AVERROR_EOF)
> + return 1;
> +
> + return 0;
> +}
> diff --git a/tests/fate/api.mak b/tests/fate/api.mak
> index d2868e57ac..a9872a5589 100644
> --- a/tests/fate/api.mak
> +++ b/tests/fate/api.mak
> @@ -1,3 +1,8 @@
> +FATE_API_LIBAVFORMAT-$(call ENCDEC, FLAC, FLAC) += api-dump-stream-meta
> +fate-api-ogg-flac-meta: $(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF)
> +fate-api-ogg-flac-meta: CMD = run $(APITESTSDIR)/api-flac-test$(EXESUF)
> +fate-api-ogg-flac-meta: CMP = null
> +
> FATE_API_LIBAVCODEC-$(call ENCDEC, FLAC, FLAC) += fate-api-flac
> fate-api-flac: $(APITESTSDIR)/api-flac-test$(EXESUF)
> fate-api-flac: CMD = run $(APITESTSDIR)/api-flac-test$(EXESUF)
> diff --git a/tests/fate/ogg-flac.mak b/tests/fate/ogg-flac.mak
> new file mode 100644
> index 0000000000..22e2030534
> --- /dev/null
> +++ b/tests/fate/ogg-flac.mak
> @@ -0,0 +1,11 @@
> +FATE_OGG_FLAC += fate-ogg-flac-chained-meta
> +fate-ogg-flac-chained-meta: REF = $(SAMPLES)/ogg-flac/chained-meta.txt
> +fate-ogg-flac-chained-meta: CMD =
> $(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF)
> $(TARGET_SAMPLES)/ogg-flac/chained-meta.ogg
> +
> +FATE_OGG_FLAC-$(call DEMDEC, OGG, FLAC) += $(FATE_OGG_FLAC)
> +
> +FATE_SAMPLES_DUMP_STREAM_META += $(FATE_OGG_FLAC-yes)
> +
> +FATE_EXTERN += $(FATE_OGG_FLAC-yes)
> +
> +fate-ogg-flac: $(FATE_OGG_FLAC-yes)
> --
> 2.39.5 (Apple Git-154)
>
>
More information about the ffmpeg-devel
mailing list