[FFmpeg-devel] [PATCH] Check if we do have also a filter_complex filter.

2019-03-27 Thread Thierry Foucu
Right now, the code check for no filter description, but if we use a filter_complex, the code will use the AVFrame.duration which could be wrong in case of using fps filter. How to reproduce the problem: ffmpeg -f lavfi -i testsrc=duration=1 -vf fps=fps=50 -vsync 1 -f null - output 50 frames ffmp

[FFmpeg-devel] [PATCH] [cbs_h2645]: Used av_realloc instead of av_malloc

2019-08-26 Thread Thierry Foucu
Follow the description of av_realloc, the memory needs to be allocated by av_realloc. --- libavcodec/cbs_h2645.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cbs_h2645.c b/libavcodec/cbs_h2645.c index 69ea6dc6bb..8da8421e47 100644 --- a/libavcodec/cbs_h2645.c +++

Re: [FFmpeg-devel] [PATCH] [cbs_h2645]: Used av_realloc instead of av_malloc

2019-08-28 Thread Thierry Foucu
On Tue, Aug 27, 2019 at 6:39 AM Paul B Mahol wrote: > Probably OK > Thanks Paul. Any change to have someone to push it? Cheers > > On Mon, Aug 26, 2019 at 10:51 PM Thierry Foucu wrote: > > > Follow the description of av_realloc, the memory needs to be alloca

Re: [FFmpeg-devel] [PATCH v3] avcodec: libdav1d AV1 decoder wrapper.

2018-10-26 Thread Thierry Foucu
On Wed, Oct 24, 2018 at 4:02 AM Rostislav Pehlivanov wrote: > On Sat, 20 Oct 2018 at 00:50, James Almer wrote: > > > Originally written by Ronald S. Bultje, with fixes, optimizations and > > improvements by James Almer. > > > > Signed-off-by: James Almer > > --- > > Updated to work with libdav1

Re: [FFmpeg-devel] [PATCH] libaomenc: Add support for tiles

2018-10-26 Thread Thierry Foucu
On Mon, Oct 22, 2018 at 3:45 PM Mark Thompson wrote: > Adds an option to specify the number of tile rows and columns, then uses > a uniform tiling if possible and otherwise a fixed tiling with equal-sized > tiles to fill the frame. > > Also adds -tile-columns and -tile-rows options to make tiling

Re: [FFmpeg-devel] [PATCH 03/12] lavfi: drop vf_qp

2020-02-26 Thread Thierry Foucu
n-Baptiste Kempf - President > +33 672 704 734 > ___ > 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-devel] [PATCH] libavformat/matroskaenc.c: Check if we can seek before writing the end of the ebml of the segment.

2020-03-10 Thread Thierry Foucu
--- libavformat/matroskaenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 42f21eae8b..cf436f9e3e 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -2630,7 +2630,7 @@ static int mkv_write_tra

[FFmpeg-devel] [PATCH] configure: Add llviddsp to mvha deps

2020-03-10 Thread Thierry Foucu
--- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 8b17134944..2d2e4d8817 100755 --- a/configure +++ b/configure @@ -2790,7 +2790,7 @@ msmpeg4v3_decoder_select="h263_decoder" msmpeg4v3_encoder_select="h263_encoder" mss2_decoder_select="m

Re: [FFmpeg-devel] [PATCH] libavformat/matroskaenc.c: Check if we can seek before writing the end of the ebml of the segment.

2020-03-11 Thread Thierry Foucu
On Wed, Mar 11, 2020 at 10:13 AM Andreas Rheinhardt < andreas.rheinha...@gmail.com> wrote: > Thierry Foucu: > > --- > > libavformat/matroskaenc.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavformat/matroskaenc.

Re: [FFmpeg-devel] [PATCH] configure: Add llviddsp to mvha deps

2020-03-12 Thread Thierry Foucu
___ > 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". -- Thierry Foucu

[FFmpeg-devel] [PATCH] configure: Add llviddsp to mvha select

2020-03-12 Thread Thierry Foucu
--- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 8b17134944..6ceb0c7af4 100755 --- a/configure +++ b/configure @@ -2791,6 +2791,7 @@ msmpeg4v3_encoder_select="h263_encoder" mss2_decoder_select="mpegvideo qpeldsp vc1_decoder" mts2_decoder_select="mss34

Re: [FFmpeg-devel] [PATCH] libavformat/matroskaenc.c: Check if we can seek before writing the end of the ebml of the segment.

2020-03-12 Thread Thierry Foucu
Thanks Andreas On Wed, Mar 11, 2020 at 5:35 PM Andreas Rheinhardt < andreas.rheinha...@gmail.com> wrote: > Thierry Foucu: > > On Wed, Mar 11, 2020 at 10:13 AM Andreas Rheinhardt < > > andreas.rheinha...@gmail.com> wrote: > > > >> Thierry Foucu: >

Re: [FFmpeg-devel] [PATCH]lavc/sbc: Remove bool usage

2020-04-01 Thread Thierry Foucu
; > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". -- Thierry Foucu ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavc/sbc: Remove bool usage

2020-04-02 Thread Thierry Foucu
(Sorry, I am not familiar with PPC...) > > - Derek > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmp

