Re: [FFmpeg-devel] [PATCH v2 2/2] avformat/img2enc: Fix integer truncation when frame_pts is enabled

2024-09-29 Thread Filip Mašić
> > > On Sep 29, 2024, at 08:05, Michael Niedermayer > wrote: > > > > On Tue, Sep 24, 2024 at 12:16:13AM +0800, Zhao Zhili wrote: > >> From: Zhao Zhili > >> > >> --- > >> libavformat/img2enc.c | 8 > >> 1 file changed, 4 insertions(+), 4 deletions(-) > > > > probably ok > > > > thx > > Ap

Re: [FFmpeg-devel] [RFC] 7.1 Release

2024-09-28 Thread Filip Mašić
On Fri, 27 Sept 2024 at 00:51, Michael Niedermayer wrote: > On Thu, Sep 26, 2024 at 12:27:58PM +0100, Filip Mašić wrote: > > On Tue, 24 Sept 2024 at 23:48, Michael Niedermayer < > mich...@niedermayer.cc> > > wrote: > > > > > if someone can help with

[FFmpeg-devel] [RFC] 7.1 Release

2024-09-26 Thread Filip Mašić
On Tue, 24 Sept 2024 at 23:48, Michael Niedermayer wrote: > if someone can help with reviewing and applying, thats certainly welcome > I dont think i will find the time for alot of non trivial patches before > the release but a bigger problem is that the release branch is for bugfixes > mainly no

[FFmpeg-devel] [PATCH v3 1/2] avformat/utils: added av_get_frame_filename3() (changed parameter int -> int64_t)

2024-09-25 Thread Filip Mašić
On Wed, 25 Sept 2024 at 09:59, Marton Balint 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,

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/internal: Add ff_get_frame_filename

2024-09-24 Thread Filip Mašić
On Mon, 23 Sept 2024 at 16:14, Zhao Zhili wrote: > From: Zhao Zhili > > It's similar to av_get_frame_filename2 but with int64_t number > support. Make av_get_frame_filename* a wrapper over > ff_get_frame_filename. > > Co-authored-by: Filip Mašić > --- &

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat: deprecated av_get_frame_filename() and av_get_frame_filename2()

2024-09-23 Thread Filip Mašić
his. On Mon, 23 Sept 2024 at 13:18, Zhao Zhili wrote: > > > On Sep 23, 2024, at 18:18, Filip Mašić wrote: > > --- > doc/APIchanges | 3 ++- > libavfilter/vf_signature.c | 4 ++-- > libavformat/avformat.h | 22 ++ > libavforma

[FFmpeg-devel] [PATCH v3 2/2] avformat: deprecated av_get_frame_filename() and av_get_frame_filename2()

2024-09-23 Thread Filip Mašić
--- doc/APIchanges | 3 ++- libavfilter/vf_signature.c | 4 ++-- libavformat/avformat.h | 22 ++ libavformat/img2dec.c | 10 +- libavformat/segment.c | 4 ++-- libavformat/utils.c | 2 +- libavformat/version_major.h | 2 +- li

[FFmpeg-devel] [PATCH v3 1/2] avformat/utils: added av_get_frame_filename3() (changed parameter int -> int64_t)

2024-09-23 Thread Filip Mašić
Resolves an integer overflow with the frame_pts option (issue 11194). Signed-off-by: Filip Mašić --- doc/APIchanges | 3 +++ libavformat/avformat.h | 7 +-- libavformat/img2enc.c | 6 ++ libavformat/utils.c| 12 +--- libavformat/version.h | 2 +- 5 files

[FFmpeg-devel] [PATCH v2] avformat/utils: added av_get_frame_filename3() (changed parameter int -> int64_t)

2024-09-21 Thread Filip Mašić
ember/333651.html I seem to have been lucky to avoid needing to rebase my changelogs 3 times so that's also ready to go :D. Thanks for your service. On Thu, 19 Sept 2024 at 17:25, Filip Mašić wrote: > Resolves an integer overflow with the frame_pts option (issue 11194). > > Signed

[FFmpeg-devel] [PATCH v2] avformat/utils: added av_get_frame_filename3() (changed parameter int -> int64_t)

2024-09-19 Thread Filip Mašić
Resolves an integer overflow with the frame_pts option (issue 11194). Signed-off-by: Filip Mašić --- doc/APIchanges | 3 +++ libavformat/avformat.h | 5 - libavformat/img2enc.c | 6 ++ libavformat/utils.c| 12 +--- 4 files changed, 18 insertions(+), 8 deletions

[FFmpeg-devel] [PATCH] avformat/img2enc: added av_get_frame_filename3() (changed parameter int -> int64_t)

2024-09-19 Thread Filip Mašić
flags and packages I need to add there or how to. We would also want to update the API changelog and maybe deprecate the older versions of the upgraded function. Thanks for your consideration :). On Monday 16 September 2024, Filip Mašić wrote: > fix for ticket 11194 (frame_pts codepath requi

[FFmpeg-devel] [PATCH] avformat/utils: added av_get_frame_filename3() (changed parameter int -> int64_t)

2024-09-16 Thread Filip Mašić
fix for ticket 11194 (frame_pts codepath requires a larger integer capacity for normal workloads) Signed-off-by: Filip Mašić --- libavformat/avformat.h | 5 - libavformat/img2enc.c | 6 ++ libavformat/utils.c| 12 +--- 3 files changed, 15 insertions(+), 8 deletions

Re: [FFmpeg-devel] [PATCH] av_get_frame_filename2 parameter int -> int64_t

2024-09-16 Thread Filip Mašić
On Mon, 16 Sept 2024 at 19:59, Leo Izen wrote: > On 9/16/24 2:22 PM, Filip Mašić wrote: > > my apologies for re-sending; i didn't spot that there was a header file. > my previous email may be disregarded. > > > > suggested fix for ticket 11194; see ticket for more i

[FFmpeg-devel] [PATCH] av_get_frame_filename2 parameter int -> int64_t

2024-09-16 Thread Filip Mašić
it seems to be a trivial change and easy to test Signed-off-by: Filip Mašić --- libavformat/avformat.h | 4 ++-- libavformat/utils.c| 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index 4a3fb00529..76ffb008b4 100644 --

[FFmpeg-devel] [PATCH] av_get_frame_filename2 parameter int -> int64_t

2024-09-16 Thread Filip Mašić
suggested fix for ticket 11194; see ticket for more info. i haven't been able to test this since i don't have any build tools installed, but decided to submit it since it seems to be a trivial change and easy to test --- libavformat/utils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions