Re: [FFmpeg-devel] [PATCH 2/3] avformat/imfdec: fail on probing non xml file extension

2023-05-07 Thread Pierre-Anthony Lemieux
On Sun, May 7, 2023 at 12:18 PM Michael Niedermayer wrote: > > On Sat, May 06, 2023 at 11:01:20AM -0700, Pierre-Anthony Lemieux wrote: > > On Sat, May 6, 2023 at 6:25 AM Michael Niedermayer > > wrote: > > > > > > Its unexpected that a .avi or other "standard" file turns into a playlist. > > >

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/bitpacked_enc: add support for uyvy422 encoding

2023-05-07 Thread Lance Wang
On Mon, May 8, 2023 at 6:56 AM James Almer wrote: > On 5/7/2023 4:00 AM, Lance Wang wrote: > > On Sun, May 7, 2023 at 1:38 AM James Almer wrote: > > > >> Signed-off-by: James Almer > >> --- > >> libavcodec/bitpacked_enc.c | 27 ++- > >> 1 file changed, 26

Re: [FFmpeg-devel] [PATCH] avfilter: add zoneplate video source filter

2023-05-07 Thread Paul B Mahol
New version, faster and with slice threading and docs. From 8b9ab6e3401d69f115b5d331fec73fd8c01ea1bd Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sat, 6 May 2023 22:52:47 +0200 Subject: [PATCH] avfilter: add zoneplate video test source Signed-off-by: Paul B Mahol --- doc/filters.texi

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/bitpacked_enc: add support for uyvy422 encoding

2023-05-07 Thread James Almer
On 5/7/2023 4:00 AM, Lance Wang wrote: On Sun, May 7, 2023 at 1:38 AM James Almer wrote: Signed-off-by: James Almer --- libavcodec/bitpacked_enc.c | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/libavcodec/bitpacked_enc.c

Re: [FFmpeg-devel] [PATCH] fftools/ffprobe: print exported stream AVOptions

2023-05-07 Thread Stefano Sabatini
On date Sunday 2023-05-07 19:30:03 -0300, James Almer wrote: > On 5/7/2023 6:20 PM, Stefano Sabatini wrote: > > On date Friday 2023-05-05 20:18:56 -0300, James Almer wrote: > > > Similar to the decoder AVOptions above. > > > > Please briefly specify the use case in the log. > > A line about how

Re: [FFmpeg-devel] [PATCH] avfilter/setpts: add command support

2023-05-07 Thread Stefano Sabatini
On date Monday 2023-05-01 22:01:05 +0100, Oleg Afanasyev wrote: > I'm using setpts to generate timelapses with slowdowns in the middle. > Using setpts filter requires complicated expr to handle intervals. This > patch allows commands to change expr and also adds a constant that > provides time of

Re: [FFmpeg-devel] [PATCH] fftools/ffprobe: print exported stream AVOptions

2023-05-07 Thread James Almer
On 5/7/2023 6:20 PM, Stefano Sabatini wrote: On date Friday 2023-05-05 20:18:56 -0300, James Almer wrote: Similar to the decoder AVOptions above. Please briefly specify the use case in the log. A line about how these are options whose values are changed by the demuxer and thus exported

Re: [FFmpeg-devel] [PATCH] avfilter: add zoneplate video source filter

2023-05-07 Thread Stefano Sabatini
On date Sunday 2023-05-07 18:04:37 +0200, Paul B Mahol wrote: > Attached. > From 7b0a8586adc0a142f0b7afcdbdf36ce526f4cd34 Mon Sep 17 00:00:00 2001 > From: Paul B Mahol > Date: Sat, 6 May 2023 22:52:47 +0200 > Subject: [PATCH] avfilter: add zoneplate video test source > > Signed-off-by: Paul B

Re: [FFmpeg-devel] Embedded documentation?

2023-05-07 Thread Stefano Sabatini
On date Monday 2023-05-01 12:13:09 +0200, Nicolas George wrote: > Hi. > > Three years ago, I shared some brief thoughts about embedding the > documentation in the libraries. For example, that would allow GUI > applications to open help dialogs about specific options. > > To see what it would

Re: [FFmpeg-devel] tiny documentation fix