Re: [FFmpeg-devel] [PATCH]lavc/sbc: Remove bool usage

2020-04-02 Thread Thierry Foucu
On Thu, Apr 2, 2020 at 11:18 AM Thierry Foucu wrote: > > > On Thu, Apr 2, 2020 at 11:11 AM Derek Buitenhuis < > derek.buitenh...@gmail.com> wrote: > >> On 02/04/2020 17:47, Moritz Barsnick wrote: >> > I though this was the practical argument?: >> >

Re: [FFmpeg-devel] [PATCH 1/9] libavutil: add API for exporting video frame quantizers

2020-05-07 Thread Thierry Foucu
n/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". -- Thierry Foucu ___ 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] [libavformat/moc.c] Read the QT Metadata Keys only once

2020-05-07 Thread Thierry Foucu
If you have a file with multiple Metadata Keys, the second time you parse the keys, you will re-alloc c->meta_keys without freeing the old one. This change will avoid parsing all the consecutive Metadata keys. --- libavformat/mov.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --gi

[FFmpeg-devel] [PATCH] [libavformat/mov.c] Read the QT Metadata Keys only once

2020-05-11 Thread Thierry Foucu
If you have a file with multiple Metadata Keys, the second time you parse the keys, you will re-alloc c->meta_keys without freeing the old one. This change will avoid parsing all the consecutive Metadata keys. --- libavformat/mov.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --gi

Re: [FFmpeg-devel] [PATCH] [libavformat/mov.c] Read the QT Metadata Keys only once

2020-05-13 Thread Thierry Foucu
On Mon, May 11, 2020 at 9:35 AM Thierry Foucu wrote: > If you have a file with multiple Metadata Keys, the second time you parse > the keys, you will re-alloc c->meta_keys without freeing the old one. > This change will avoid parsing all the consecutive Metadata keys. > --- >

Re: [FFmpeg-devel] [PATCH] [libavformat/mov.c] Read the QT Metadata Keys only once

2020-05-14 Thread Thierry Foucu
On Thu, May 14, 2020 at 5:09 AM Derek Buitenhuis wrote: > On 11/05/2020 17:35, Thierry Foucu wrote: > > If you have a file with multiple Metadata Keys, the second time you parse > > the keys, you will re-alloc c->meta_keys without freeing the old one. > > This change wi

Re: [FFmpeg-devel] [PATCH] [libavformat/mov.c] Read the QT Metadata Keys only once

2020-05-15 Thread Thierry Foucu
On Thu, May 14, 2020 at 12:26 PM Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 14/05/2020 18:19, Thierry Foucu wrote: > > Looking at > > > https://developer.apple.com/library/archive/documentation/QuickTime/QTFF/Metadata/Metadata.html > > The spec does

[FFmpeg-devel] [PATCH] libavcodec/png_parser.c: fix a use_of_uninitialized_value in target_dec_fuzzer.

2020-06-04 Thread Thierry Foucu
target_dec_fuzzer is checking for the avpkt.data pointer but if the png parser cannot combine the frame, the poutbuf is not set and so, the avpkt.data is not initialized. --- libavcodec/png_parser.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/png_parser.c b/l

[FFmpeg-devel] [PATCH 1/3] libavcodec/png_parser.c: fix a use_of_uninitialized_value in target_dec_fuzzer.

2020-06-04 Thread Thierry Foucu
the target_dec_fuzzer is checking for the avpkt.data pointer but if the png parser cannot combine the frame, the poutbuf is not set. --- libavcodec/png_parser.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/png_parser.c b/libavcodec/png_parser.c index 74f2964118..9ec8551a1b 100644

[FFmpeg-devel] [PATCH 2/3] libavcodec/mlp_parser.c: fix a use_of_uninitialized_value in target_dec_fuzzer.

2020-06-04 Thread Thierry Foucu
the target_dec_fuzzer is checking for the avpkt.data pointer but if the mlp parser cannot combine the frame, the poutbuf is not set. --- libavcodec/mlp_parser.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/mlp_parser.c b/libavcodec/mlp_parser.c index 5d2ddc5a70..e7162f4aa8 100644

[FFmpeg-devel] [PATCH 3/3] libavcodec/bmp_parser.c: fix a use_of_uninitialized_value in target_dec_fuzzer.

2020-06-04 Thread Thierry Foucu
the target_dec_fuzzer is checking for the avpkt.data pointer but if the bmp parser cannot combine the frame, the poutbuf is not set. --- libavcodec/bmp_parser.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/bmp_parser.c b/libavcodec/bmp_parser.c index cd65f02a2e..700bf27af1 100644

Re: [FFmpeg-devel] [PATCH] libavcodec/png_parser.c: fix a use_of_uninitialized_value in target_dec_fuzzer.

2020-06-04 Thread Thierry Foucu
On Thu, Jun 4, 2020 at 11:52 AM James Almer wrote: > On 6/4/2020 3:40 PM, Thierry Foucu wrote: > > target_dec_fuzzer is checking for the avpkt.data pointer but if the > > png parser cannot combine the frame, the poutbuf is not set and so, the > > avpkt.da

Re: [FFmpeg-devel] [PATCH] avfilter/vf_fps: Avoid inlink fifo build up.

