[FFmpeg-devel] [PATCH] ffprobe: remove ad-hoc behavior when nb_fmt_entries_to_show is 1

Stefano Sabatini stefasab at gmail.com
Sun May 20 16:10:15 CEST 2012


The special-case behavior may complicate parsing when the -show_fmt_entry
option is used programmatically in a script.

The option default=nk=1 achieves the same purpose, if the objective is to
skip printing the single field key.
---
 ffprobe.c         |    5 -----
 tests/fate-run.sh |    2 +-
 2 files changed, 1 insertions(+), 6 deletions(-)

diff --git a/ffprobe.c b/ffprobe.c
index d76f0e2..d53ec11 100644
--- a/ffprobe.c
+++ b/ffprobe.c
@@ -51,7 +51,6 @@ static int do_show_error   = 0;
 static int do_show_format  = 0;
 static int do_show_frames  = 0;
 static AVDictionary *fmt_entries_to_show = NULL;
-static int nb_fmt_entries_to_show;
 static int do_show_packets = 0;
 static int do_show_streams = 0;
 static int do_show_program_version  = 0;
@@ -399,9 +398,6 @@ static av_cold int default_init(WriterContext *wctx, const char *args, void *opa
     def->class = &default_class;
     av_opt_set_defaults(def);
 
-    if (nb_fmt_entries_to_show == 1)
-        def->nokey = 1;
-
     if (args &&
         (err = (av_set_options_string(def, args, "=", ":"))) < 0) {
         av_log(wctx, AV_LOG_ERROR, "Error parsing options string: '%s'\n", args);
@@ -1679,7 +1675,6 @@ static int opt_format(const char *opt, const char *arg)
 static int opt_show_format_entry(const char *opt, const char *arg)
 {
     do_show_format = 1;
-    nb_fmt_entries_to_show++;
     av_dict_set(&fmt_entries_to_show, arg, "", 0);
     return 0;
 }
diff --git a/tests/fate-run.sh b/tests/fate-run.sh
index 126f2ce..cd94d4a 100755
--- a/tests/fate-run.sh
+++ b/tests/fate-run.sh
@@ -70,7 +70,7 @@ run(){
 }
 
 probefmt(){
-    run ffprobe -show_format_entry format_name -print_format default=nw=1 -v 0 "$@"
+    run ffprobe -show_format_entry format_name -print_format default=nw=1:nk=1 -v 0 "$@"
 }
 
 avconv(){
-- 
1.7.5.4



More information about the ffmpeg-devel mailing list