2023-05-07 Thread Stefano Sabatini
On date Sunday 2023-05-07 07:56:01 -0500, Jonathan Gilbert wrote: > Hello, > > I found a mistake in the documentation, a parameter that was copy/pasted > but not updated. > > In filters.texi, the documentation for the colorcorrect filter describes > "rh" as "red highlight spot" and "bh" also as

Re: [FFmpeg-devel] [PATCH] fftools/ffprobe: print exported stream AVOptions

2023-05-07 Thread Stefano Sabatini
On date Friday 2023-05-05 20:18:56 -0300, James Almer wrote: > Similar to the decoder AVOptions above. Please briefly specify the use case in the log. > Signed-off-by: James Almer > --- > fftools/ffprobe.c | 12 > tests/ref/fate/flv-demux | 4 ++-- >

Re: [FFmpeg-devel] [PATCH] avformat/hls: fail on probing non hls/m3u8 file extensions

2023-05-07 Thread Anton Khirnov
Quoting Michael Niedermayer (2023-05-03 14:30:38) > Its unexpected that a .avi or other "standard" file turns into a playlist. > The goal of this patch is to avoid this unexpected behavior and possible > privacy or security differences. > I very much dislike this approach. -- Anton Khirnov

Re: [FFmpeg-devel] [PATCH 1/3] avformat/dashdec: fail on probing non mpd file extension

2023-05-07 Thread Anton Khirnov
Quoting Michael Niedermayer (2023-05-06 15:25:01) > Its unexpected that a .avi or other "standard" file turns into a playlist. > The goal of this patch is to avoid this unexpected behavior and possible > privacy or security differences. > > This is similar to the same change to hls I heavily

Re: [FFmpeg-devel] [PATCH 2/3] avformat/imfdec: fail on probing non xml file extension

2023-05-07 Thread Michael Niedermayer
On Sat, May 06, 2023 at 11:01:20AM -0700, Pierre-Anthony Lemieux wrote: > On Sat, May 6, 2023 at 6:25 AM Michael Niedermayer > wrote: > > > > Its unexpected that a .avi or other "standard" file turns into a playlist. > > The goal of this patch is to avoid this unexpected behavior and possible > >

Re: [FFmpeg-devel] [PATCH 01/13] lavu/frame: extend AVFrame.repeat_pict documentation

2023-05-07 Thread Anton Khirnov
Quoting Kieran Kunhya (2023-05-07 18:59:22) > On Sun, 7 May 2023, 14:34 Anton Khirnov, wrote: > > > --- > > libavutil/frame.h | 18 -- > > 1 file changed, 16 insertions(+), 2 deletions(-) > > > > diff --git a/libavutil/frame.h b/libavutil/frame.h > > index f2b56beebb..ed3f199ce1

Re: [FFmpeg-devel] [PATCH 01/13] lavu/frame: extend AVFrame.repeat_pict documentation

2023-05-07 Thread Kieran Kunhya
On Sun, 7 May 2023, 14:34 Anton Khirnov, wrote: > --- > libavutil/frame.h | 18 -- > 1 file changed, 16 insertions(+), 2 deletions(-) > > diff --git a/libavutil/frame.h b/libavutil/frame.h > index f2b56beebb..ed3f199ce1 100644 > --- a/libavutil/frame.h > +++ b/libavutil/frame.h

[FFmpeg-devel] [PATCH] avfilter: add zoneplate video source filter

2023-05-07 Thread Paul B Mahol
Attached. From 7b0a8586adc0a142f0b7afcdbdf36ce526f4cd34 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sat, 6 May 2023 22:52:47 +0200 Subject: [PATCH] avfilter: add zoneplate video test source Signed-off-by: Paul B Mahol --- libavfilter/Makefile | 1 + libavfilter/allfilters.c |

Re: [FFmpeg-devel] [PATCH] avdevice/lavfi: update documentation

2023-05-07 Thread Paul B Mahol
approved and applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH] avdevice/lavfi: update documentation

2023-05-07 Thread metamuffin
lavfi not only supports video but also audio by now. --- doc/indevs.texi | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/doc/indevs.texi b/doc/indevs.texi index 8a198c4b44..863536a34d 100644 --- a/doc/indevs.texi +++ b/doc/indevs.texi @@ -991,9 +991,8 @@ This input device