2019-09-08 Thread Thierry Foucu
ping? On Wed, Aug 28, 2019 at 7:14 PM Nikolas Bowe wrote: > When duplicating frames we need to schedule for activation again, > otherwise frames can build up in the inlink fifo. > --- > libavfilter/vf_fps.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/vf_

[FFmpeg-devel] [PATCH 2/2] [monvenc] Add extra padding when allocating trk->vos_data

2019-09-26 Thread Thierry Foucu
trk->vos_data is mostly used to store the extradata from the codec. Most encoder when storing their extradata, are allocating with padding. But the current code was ignoring the padding, which could causes heap-buffer-overflow --- libavformat/movenc.c | 2 +- 1 file changed, 1 insertion(+), 1 dele

Re: [FFmpeg-devel] [PATCH] aformat/movenc: add missing padding to output track extradata

2019-09-26 Thread Thierry Foucu
On Wed, Sep 25, 2019 at 10:30 AM James Almer wrote: > Fixes ticket #8183. > > Signed-off-by: James Almer > --- > libavformat/movenc.c | 15 ++- > 1 file changed, 10 insertions(+), 5 deletions(-) > > diff --git a/libavformat/movenc.c b/libavformat/movenc.c > index c824ff5ba1..e328387

Re: [FFmpeg-devel] [PATCH 2/2] [monvenc] Add extra padding when allocating trk->vos_data

2019-09-26 Thread Thierry Foucu
On Thu, Sep 26, 2019 at 11:00 AM James Almer wrote: > On 9/26/2019 2:58 PM, Thierry Foucu wrote: > > trk->vos_data is mostly used to store the extradata from the codec. > > Most encoder when storing their extradata, are allocating with padding. > > But the current code

[FFmpeg-devel] [PATCH] [libdav1d.c]: Add option to output all the spatial layers.

2019-11-14 Thread Thierry Foucu
Set the option to false by default, to match libaomdec wrapper. --- libavcodec/libdav1d.c | 4 1 file changed, 4 insertions(+) diff --git a/libavcodec/libdav1d.c b/libavcodec/libdav1d.c index cf4b178f1d..5efa8eeb48 100644 --- a/libavcodec/libdav1d.c +++ b/libavcodec/libdav1d.c @@ -40,6 +40,7

Re: [FFmpeg-devel] [PATCH] [libdav1d.c]: Add option to output all the spatial layers.

2019-11-14 Thread Thierry Foucu
On Thu, Nov 14, 2019 at 9:20 AM James Almer wrote: > On 11/14/2019 2:15 PM, Thierry Foucu wrote: > > Set the option to false by default, to match libaomdec wrapper. > > --- > > libavcodec/libdav1d.c | 4 > > 1 file changed, 4 insertions(+) > > > &g

[FFmpeg-devel] [PATCH] [libdav1d.c]: Add options for spatial layers.

2019-11-14 Thread Thierry Foucu
Disable by default to output all the layers, to match libaomdec wrapper. Add option to select the operating point for the spatial layers. --- libavcodec/libdav1d.c | 8 1 file changed, 8 insertions(+) diff --git a/libavcodec/libdav1d.c b/libavcodec/libdav1d.c index cf4b178f1d..ea4ef641bf

Re: [FFmpeg-devel] [PATCH] [libdav1d.c]: Add option to output all the spatial layers.

2019-11-14 Thread Thierry Foucu
On Thu, Nov 14, 2019 at 11:33 AM Ronald S. Bultje wrote: > Hi, > > On Thu, Nov 14, 2019 at 2:12 PM Andrey Semashev > > wrote: > > > I think there needs to be some consistency across different lavc > > decoders. If we consider that lavc should produce one decoded frame per > > one encoded one, ev

Re: [FFmpeg-devel] [PATCH] [libdav1d.c]: Add options for spatial layers.

2019-11-20 Thread Thierry Foucu
On Thu, Nov 14, 2019 at 9:36 AM Thierry Foucu wrote: > Disable by default to output all the layers, to match libaomdec wrapper. > Add option to select the operating point for the spatial layers. > --- > libavcodec/libdav1d.c | 8 > 1 file changed, 8 insertions(+) &

Re: [FFmpeg-devel] [PATCH] [libdav1d.c]: Add options for spatial layers.

2019-11-25 Thread Thierry Foucu
On Wed, Nov 20, 2019 at 9:55 AM Thierry Foucu wrote: > > > On Thu, Nov 14, 2019 at 9:36 AM Thierry Foucu wrote: > >> Disable by default to output all the layers, to match libaomdec wrapper. >> Add option to select the operating point for the spatial layers. >> ---

[FFmpeg-devel] [PATCH] [libdav1d.c]: Add options for spatial layers.

2019-11-27 Thread Thierry Foucu
Disable by default to output all the layers, to match libaomdec wrapper. Add option to select the operating point for the spatial layers. Update the documentation with the new options. --- doc/decoders.texi | 6 ++ libavcodec/libdav1d.c | 8 2 files changed, 14 insertions(+) diff

Re: [FFmpeg-devel] [PATCH] [libdav1d.c]: Add options for spatial layers.

