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

[Thilo Borgmann]

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 ++++++++++++++++++++++++++++++++++++++
>>   libavutil/parseutils.h    | 102 ++++++++++++++++++++++
>>   libavutil/version.h       |   2 +-
>>   6 files changed, 296 insertions(+), 170 deletions(-)
> 
> Absolutely not.
> 
> This is application code and does not belong in the libraries.

How else do we not have a redundant copy of all that and make sure that -stats_* options and the filter understand the same {..} directives?

-Thilo