[FFmpeg-devel] [PATCH 03/13] lavc/codec_desc: add a property for codecs that support field coding

2023-05-07 Thread Anton Khirnov
Multiple places currently use AVCodecContext.ticks_per_frame > 1 to identify such codecs, which * requires a codec context * requires it to be open --- doc/APIchanges | 3 +++ libavcodec/codec_desc.c | 16 libavcodec/codec_desc.h | 6 ++ libavcodec/version.h|

[FFmpeg-devel] [PATCH 02/13] fftools/ffmpeg: fix computing video frame duration from repeat_pict

2023-05-07 Thread Anton Khirnov
This field contains the number of _field_ durations by which the standard frame duration should be extended. --- fftools/ffmpeg.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 8323c32ffd..4e45ab74b9 100644 ---

[FFmpeg-devel] [PATCH 04/13] lavf: use AV_CODEC_PROP_FIELDS where appropriate

2023-05-07 Thread Anton Khirnov
H.264 and mpeg12 parsers need to be adjusted at the same time to stop using the value of AVCodecContext.ticks_per_frame, because it is not set correctly unless the codec has been opened. Previously this would result in both the parser and lavf seeing the same incorrect value, which would cancel

[FFmpeg-devel] [PATCH 12/13] lavc: deprecate AVCodecContext.ticks_per_frame

2023-05-07 Thread Anton Khirnov
For encoding, this field is entirely redundant with AVCodecContext.framerate. For decoding, this field is entirely redundant with AV_CODEC_PROP_FIELDS. --- doc/codecs.texi | 1 - libavcodec/amfenc_av1.c | 8 +++- libavcodec/amfenc_h264.c | 8 +++-

[FFmpeg-devel] [PATCH 11/13] lavc/libvpxenc: send frame durations to the encoder

2023-05-07 Thread Anton Khirnov
Adapt similar code from libaomenc - stop using ticks_per_frame except as a last resort. --- libavcodec/libvpxenc.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index a20e949842..a89497665b 100644 ---

[FFmpeg-devel] [PATCH 13/13] fftools/ffmpeg: stop using deprecated ticks_per_frame

2023-05-07 Thread Anton Khirnov
--- fftools/ffmpeg.c | 13 +++-- fftools/ffmpeg.h | 1 + fftools/ffmpeg_demux.c | 2 ++ 3 files changed, 10 insertions(+), 6 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 4e45ab74b9..a731b4bd7d 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@

[FFmpeg-devel] [PATCH 08/13] lavc/msmpeg4enc: use AVCodecContext.framerate when available

2023-05-07 Thread Anton Khirnov
--- libavcodec/msmpeg4enc.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavcodec/msmpeg4enc.c b/libavcodec/msmpeg4enc.c index 54121438a0..df190d376c 100644 --- a/libavcodec/msmpeg4enc.c +++ b/libavcodec/msmpeg4enc.c @@ -280,7 +280,13 @@ void

[FFmpeg-devel] [PATCH 01/13] lavu/frame: extend AVFrame.repeat_pict documentation

