On 2025-02-23 01:06 pm, Gyan Doshi wrote:
On 2025-02-21 05:34 pm, Gyan Doshi wrote:
On 2025-02-21 04:34 pm, Andreas Rheinhardt wrote:
Gyan Doshi:
On 2025-02-21 02:21 pm, Andreas Rheinhardt wrote:
Gyan Doshi:
---
fftools/opt_common.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/fftools/opt_common.c b/fftools/opt_common.c
index 317e8458c1..2ac3fd4fb3 100644
--- a/fftools/opt_common.c
+++ b/fftools/opt_common.c
@@ -1330,6 +1330,11 @@ int opt_loglevel(void *optctx, const char
*opt, const char *arg)
"Possible levels are numbers or:\n", arg);
for (i = 0; i < FF_ARRAY_ELEMS(log_levels); i++)
av_log(NULL, AV_LOG_FATAL, "\"%s\"\n",
log_levels[i].name);
+ av_log(NULL, AV_LOG_FATAL, "Possible flags are:\n");
+ av_log(NULL, AV_LOG_FATAL, "\"repeat\"\n");
+ av_log(NULL, AV_LOG_FATAL, "\"level\"\n");
+ av_log(NULL, AV_LOG_FATAL, "\"time\"\n");
+ av_log(NULL, AV_LOG_FATAL, "\"datetime\"\n");
return AVERROR(EINVAL);
}
This would all fit into one line.
Do you mean a single log call, or also printed on one line?
If the former, the remaining values will be misaligned with the
first one.
If the latter, it's prettier with one value per line, just like the
log
level values printed just above.
I meant the latter. And I actually prefer the more compact form
"repeat", "level", "time", "datetime" to the one-value-per-line form.
I think the per-line approach is visually clearer, so I'd like to
keep it as it is. You can submit a patch to reformat it if you like.
Plan to push tomorrow.
Pushed as a060b4d9a9e6ab627af5275253a1b5baa4fc2ea0
Regards,
Gyan
_______________________________________________
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".
