Message ID | 20240115235802.525608-1-stefasab@gmail.com |
---|---|
State | New |
Headers | show |
Series | [FFmpeg-devel] fftools/opt_common: show if muxer is device | expand |
Context | Check | Description |
---|---|---|
yinshiyou/make_loongarch64 | success | Make finished |
yinshiyou/make_fate_loongarch64 | success | Make fate finished |
andriy/make_x86 | success | Make finished |
andriy/make_fate_x86 | success | Make fate finished |
On date Tuesday 2024-01-16 00:58:02 +0100, Stefano Sabatini wrote: > --- > fftools/opt_common.c | 28 +++++++++++++++++++++------- > 1 file changed, 21 insertions(+), 7 deletions(-) > > diff --git a/fftools/opt_common.c b/fftools/opt_common.c > index f5b73c9f2f..e263a5297e 100644 > --- a/fftools/opt_common.c > +++ b/fftools/opt_common.c > @@ -853,14 +853,23 @@ static int show_formats_devices(void *optctx, const char *opt, const char *arg, > const char *last_name; > int is_dev; > > - printf("%s\n" > - " D. = Demuxing supported\n" > - " .E = Muxing supported\n" > - " --\n", device_only ? "Devices:" : "File formats:"); > + if (device_only) { > + printf("Devices:\n" > + " D. = Demuxing supported\n" > + " .E = Muxing supported\n" > + " --\n"); > + } else { > + printf("Formats:\n" > + " D.. = Demuxing supported\n" > + " .E. = Muxing supported\n" > + " ..d = Is a device\n" > + " ---\n"); > + } > last_name = "000"; > for (;;) { > int decode = 0; > int encode = 0; > + int device = 0; > const char *name = NULL; > const char *long_name = NULL; > > @@ -875,6 +884,7 @@ static int show_formats_devices(void *optctx, const char *opt, const char *arg, > name = ofmt->name; > long_name = ofmt->long_name; > encode = 1; > + device = is_dev; > } > } > } > @@ -889,20 +899,24 @@ static int show_formats_devices(void *optctx, const char *opt, const char *arg, > name = ifmt->name; > long_name = ifmt->long_name; > encode = 0; > + device = is_dev; > } > - if (name && strcmp(ifmt->name, name) == 0) > + if (name && strcmp(ifmt->name, name) == 0) { > decode = 1; > + device = is_dev; > + } > } > } > if (!name) > break; > last_name = name; > > - printf(" %c%c %-15s %s\n", > + printf(" %c%c%s %-15s %s\n", > decode ? 'D' : ' ', > encode ? 'E' : ' ', > + device_only ? "" : (device ? "d" : " "), > name, > - long_name ? long_name:" "); > + long_name ? long_name : " "); > } > return 0; > } Ping about this.
On date Saturday 2024-01-20 12:41:41 +0100, Stefano Sabatini wrote: > On date Tuesday 2024-01-16 00:58:02 +0100, Stefano Sabatini wrote: > > --- > > fftools/opt_common.c | 28 +++++++++++++++++++++------- > > 1 file changed, 21 insertions(+), 7 deletions(-) Ping, will push in a few days if I see no comments.
Quoting Stefano Sabatini (2024-01-16 00:58:02) > --- > fftools/opt_common.c | 28 +++++++++++++++++++++------- > 1 file changed, 21 insertions(+), 7 deletions(-) > > diff --git a/fftools/opt_common.c b/fftools/opt_common.c > index f5b73c9f2f..e263a5297e 100644 > --- a/fftools/opt_common.c > +++ b/fftools/opt_common.c > @@ -853,14 +853,23 @@ static int show_formats_devices(void *optctx, const char *opt, const char *arg, > const char *last_name; > int is_dev; > > - printf("%s\n" > - " D. = Demuxing supported\n" > - " .E = Muxing supported\n" > - " --\n", device_only ? "Devices:" : "File formats:"); > + if (device_only) { > + printf("Devices:\n" > + " D. = Demuxing supported\n" > + " .E = Muxing supported\n" > + " --\n"); > + } else { > + printf("Formats:\n" > + " D.. = Demuxing supported\n" > + " .E. = Muxing supported\n" > + " ..d = Is a device\n" > + " ---\n"); I don't like this duplication.
On date Wednesday 2024-01-24 08:47:52 +0100, Anton Khirnov wrote: > Quoting Stefano Sabatini (2024-01-16 00:58:02) > > --- > > fftools/opt_common.c | 28 +++++++++++++++++++++------- > > 1 file changed, 21 insertions(+), 7 deletions(-) > > > > diff --git a/fftools/opt_common.c b/fftools/opt_common.c > > index f5b73c9f2f..e263a5297e 100644 > > --- a/fftools/opt_common.c > > +++ b/fftools/opt_common.c > > @@ -853,14 +853,23 @@ static int show_formats_devices(void *optctx, const char *opt, const char *arg, > > const char *last_name; > > int is_dev; > > > > - printf("%s\n" > > - " D. = Demuxing supported\n" > > - " .E = Muxing supported\n" > > - " --\n", device_only ? "Devices:" : "File formats:"); > > + if (device_only) { > > + printf("Devices:\n" > > + " D. = Demuxing supported\n" > > + " .E = Muxing supported\n" > > + " --\n"); > > + } else { > > + printf("Formats:\n" > > + " D.. = Demuxing supported\n" > > + " .E. = Muxing supported\n" > > + " ..d = Is a device\n" > > + " ---\n"); > > I don't like this duplication. will apply with the code re-factored
diff --git a/fftools/opt_common.c b/fftools/opt_common.c index f5b73c9f2f..e263a5297e 100644 --- a/fftools/opt_common.c +++ b/fftools/opt_common.c @@ -853,14 +853,23 @@ static int show_formats_devices(void *optctx, const char *opt, const char *arg, const char *last_name; int is_dev; - printf("%s\n" - " D. = Demuxing supported\n" - " .E = Muxing supported\n" - " --\n", device_only ? "Devices:" : "File formats:"); + if (device_only) { + printf("Devices:\n" + " D. = Demuxing supported\n" + " .E = Muxing supported\n" + " --\n"); + } else { + printf("Formats:\n" + " D.. = Demuxing supported\n" + " .E. = Muxing supported\n" + " ..d = Is a device\n" + " ---\n"); + } last_name = "000"; for (;;) { int decode = 0; int encode = 0; + int device = 0; const char *name = NULL; const char *long_name = NULL; @@ -875,6 +884,7 @@ static int show_formats_devices(void *optctx, const char *opt, const char *arg, name = ofmt->name; long_name = ofmt->long_name; encode = 1; + device = is_dev; } } } @@ -889,20 +899,24 @@ static int show_formats_devices(void *optctx, const char *opt, const char *arg, name = ifmt->name; long_name = ifmt->long_name; encode = 0; + device = is_dev; } - if (name && strcmp(ifmt->name, name) == 0) + if (name && strcmp(ifmt->name, name) == 0) { decode = 1; + device = is_dev; + } } } if (!name) break; last_name = name; - printf(" %c%c %-15s %s\n", + printf(" %c%c%s %-15s %s\n", decode ? 'D' : ' ', encode ? 'E' : ' ', + device_only ? "" : (device ? "d" : " "), name, - long_name ? long_name:" "); + long_name ? long_name : " "); } return 0; }