2023-05-07 Thread Anton Khirnov
--- libavutil/frame.h | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/libavutil/frame.h b/libavutil/frame.h index f2b56beebb..ed3f199ce1 100644 --- a/libavutil/frame.h +++ b/libavutil/frame.h @@ -491,8 +491,22 @@ typedef struct AVFrame { void *opaque;

[FFmpeg-devel] [PATCH 10/13] lavc/libkvazaar, libopenh264enc: drop redundant checks

2023-05-07 Thread Anton Khirnov
The same check is present in encode_preinit_video(). --- libavcodec/libkvazaar.c | 5 - libavcodec/libopenh264enc.c | 5 - 2 files changed, 10 deletions(-) diff --git a/libavcodec/libkvazaar.c b/libavcodec/libkvazaar.c index 168486f4ec..b3faa0a64b 100644 --- a/libavcodec/libkvazaar.c

[FFmpeg-devel] [PATCH 09/13] libaomenc: use AVCodecContext.framerate when available

2023-05-07 Thread Anton Khirnov
--- libavcodec/libaomenc.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/libavcodec/libaomenc.c b/libavcodec/libaomenc.c index 0b88102c77..1d3a4ae64c 100644 --- a/libavcodec/libaomenc.c +++ b/libavcodec/libaomenc.c @@ -1295,8 +1295,12 @@ static int

[FFmpeg-devel] [PATCH 05/13] lavc/av1*: fix exporting framerate

2023-05-07 Thread Anton Khirnov
* take num_ticks_per_picture_minus_1 into account, since that is a part of the framerate computation * stop exporting num_ticks_per_picture_minus_1 into AVCodecContext.ticks_per_frame, as that field is used for other purposes (in conjunction with repeat_pict, which is not used at all by

[FFmpeg-devel] [PATCH 07/13] lavc/ratecontrol: use AVCodecContext.framerate when available

2023-05-07 Thread Anton Khirnov
--- libavcodec/ratecontrol.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/ratecontrol.c b/libavcodec/ratecontrol.c index 6a40f9cbdc..890ae33cb3 100644 --- a/libavcodec/ratecontrol.c +++ b/libavcodec/ratecontrol.c @@ -57,6 +57,9 @@ void ff_write_pass1_stats(MpegEncContext *s)

[FFmpeg-devel] [PATCH 06/13] lavc/libdav1d: fix exporting framerate

2023-05-07 Thread Anton Khirnov
Same issues as in the previous commit. --- libavcodec/libdav1d.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/libdav1d.c b/libavcodec/libdav1d.c index 87aed16749..19b6a880da 100644 --- a/libavcodec/libdav1d.c +++ b/libavcodec/libdav1d.c @@ -154,11 +154,11

[FFmpeg-devel] tiny documentation fix

2023-05-07 Thread Jonathan Gilbert
Hello, I found a mistake in the documentation, a parameter that was copy/pasted but not updated. In filters.texi, the documentation for the colorcorrect filter describes "rh" as "red highlight spot" and "bh" also as "red highlight spot". I'm reasonably confident it should say "blue highlight

[FFmpeg-devel] [PATCH] avdevice/lavfi: fix crash on unconnected outpads

2023-05-07 Thread metamuffin
Nameless outpads would cause an invocation to sscanf with NULL. Example: ffmpeg -f lavfi -i 'nullsrc;nullsrc' - Changed to throwing an error instead. --- libavdevice/lavfi.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c index

Re: [FFmpeg-devel] [PATCH v2] avcodec/mediacodec: Add AV1 encoder

2023-05-07 Thread Zhao Zhili
On Tue, 2023-05-02 at 12:22 +, Samuel Raposo Vieira Mira wrote: > @@ -3162,6 +3162,7 @@ aac_mf_encoder_deps="mediafoundation" > ac3_mf_encoder_deps="mediafoundation" > av1_cuvid_decoder_deps="cuvid CUVIDAV1PICPARAMS" > av1_mediacodec_decoder_deps="mediacodec" >

Re: [FFmpeg-devel] [PATCH] avcodec/mediacodec: Add VP8 encoder

2023-05-07 Thread Zhao Zhili
On Fri, 2023-04-28 at 11:07 +, Samuel Raposo Vieira Mira wrote: > Connected FFmpeg to Mediacodec VP8 encoder. > Minor Version bump. > --- > configure | 1 + > libavcodec/Makefile | 1 + > libavcodec/allcodecs.c | 1 + > libavcodec/mediacodec_wrapper.c

Re: [FFmpeg-devel] [PATCH v6 1/6] ccfifo: Properly handle CEA-708 captions through framerate conversion

2023-05-07 Thread Lance Wang
LGTM for the patchset. On Sat, May 6, 2023 at 2:13 AM Devin Heitmueller < devin.heitmuel...@ltnglobal.com> wrote: > When transcoding video that contains 708 closed captions, the > caption data is tied to the frames as side data. Simply dropping > or adding frames to change the framerate will

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/bitpacked_enc: add support for uyvy422 encoding

2023-05-07 Thread Lance Wang
On Sun, May 7, 2023 at 1:38 AM James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/bitpacked_enc.c | 27 ++- > 1 file changed, 26 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/bitpacked_enc.c b/libavcodec/bitpacked_enc.c > index