2019-11-27 Thread Thierry Foucu
On Wed, Nov 27, 2019 at 4:02 PM James Almer wrote: > On 11/27/2019 9:00 PM, Moritz Barsnick wrote: > > On Wed, Nov 27, 2019 at 09:21:28 -0800, Thierry Foucu wrote: > >> +@item oppoint > >> +Select an operating point of a scalable AV1 bitstream (0 - 32) > > [...

[FFmpeg-devel] [PATCH] [libdav1d.c]: Add options for spatial layers.

2019-11-27 Thread Thierry Foucu
Disable by default to output all the layers, to match libaomdec wrapper. Add option to select the operating point for the spatial layers. Update the documentation with the new options. --- doc/decoders.texi | 6 ++ libavcodec/libdav1d.c | 8 2 files changed, 14 insertions(+) diff

Re: [FFmpeg-devel] [PATCH] Workaround to build ffmpeg on MacOs 10.15

2020-01-03 Thread Thierry Foucu
> To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". -- --- Thierry Foucu ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo

Re: [FFmpeg-devel] [PATCH] Workaround to build ffmpeg on MacOs 10.15

2020-01-03 Thread Thierry Foucu
On Fri, Jan 3, 2020 at 1:56 PM Nicolas George wrote: > Thierry Foucu (12020-01-03): > > So, right now, we cannot build ffmpeg with optimization on Mac Os 10.15. > > Can't you pass the option as extra-cflags to configure? It should work, > until a proper fix is proposed

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Allow changing the time stamp precision via option

2021-06-14 Thread Thierry Foucu
000}, > >>> mkv->time_base); > >>> + > >>> if (mkv->is_dash && nb_tracks != 1) > >>> return AVERROR(EINVAL); > >>> @@ -2826,6 +2844,7 @@ static const AVOption options[] = { > >>>

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Allow changing the time stamp precision via option

2021-06-14 Thread Thierry Foucu
On Mon, Jun 14, 2021 at 12:09 PM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote: > Thierry Foucu: > > On Fri, Jun 4, 2021 at 10:18 PM Andreas Rheinhardt < > > andreas.rheinha...@outlook.com> wrote: > > > >> Michael Fabian 'Xaymar'

[FFmpeg-devel] libavcodec/mpeg12dec.c: Check return value of init_get_bits

2021-07-19 Thread Thierry Foucu
--- libavcodec/mpeg12dec.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index b27ed5bd6d..edca202f0e 100644 --- a/libavcodec/mpeg12dec.c +++ b/libavcodec/mpeg12dec.c @@ -1341,8 +1341,11 @@ static int mpeg1_decode_picture(AV

[FFmpeg-devel] [PATCH] libavcodec/mpeg12dec.c: Switch to init_get_bits8 and checks return value

2021-07-21 Thread Thierry Foucu
--- libavcodec/mpeg12dec.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index b27ed5bd6d..a786e48054 100644 --- a/libavcodec/mpeg12dec.c +++ b/libavcodec/mpeg12dec.c @@ -1341,8 +1341,11 @@ static int mpeg1_decode_picture(AV

Re: [FFmpeg-devel] libavcodec/mpeg12dec.c: Check return value of init_get_bits

2021-07-21 Thread Thierry Foucu
On Mon, Jul 19, 2021 at 3:50 PM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote: > Thierry Foucu: > > --- > > libavcodec/mpeg12dec.c | 5 - > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/libavcodec/mpeg12dec.c b/liba

Re: [FFmpeg-devel] [PATCH] libavcodec/mpeg12dec.c: Switch to init_get_bits8 and checks return value

2021-07-22 Thread Thierry Foucu
On Wed, Jul 21, 2021 at 6:19 PM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote: > Thierry Foucu: > > --- > > libavcodec/mpeg12dec.c | 5 - > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/libavcodec/mpeg12dec.c b/liba

[FFmpeg-devel] [PATCH] libavcodec/mpeg12dec.c: Switch to init_get_bits8 and checks return value

2021-07-22 Thread Thierry Foucu
--- libavcodec/mpeg12dec.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index b27ed5bd6d..269619540a 100644 --- a/libavcodec/mpeg12dec.c +++ b/libavcodec/mpeg12dec.c @@ -1341,8 +1341,11 @@ static int mpeg1_decode_picture(AV

Re: [FFmpeg-devel] [PATCH] libavcodec/mpeg12dec.c: Switch to init_get_bits8 and checks return value

2021-07-27 Thread Thierry Foucu
On Fri, Jul 23, 2021 at 2:13 PM Michael Niedermayer wrote: > On Thu, Jul 22, 2021 at 11:08:51AM -0700, Thierry Foucu wrote: > > --- > > libavcodec/mpeg12dec.c | 5 - > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/libavcodec/mpe

[FFmpeg-devel] [PATCH] libavcodec/mpeg12dec.c: Switch to init_get_bits8 and checks return value

2021-07-27 Thread Thierry Foucu
--- libavcodec/mpeg12dec.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index b27ed5bd6d..858dca660c 100644 --- a/libavcodec/mpeg12dec.c +++ b/libavcodec/mpeg12dec.c @@ -1340,9 +1340,11 @@ static int mpeg1_decode_picture(

[FFmpeg-devel] [PATCH] libavformat/mov: Fix memory leak, need to free the chapter tracks array

2017-04-05 Thread Thierry Foucu
--- libavformat/mov.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 90bc2499da..f2296f8917 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -5794,6 +5794,7 @@ static int mov_read_close(AVFormatContext *s) av_freep(&mov->fragment_index

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Thierry Foucu
On Tue, Nov 14, 2017 at 3:23 PM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 11/14/2017 10:11 PM, Sasi Inguva wrote: > > I don't know if the patch can be made more generic to work for all > > demuxers, because this patch requires that PTS of all packets be > available > > in the hea

[FFmpeg-devel] [PATCH] vf_fps: uses the last frame duration to duplicate it if needed

2017-09-06 Thread Thierry Foucu
--- libavfilter/vf_fps.c| 40 +++- tests/ref/fate/filter-fps | 6 ++ tests/ref/fate/filter-fps-r | 4 3 files changed, 45 insertions(+), 5 deletions(-) diff --git a/libavfilter/vf_fps.c b/libavfilter/vf_fps.c index 20ccd797d1..a231da415c 10

Re: [FFmpeg-devel] [PATCH] vf_fps: uses the last frame duration to duplicate it if needed

2017-09-07 Thread Thierry Foucu
Hi Nicolas, On Wed, Sep 6, 2017 at 2:07 PM, Nicolas George wrote: > Le decadi 20 fructidor, an CCXXV, Thierry Foucu a écrit : > > --- > > libavfilter/vf_fps.c| 40 ++ > +- > > tests/ref/fate/filter-fps | 6 ++ > >

Re: [FFmpeg-devel] [PATCH] vf_fps: uses the last frame duration to duplicate it if needed

2017-09-07 Thread Thierry Foucu
On Thu, Sep 7, 2017 at 11:37 AM, Nicolas George wrote: > [ Ccing you, but no need to Cc me. ] > > Le primidi 21 fructidor, an CCXXV, Thierry Foucu a écrit : > > Sorry I did not know and I did not see any documentation in the header > > explaining which field of the AVFrame

[FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-08 Thread Thierry Foucu
--- libavfilter/vf_fps.c| 42 +- tests/ref/fate/filter-fps | 6 ++ tests/ref/fate/filter-fps-r | 4 3 files changed, 47 insertions(+), 5 deletions(-) diff --git a/libavfilter/vf_fps.c b/libavfilter/vf_fps.c index 20ccd797d1..e450723173

[FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-11 Thread Thierry Foucu
Fix ticket #2674 Tested with examples from ticket 2674. --- libavfilter/vf_fps.c| 40 +++- tests/ref/fate/filter-fps | 6 ++ tests/ref/fate/filter-fps-r | 4 tests/ref/fate/m4v-cfr | 1 - 4 files changed, 45 insertions(+), 6 deletions(

[FFmpeg-devel] (no subject)

2017-09-11 Thread Thierry Foucu
Please find attached new patch. I tried it against ticket 2674 and it seems to work Here are the output: For the up-sample, getting 48 frames out, instead of 47 -- ./ffmpeg -loglevel verbose -i 24fps.avi -vf fps=48 -y 48fps.avi ffmpeg version N-87234-g9a32769f5e Copyr

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-11 Thread Thierry Foucu
put file #0 (48fps.avi): Output stream #0:0 (video): 48 frames encoded; 48 packets muxed (62176 bytes); Total: 48 packets (62176 bytes) muxed [Parsed_fps_0 @ 0x34ea4a0] 24 frames in, 48 frames out; 0 frames dropped, 23 frames duplicated. On Fri, Sep 8, 2017 at 4:33 PM, Thomas Mundt wrote:

[FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-11 Thread Thierry Foucu
Fix ticket #2674 Tested with examples from ticket 2674. --- Update the Patch with the correct number of duplicate showing. For exmaple, in case we up-sample one second video at 24fps by 2, we should be getting 24 duplicate frames. ffmpeg -loglevel verbose -i 24fps.avi -vf fps=48 -f null - ffmpeg

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-12 Thread Thierry Foucu
On Tue, Sep 12, 2017 at 1:35 AM, Thomas Mundt wrote: > Hi Thierry, > > 2017-09-12 3:25 GMT+02:00 Thierry Foucu : > >> Fix ticket #2674 >> Tested with examples from ticket 2674. >> --- >> >> Update the Patch with the correct number of duplicate showing

[FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-12 Thread Thierry Foucu
Fix ticket #2674 Tested with examples from ticket 2674. --- Sorry Michael, I forgot to configure using --enable-gpl. Please find new patch. libavfilter/vf_fps.c | 44 +++- tests/ref/fate/filter-fps| 6 ++ tests/ref/fate/filter-fps-r

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-14 Thread Thierry Foucu
On Tue, Sep 12, 2017 at 6:45 PM, Thierry Foucu wrote: > Fix ticket #2674 > Tested with examples from ticket 2674. > --- > Sorry Michael, I forgot to configure using --enable-gpl. > Please find new patch. > > > ping? > libavfilte

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-15 Thread Thierry Foucu
p 15, 2017 at 02:44:59AM +0200, Michael Niedermayer wrote: > > > >> On Tue, Sep 12, 2017 at 06:45:57PM -0700, Thierry Foucu wrote: > > > >>> Fix ticket #2674 > > > >>> Tested with examples from ticket 2674. > > > >>> ---

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-15 Thread Thierry Foucu
On Fri, Sep 15, 2017 at 8:24 AM, Carl Eugen Hoyos wrote: > 2017-09-15 17:13 GMT+02:00 Thierry Foucu : > > >> Too fast to reply again, my bad. It's indeed a GCC bug. > > > > Sorry about it. I'm glad you found the problem. > > I suspect the comment above

[FFmpeg-devel] [PATCH 2/2] vf_fps: Fix memory leak introduced by eea64ef4

2017-09-15 Thread Thierry Foucu
--- libavfilter/vf_fps.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/vf_fps.c b/libavfilter/vf_fps.c index 09fc66a73c..1e5d07e31c 100644 --- a/libavfilter/vf_fps.c +++ b/libavfilter/vf_fps.c @@ -171,6 +171,7 @@ static int request_frame(AVFilterLink *outlink)

[FFmpeg-devel] [PATCH] libavcodec/h263dec.c: Duplicate the last decoded frame when xvid marks the packet as skipped.

2017-10-19 Thread Thierry Foucu
Instead of returning nothing when we detect the xvid skipped frame, we could return the last decoded frame, if we do have one. --- libavcodec/h263dec.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c index bcb2b08bb0..e1667bcb

Re: [FFmpeg-devel] [PATCH] libavcodec/h263dec.c: Duplicate the last decoded frame when xvid marks the packet as skipped.

2017-10-19 Thread Thierry Foucu
On Thu, Oct 19, 2017 at 3:43 PM, Michael Niedermayer wrote: > On Thu, Oct 19, 2017 at 09:51:05AM -0700, Thierry Foucu wrote: > > Instead of returning nothing when we detect the xvid skipped frame, we > > could return the last decoded frame, if we do have one. > > FRAME_SKIP

Re: [FFmpeg-devel] [PATCH] libavcodec/h263dec.c: Duplicate the last decoded frame when xvid marks the packet as skipped.

2017-10-24 Thread Thierry Foucu
Michael, what do you think of this patch? I tried to reflect what the xvid decoder does for those N_VOP frame and in case it is not packed. On Tue, Oct 24, 2017 at 6:42 PM, Thierry Foucu wrote: > Changed the return value when no VOD were encoded in Mpeg4 bistream. > And if we d

[FFmpeg-devel] [PATCH] libavcodec/h263dec.c: Duplicate the last decoded frame when xvid marks the packet as skipped.

2017-10-24 Thread Thierry Foucu
Changed the return value when no VOD were encoded in Mpeg4 bistream. And if we do have already a decoded frames and we are not in xvid_packed mode, output the existing decoded frame instead of nothing. --- libavcodec/h263dec.c | 9 - libavcodec/mpeg4videodec.c | 2 +- libavcodec/mpeg

[FFmpeg-devel] (no subject)

2017-10-24 Thread Thierry Foucu
Michael, the following patch seems to work. I restricted the change to N_VOP (from xvid) frame. I tried to match it with what the xvidcore decoder does for such packets. What do you think? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://

Re: [FFmpeg-devel] [PATCH] libavcodec/h263dec.c: Duplicate the last decoded frame when xvid marks the packet as skipped.

2017-10-25 Thread Thierry Foucu
Derek, On Wed, Oct 25, 2017 at 9:43 AM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 10/25/2017 2:42 AM, Thierry Foucu wrote: > > Changed the return value when no VOD were encoded in Mpeg4 bistream. > > And if we do have already a decoded frames and we a

Re: [FFmpeg-devel] [PATCH] libavcodec/h263dec.c: Duplicate the last decoded frame when xvid marks the packet as skipped.

2017-10-25 Thread Thierry Foucu
On Wed, Oct 25, 2017 at 4:59 AM, Michael Niedermayer wrote: > On Tue, Oct 24, 2017 at 06:42:54PM -0700, Thierry Foucu wrote: > > Changed the return value when no VOD were encoded in Mpeg4 bistream. > > And if we do have already a decoded frames and we are not in xvid_packed >

Re: [FFmpeg-devel] [PATCH] libavcodec/h263dec.c: Duplicate the last decoded frame when xvid marks the packet as skipped.

2017-10-25 Thread Thierry Foucu
On Wed, Oct 25, 2017 at 2:43 PM, Carl Eugen Hoyos wrote: > 2017-10-25 20:38 GMT+02:00 Thierry Foucu : > > > Without the patch, the decoder will return only 29 frames > > Isn't that the correct and expected output? > Not sure what you mean by correct. There are only 29

Re: [FFmpeg-devel] [PATCH] libavcodec/h263dec.c: Duplicate the last decoded frame when xvid marks the packet as skipped.

2017-10-25 Thread Thierry Foucu
On Wed, Oct 25, 2017 at 4:36 PM, Michael Niedermayer wrote: > On Wed, Oct 25, 2017 at 04:21:28PM -0700, Thierry Foucu wrote: > > On Wed, Oct 25, 2017 at 2:43 PM, Carl Eugen Hoyos > > wrote: > > > > > 2017-10-25 20:38 GMT+02:00 Thierry Foucu : > > > >

[FFmpeg-devel] Problem with vstats file and frame number

2016-11-11 Thread Thierry Foucu
Hi, Before sending you some patches to fix it, I would like to know how you want this issue to be fixed. Let me explain it: When we enable vstats file to be generated, each line of the file contains the frame number of the encoded video frames. The frame number is the one from AVStream.nb_frames

[FFmpeg-devel] [PATCH 3/3] Fix -Werror=parentheses error

2016-11-15 Thread Thierry Foucu
--- libavcodec/internal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/internal.h b/libavcodec/internal.h index 35b9630..c92dba4 100644 --- a/libavcodec/internal.h +++ b/libavcodec/internal.h @@ -262,7 +262,7 @@ static av_always_inline int64_t ff_samples_to_time_

Re: [FFmpeg-devel] [PATCH v3 1/2] lavc/ccaption_dec: do not ignore repeated character commands

2016-04-18 Thread Thierry Foucu
Hi all On Sun, Feb 14, 2016 at 6:11 PM, Aman Gupta wrote: > From: Aman Gupta > > control codes in a cc stream can be repeated, and must be ignored. > however, repeated characters must not be ignored. the code attempted to > wipe prev_cmd in handle_char to allow repeated characters to be > proce

Re: [FFmpeg-devel] [PATCH v3 1/2] lavc/ccaption_dec: do not ignore repeated character commands

2016-04-25 Thread Thierry Foucu
ter%20set%20test/README.TXT), > but three times. > > Aman > > On Mon, Apr 18, 2016 at 1:01 PM, Aman Gupta wrote: > >> Please send me the sample and I will try to fix the issue. >> >> Aman >> >> On Mon, Apr 18, 2016 at 1:22 PM Thierry Foucu wrote: >>

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Thierry Foucu
Hi Team On Mon, Feb 29, 2016 at 8:51 AM, Derek Buitenhuis wrote: > ffmpeg | branch: master | Derek Buitenhuis | > Mon Feb 29 16:50:39 2016 +| [9c75148e6ebc88a0501e3d0242defb6dbdc3c23d] > | committer: Derek Buitenhuis > > Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d' > > This commi

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Thierry Foucu
On Wed, May 4, 2016 at 1:58 PM, Derek Buitenhuis wrote: > On 5/4/2016 9:52 PM, wm4 wrote: > > This was never allowed and was never public API. Use custom I/O instead > > (creating an avio context with your own read/write callbacks). > > Can i ask why it cannot be part of the public API? It was re

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-04 Thread Thierry Foucu
On Wed, May 4, 2016 at 3:46 PM, Thierry Foucu wrote: > > > On Wed, May 4, 2016 at 1:58 PM, Derek Buitenhuis < > derek.buitenh...@gmail.com> wrote: > >> On 5/4/2016 9:52 PM, wm4 wrote: >> > This was never allowed and was never public API. Use custom I/O instead

Re: [FFmpeg-devel] [PATCH] avfilter/vf_idet: Fixes issue with idet not flushing last frame.

2014-10-22 Thread Thierry Foucu
On Wed, Oct 22, 2014 at 7:43 AM, Pascal Massimino < pascal.massim...@gmail.com> wrote: > Hi, > > On Tue, Oct 21, 2014 at 11:40 PM, Neil Birkbeck > wrote: > > > Uses a similar approach as vf_yadif to flush the last frame in idet. > > > > Quick test with 50 frames from vsynth1: > > ./ffmpeg.old -i

Re: [FFmpeg-devel] [RFC PATCH 1/3] aacdec: always skip the first 2048 samples if there's no side data

2023-09-12 Thread Thierry Foucu
fe735b5824c7d10ba42932a17d786db50e3b2d4), and it's only for > faac. > It's less of a guess, as most encoders to use the FIL extension to signal > themselves. > _______ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > htt

Re: [FFmpeg-devel] [PATCH] aacdec: padding skip improvements

2023-10-06 Thread Thierry Foucu
ntation is padding 1600 samples. > > - Derek > ___ > 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...@ffmpe

[FFmpeg-devel] [PATCH] libavformat/rmdec.c: Fix Use-of-uninitialized-value in ff_codec_get_id

2020-09-14 Thread Thierry Foucu
In case the pb does not contain 4 bytes, the buf[256] will not be initialize before we pass it to ff_codec_get_id --- libavformat/rmdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c index a36e693ab2..220aa8aee2 100644 --- a/libavfor

Re: [FFmpeg-devel] [PATCH] libavformat/rmdec.c: Fix Use-of-uninitialized-value in ff_codec_get_id

2020-09-16 Thread Thierry Foucu
On Mon, Sep 14, 2020 at 10:49 AM Thierry Foucu wrote: > In case the pb does not contain 4 bytes, the buf[256] will not be > initialize before we pass it to ff_codec_get_id > --- > libavformat/rmdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/

[FFmpeg-devel] [PATCH] libavformat/avidec: Extract more metadata from the header

2020-11-06 Thread Thierry Foucu
--- libavformat/avidec.c | 17 + 1 file changed, 17 insertions(+) diff --git a/libavformat/avidec.c b/libavformat/avidec.c index 578cf68ce1..7e527e15ee 100644 --- a/libavformat/avidec.c +++ b/libavformat/avidec.c @@ -110,6 +110,15 @@ static const char avi_headers[][8] = { stati

[FFmpeg-devel] [PATCH] libavformat/matroskaenc: move a warning log to debug log

2020-11-06 Thread Thierry Foucu
At high frame rate, the message "Starting new cluster due to timestamp" happens too often. Moving to debug to reduce the warning log --- libavformat/matroskaenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 233c472

[FFmpeg-devel] [PATCH] libavformat/mov.c: export vendor id as metadata

2020-11-17 Thread Thierry Foucu
--- libavformat/mov.c | 24 ++-- 1 file changed, 22 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 2b90e31170..1f9163d658 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2095,6 +2095,8 @@ static void mov_parse_stsd_video(MOVCo

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: export vendor id as metadata

2020-11-17 Thread Thierry Foucu
On Tue, Nov 17, 2020 at 9:54 PM Gyan Doshi wrote: > > > On 18-11-2020 02:41 am, Thierry Foucu wrote: > > --- > > libavformat/mov.c | 24 ++-- > > 1 file changed, 22 insertions(+), 2 deletions(-) > > > > diff --git a/libavfor

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: export vendor id as metadata

2020-11-18 Thread Thierry Foucu
On Wed, Nov 18, 2020 at 12:30 AM Gyan Doshi wrote: > > > On 18-11-2020 12:16 pm, Thierry Foucu wrote: > > On Tue, Nov 17, 2020 at 9:54 PM Gyan Doshi wrote: > > > >> > >> On 18-11-2020 02:41 am, Thierry Foucu wrote: > >>> --- > >>>

[FFmpeg-devel] [PATCH] libavformat/mov.c: export vendor id as metadata

2020-11-18 Thread Thierry Foucu
--- libavformat/mov.c | 24 +-- ...hapqa-extract-nosnappy-to-hapalphaonly-mov | 1 + .../fate/hapqa-extract-nosnappy-to-hapq-mov | 1 + tests/ref/fate/mov-zombie | 2 +- tests/ref/fate/rgb24-mkv | 4 ++-- 5

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: export vendor id as metadata

2020-11-18 Thread Thierry Foucu
On Wed, Nov 18, 2020 at 10:56 AM Andreas Rheinhardt < andreas.rheinha...@gmail.com> wrote: > Thierry Foucu: > > --- > > libavformat/mov.c | 24 +-- > > ...hapqa-extract-nosnappy-to-hapalphaonly-mov | 1 + > > .../fa

[FFmpeg-devel] [PATCH] libavformat/mov.c: export vendor id as metadata

2020-11-18 Thread Thierry Foucu
--- libavformat/mov.c | 8 ++-- tests/ref/fate/hapqa-extract-nosnappy-to-hapalphaonly-mov | 1 + tests/ref/fate/hapqa-extract-nosnappy-to-hapq-mov | 1 + tests/ref/fate/mov-zombie | 2 +- tests/ref/fate/rgb24-mkv

[FFmpeg-devel] [PATCH] motion_est: fixed signed_integer_overflow when computing score

2022-11-09 Thread Thierry Foucu
--- libavcodec/motion_est.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/motion_est.c b/libavcodec/motion_est.c index d17ffe42b4..95978d95dd 100644 --- a/libavcodec/motion_est.c +++ b/libavcodec/motion_est.c @@ -1571,7 +1571,7 @@ void ff_estimate_b_frame_motion(Mp

[FFmpeg-devel] [PATCH] libavformat/avidec.c: remove duplicated conversion

2020-11-23 Thread Thierry Foucu
ff_riff_info_conv deals with riff metadata already, so, not need to add those metadata in avi_metadata_conv --- libavformat/avidec.c | 8 1 file changed, 8 deletions(-) diff --git a/libavformat/avidec.c b/libavformat/avidec.c index 7e527e15ee..75a76b995d 100644 --- a/libavformat/avidec.c

Re: [FFmpeg-devel] [PATCH] libavformat/avidec: Extract more metadata from the header

2020-11-23 Thread Thierry Foucu
On Wed, Nov 11, 2020 at 7:39 AM Anton Khirnov wrote: > Quoting Thierry Foucu (2020-11-07 00:10:17) > > --- > > libavformat/avidec.c | 17 + > > 1 file changed, 17 insertions(+) > > > > diff --git a/libavformat/avidec.c b/libavformat/avidec.c

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: export vendor id as metadata

2020-11-27 Thread Thierry Foucu
ping? On Wed, Nov 18, 2020 at 12:09 PM Thierry Foucu wrote: > --- > libavformat/mov.c | 8 ++-- > tests/ref/fate/hapqa-extract-nosnappy-to-hapalphaonly-mov | 1 + > tests/ref/fate/hapqa-extract-nosnappy-to-hapq-mov | 1 + > tes

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: export vendor id as metadata

2020-12-03 Thread Thierry Foucu
On Wed, Nov 18, 2020 at 12:09 PM Thierry Foucu wrote: > --- > libavformat/mov.c | 8 ++-- > tests/ref/fate/hapqa-extract-nosnappy-to-hapalphaonly-mov | 1 + > tests/ref/fate/hapqa-extract-nosnappy-to-hapq-mov | 1 + > tests/ref/

  1   2   >