Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-14 Thread Thilo Borgmann via ffmpeg-devel
Am 14.12.23 um 18:51 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-14 11:34:11) Am 14.12.23 um 06:23 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 19:17:04) Am 13.12.23 um 17:28 schrieb Anton Khirnov: It is bad practice to design library

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-14 Thread Anton Khirnov
Quoting Thilo Borgmann via ffmpeg-devel (2023-12-14 11:34:11) > Am 14.12.23 um 06:23 schrieb Anton Khirnov: > > Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 19:17:04) > >> Am 13.12.23 um 17:28 schrieb Anton Khirnov: > >>> It is bad practice to design library features around the needs and >

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-14 Thread Thilo Borgmann via ffmpeg-devel
Am 14.12.23 um 06:23 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 19:17:04) Am 13.12.23 um 17:28 schrieb Anton Khirnov: It is bad practice to design library features around the needs and limitations of a single specific caller. The callers here would be the CLI

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Anton Khirnov
Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 19:17:04) > Am 13.12.23 um 17:28 schrieb Anton Khirnov: > > It is bad practice to design library features around the needs and > > limitations of a single specific caller. > > The callers here would be the CLI and this filter. First, public

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Thilo Borgmann via ffmpeg-devel
Am 13.12.23 um 17:28 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:50:09) Am 13.12.23 um 13:39 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:15:27) Am 13.12.23 um 13:08 schrieb Anton Khirnov: Quoting Thilo Borgmann via

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Anton Khirnov
Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:50:09) > Am 13.12.23 um 13:39 schrieb Anton Khirnov: > > Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:15:27) > >> Am 13.12.23 um 13:08 schrieb Anton Khirnov: > >>> Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:05:35) >

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Thilo Borgmann via ffmpeg-devel
Am 13.12.23 um 13:39 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:15:27) Am 13.12.23 um 13:08 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:05:35) Am 13.12.23 um 13:00 schrieb Anton Khirnov: Quoting Thilo Borgmann via

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Anton Khirnov
Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:15:27) > Am 13.12.23 um 13:08 schrieb Anton Khirnov: > > Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:05:35) > >> Am 13.12.23 um 13:00 schrieb Anton Khirnov: > >>> Quoting Thilo Borgmann via ffmpeg-devel (2023-12-11 16:07:22) >

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Thilo Borgmann via ffmpeg-devel
Am 13.12.23 um 13:08 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:05:35) Am 13.12.23 um 13:00 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-11 16:07:22) --- fftools/ffmpeg.h | 31 +-- fftools/ffmpeg_enc.c | 3

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Nicolas George
Thilo Borgmann via ffmpeg-devel (12023-12-11): > --- > fftools/ffmpeg.h | 31 +-- > fftools/ffmpeg_enc.c | 3 +- > fftools/ffmpeg_mux_init.c | 152 +++- > libavutil/parseutils.c| 176 ++ >

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Anton Khirnov
Quoting Thilo Borgmann via ffmpeg-devel (2023-12-13 13:05:35) > Am 13.12.23 um 13:00 schrieb Anton Khirnov: > > Quoting Thilo Borgmann via ffmpeg-devel (2023-12-11 16:07:22) > >> --- > >> fftools/ffmpeg.h | 31 +-- > >> fftools/ffmpeg_enc.c | 3 +- > >>

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Thilo Borgmann via ffmpeg-devel
Am 13.12.23 um 13:00 schrieb Anton Khirnov: Quoting Thilo Borgmann via ffmpeg-devel (2023-12-11 16:07:22) --- fftools/ffmpeg.h | 31 +-- fftools/ffmpeg_enc.c | 3 +- fftools/ffmpeg_mux_init.c | 152 +++- libavutil/parseutils.c| 176

Re: [FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-13 Thread Anton Khirnov
Quoting Thilo Borgmann via ffmpeg-devel (2023-12-11 16:07:22) > --- > fftools/ffmpeg.h | 31 +-- > fftools/ffmpeg_enc.c | 3 +- > fftools/ffmpeg_mux_init.c | 152 +++- > libavutil/parseutils.c| 176 ++ >

[FFmpeg-devel] [PATCH 2/5] fftools/ffmpeg: move parsing of -stats_* specifiers to lavu/parseutils

2023-12-11 Thread Thilo Borgmann via ffmpeg-devel
--- fftools/ffmpeg.h | 31 +-- fftools/ffmpeg_enc.c | 3 +- fftools/ffmpeg_mux_init.c | 152 +++- libavutil/parseutils.c| 176 ++ libavutil/parseutils.h| 102 ++ libavutil/version.h