[FFmpeg-devel] [PATCH] avformat/dump: lowercase 'Start' prefix for start offset

[FFmpeg-devel] [PATCH] avformat/dump: lowercase 'Start' prefix for start offset

[Marth64]

Just applying some UX polish.
This is to match the lowercase trend of attributes in
the dump string (and similar to chapters).

Signed-off-by: Marth64 <marth64@proxyid.net>
---
 libavformat/dump.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/dump.c b/libavformat/dump.c
index c263921b99..f22b70b660 100644
--- a/libavformat/dump.c
+++ b/libavformat/dump.c
@@ -680,7 +680,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
 
     if (st->start_time != AV_NOPTS_VALUE && st->start_time != 0 && st->time_base.den && st->time_base.num) {
         const double stream_start = av_q2d(st->time_base) * st->start_time;
-        av_log(NULL, AV_LOG_INFO, ", Start %.6f", stream_start);
+        av_log(NULL, AV_LOG_INFO, ", start %.6f", stream_start);
     }
 
     dump_disposition(st->disposition, log_level);
-- 
2.43.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".

Re: [FFmpeg-devel] [PATCH] avformat/dump: lowercase 'Start' prefix for start offset

[Marton Balint]

On Sun, 15 Jun 2025, Marth64 wrote:

> Just applying some UX polish.
> This is to match the lowercase trend of attributes in
> the dump string (and similar to chapters).
>
> Signed-off-by: Marth64 <marth64@proxyid.net>
> ---
> libavformat/dump.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/dump.c b/libavformat/dump.c
> index c263921b99..f22b70b660 100644
> --- a/libavformat/dump.c
> +++ b/libavformat/dump.c
> @@ -680,7 +680,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
>
>     if (st->start_time != AV_NOPTS_VALUE && st->start_time != 0 && st->time_base.den && st->time_base.num) {
>         const double stream_start = av_q2d(st->time_base) * st->start_time;
> -        av_log(NULL, AV_LOG_INFO, ", Start %.6f", stream_start);
> +        av_log(NULL, AV_LOG_INFO, ", start %.6f", stream_start);
>     }
>
>     dump_disposition(st->disposition, log_level);
> --

LGTM.

Thanks,
Marton
_______________________________________________
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".

Re: [FFmpeg-devel] [PATCH] avformat/dump: lowercase 'Start' prefix for start offset

[Marth64]

Thanks for the ack, pushed
_______________________________________________
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".