On 5/7/2023 6:20 PM, Stefano Sabatini wrote:
On date Friday 2023-05-05 20:18:56 -0300, James Almer wrote:
Similar to the decoder AVOptions above.
Please briefly specify the use case in the log.
A line about how these are options whose values are changed by the
demuxer and thus exported back to the caller?
Signed-off-by: James Almer <jamr...@gmail.com>
---
fftools/ffprobe.c | 12 ++++++++++++
tests/ref/fate/flv-demux | 4 ++--
tests/ref/fate/ts-demux | 6 +++---
tests/ref/fate/ts-opus-demux | 2 +-
tests/ref/fate/ts-small-demux | 2 +-
5 files changed, 19 insertions(+), 7 deletions(-)
diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c
index 43bf10db54..9599b3b1b2 100644
--- a/fftools/ffprobe.c
+++ b/fftools/ffprobe.c
@@ -3127,6 +3127,18 @@ static int show_stream(WriterContext *w, AVFormatContext
*fmt_ctx, int stream_id
}
}
+ if (fmt_ctx->iformat->priv_class && show_private_data) {
+ const AVOption *opt = NULL;
+ while (opt = av_opt_next(fmt_ctx->priv_data, opt)) {
+ uint8_t *str;
+ if (!(opt->flags & AV_OPT_FLAG_EXPORT)) continue;
+ if (av_opt_get(fmt_ctx->priv_data, opt->name, 0, &str) >= 0) {
+ print_str(opt->name, str);
+ av_free(str);
+ }
+ }
+ }
maybe factorize with the other similar code?
Sure.
[...]
LGTM otherwise.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
