Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH v2] lavf/dash: Forward strict flag to component demuxers
@ 2024-05-22  9:02 Frank Plowman
  2024-05-22  9:12 ` Andreas Rheinhardt
  0 siblings, 1 reply; 2+ messages in thread
From: Frank Plowman @ 2024-05-22  9:02 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Frank Plowman

Before the patch, opening a DASH file containing streams which require
experimental decoders was problematic.  No matter where the -strict -2
was put on the command line, the option was not passed to the demuxer
for that component.  This resulted in an error, prompting the user to
add the -strict -2 flag, which is already present.  Decoding appeared to
continue correctly however.

Patch removes the error message by creating an options object for the
demuxer created for the component, which inherits from the parent
demuxer.

Signed-off-by: Frank Plowman <post@frankplowman.com>
---
Changes since v1: Use nb_streams stream_info_opts, not just one.

 libavformat/dashdec.c | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)

diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c
index 555e21bf69..9c5ef5801a 100644
--- a/libavformat/dashdec.c
+++ b/libavformat/dashdec.c
@@ -1911,13 +1911,28 @@ static int reopen_demux_for_component(AVFormatContext *s, struct representation
     if (ret < 0)
         goto fail;
     if (pls->n_fragments) {
+        AVDictionary **stream_info_opts;
+
+        stream_info_opts = av_calloc(pls->ctx->nb_streams, sizeof(*stream_info_opts));
+        if (!stream_info_opts)
+            goto fail;
+
 #if FF_API_R_FRAME_RATE
         if (pls->framerate.den) {
-            for (i = 0; i < pls->ctx->nb_streams; i++)
+            for (i = 0; i < pls->ctx->nb_streams; i++) {
                 pls->ctx->streams[i]->r_frame_rate = pls->framerate;
+                av_dict_set_int(&stream_info_opts[i], "strict", s->strict_std_compliance, 0);
+            }
+
         }
 #endif
-        ret = avformat_find_stream_info(pls->ctx, NULL);
+
+        ret = avformat_find_stream_info(pls->ctx, stream_info_opts);
+
+        for (i = 0; i < pls->ctx->nb_streams; i++)
+            av_dict_free(&stream_info_opts[i]);
+        av_dict_free(stream_info_opts);
+
         if (ret < 0)
             goto fail;
     }
-- 
2.44.0

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-05-22  9:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-05-22  9:02 [FFmpeg-devel] [PATCH v2] lavf/dash: Forward strict flag to component demuxers Frank Plowman
2024-05-22  9:12 ` Andreas Rheinhardt

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git