[FFmpeg-devel] [PATCH v3 1/2] avformat/utils: added av_get_frame_filename3() (changed parameter int -> int64_t)
Filip Mašić
shoutplenty at gmail.com
Wed Sep 25 12:26:01 EEST 2024
On Wed, 25 Sept 2024 at 09:59, Marton Balint <cus at passwd.hu> wrote:
> In general AVBPrint based API for functions like this is preferred to
> discuourage users from limiting the filenames to a fixed size (which
> usually becomes a limitation sooner or later).
>
> E.g: av_get_frame_filename_bprint()
>
> Regards,
> Marton
>
Currently, this patch to reimplement the function internally is preferred:
https://ffmpeg.org/pipermail/ffmpeg-devel/2024-September/333854.html
https://ffmpeg.org/pipermail/ffmpeg-devel/2024-September/333857.html
That's because it's been suggested by a maintainer and doesn't change the
API/ABI so we can pray it will be merged into the upcoming release and the
bug it was written for finally fixed.
Regarding the API, I don't know what purpose the function serves externally
but in my patch, I did attempt to deprecate all the previous API functions,
so you're welcome to extend it from there and make the replacement
interface use AVBPrint (idk much about that). I'm only interested in the
bugfix I mentioned.
Thanks for your interest in this patch :)
More information about the ffmpeg-devel
mailing list