[libav-devel] [PATCH] GIF demuxer

2016-03-19 Thread Vittorio Giovara
From: Vitaliy E Sugrobov This demuxer is capable of extracting multiple frames from gif file. In conjunction with gif decoder it implements support for reading animated gifs. Signed-off-by: Vitaliy E Sugrobov Signed-off-by: Vittorio Giovara

[libav-devel] [PATCH] nuv: Use the correct context for av_image_check_size

2016-03-19 Thread Vittorio Giovara
--- libavformat/nuv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/nuv.c b/libavformat/nuv.c index 7fc339a..492f68f 100644 --- a/libavformat/nuv.c +++ b/libavformat/nuv.c @@ -198,7 +198,7 @@ static int nuv_header(AVFormatContext *s) return

Re: [libav-devel] [FFmpeg-devel] [PATCH] aacpsy: avoid norm_fac becoming NaN

2016-03-19 Thread Vittorio Giovara
On Sat, Mar 19, 2016 at 8:05 PM, Luca Barbato wrote: > On 20/03/16 00:48, Luca Barbato wrote: >> On 19/03/16 21:57, Vittorio Giovara wrote: >>> On Tue, Apr 21, 2015 at 12:50 PM, Andreas Cadhalpun >>> wrote: On 21.04.2015 02:20, Claudio

[libav-devel] [PATCH 8/8] unix: Use rw_timeout for setting the connect timeout

2016-03-19 Thread Martin Storsjö
--- libavformat/unix.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/unix.c b/libavformat/unix.c index 6bb677d..647e7e8 100644 --- a/libavformat/unix.c +++ b/libavformat/unix.c @@ -73,6 +73,9 @@ static int unix_open(URLContext *h, const char *filename, int flags) if

Re: [libav-devel] [FFmpeg-devel] [PATCH] aacpsy: avoid norm_fac becoming NaN

2016-03-19 Thread Luca Barbato
On 20/03/16 00:48, Luca Barbato wrote: > On 19/03/16 21:57, Vittorio Giovara wrote: >> On Tue, Apr 21, 2015 at 12:50 PM, Andreas Cadhalpun >> wrote: >>> On 21.04.2015 02:20, Claudio Freire wrote: On Mon, Apr 20, 2015 at 9:13 PM, Michael Niedermayer

Re: [libav-devel] [PATCH] avio: Add avio_check2

2016-03-19 Thread Luca Barbato
On 19/03/16 21:38, Vittorio Giovara wrote: > On Thu, Feb 4, 2016 at 6:56 AM, Vittorio Giovara > wrote: >> On Sun, Nov 8, 2015 at 5:12 PM, Luca Barbato wrote: >>> On 08/11/15 16:43, Anton Khirnov wrote: Ok, now that sounds like a real use case.

Re: [libav-devel] [PATCH 5/6] avio: Add avio_read wrapper to simplify error checking

2016-03-19 Thread Luca Barbato
On 19/03/16 21:55, Vittorio Giovara wrote: > On Sun, Jun 7, 2015 at 1:07 PM, Luca Barbato wrote: >> On 07/06/15 16:30, Andreas Cadhalpun wrote: >>> It seems Luca accidentally pushed an old version of this patch. >>> The pushed version still has the ffio_read_size name and

Re: [libav-devel] [FFmpeg-devel] [PATCH] aacpsy: avoid norm_fac becoming NaN

2016-03-19 Thread Luca Barbato
On 19/03/16 21:57, Vittorio Giovara wrote: > On Tue, Apr 21, 2015 at 12:50 PM, Andreas Cadhalpun > wrote: >> On 21.04.2015 02:20, Claudio Freire wrote: >>> On Mon, Apr 20, 2015 at 9:13 PM, Michael Niedermayer >>> wrote: On Mon, Apr 20,

Re: [libav-devel] [PATCH 1/2] avlanguage: add av_islang

2016-03-19 Thread Luca Barbato
On 24/07/14 09:55, Jan Gerber wrote: > --- > libavformat/avlanguage.c | 14 ++ > libavformat/avlanguage.h | 5 + > 2 files changed, 19 insertions(+) > I'd keep it private its usage is _quite_ specific. lu ___ libav-devel mailing

Re: [libav-devel] [PATCH 2/2] GIF demuxer

2016-03-19 Thread Luca Barbato
On 19/03/16 22:48, Vittorio Giovara wrote: > From: Vitaliy E Sugrobov > > This demuxer is capable of extracting multiple frames from gif file. > In conjunction with gif decoder it implements support for reading > animated gifs. > > Signed-off-by: Vitaliy E Sugrobov

Re: [libav-devel] [PATCH 01/25] fate: Add test for WMV2 with jframes

2016-03-19 Thread Luca Barbato
On 19/03/16 22:42, Vittorio Giovara wrote: > --- > tests/fate/microsoft.mak| 3 + > tests/ref/fate/wmv8-intrax8 | 475 > > 2 files changed, 478 insertions(+) > create mode 100644 tests/ref/fate/wmv8-intrax8 > > This test breaks on an oracle

Re: [libav-devel] [PATCH 1/2] lavf: Raw G.729 demuxer

2016-03-19 Thread Luca Barbato
On 19/03/16 22:48, Vittorio Giovara wrote: > From: Vladimir Voroshilov > > Signed-off-by: Vittorio Giovara > --- > Changelog| 1 + > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/g729dec.c

Re: [libav-devel] [PATCH] indeo4: Consistently initialize variables

2016-03-19 Thread Luca Barbato
On 19/03/16 23:07, Vittorio Giovara wrote: > Avoid using multiple variables for the same purpose. > --- > libavcodec/indeo4.c | 9 +++-- > libavcodec/ivi.h| 3 +-- > 2 files changed, 4 insertions(+), 8 deletions(-) > Ok. ___ libav-devel

Re: [libav-devel] [PATCH] avconv: switch to the new BSF API

2016-03-19 Thread Luca Barbato
On 04/03/16 19:01, Anton Khirnov wrote: > --- > Changed to not send multiple packets at once to the filter > --- > avconv.c | 147 > +++ > avconv.h | 6 ++- > avconv_opt.c | 22 + > cmdutils.c | 5 +- > 4 files

Re: [libav-devel] [PATCH 13/14] bitstream_filter: add a compatibility wrapper for the old bsf API

2016-03-19 Thread Luca Barbato
On 04/03/16 09:15, Anton Khirnov wrote: > --- > libavcodec/bitstream_filter.c | 147 > -- > 1 file changed, 112 insertions(+), 35 deletions(-) > Possibly OK. ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] aac_adtstoasc_bsf: convert to the new API

2016-03-19 Thread Luca Barbato
On 19/03/16 17:20, Anton Khirnov wrote: > --- > libavcodec/aac_adtstoasc_bsf.c | 104 > +++-- > libavcodec/allcodecs.c | 1 - > libavcodec/bitstream_filters.c | 5 ++ > 3 files changed, 74 insertions(+), 36 deletions(-) > OK.

[libav-devel] [PATCH] indeo4: Consistently initialize variables

2016-03-19 Thread Vittorio Giovara
Avoid using multiple variables for the same purpose. --- libavcodec/indeo4.c | 9 +++-- libavcodec/ivi.h| 3 +-- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/libavcodec/indeo4.c b/libavcodec/indeo4.c index 4ec09dc..217311f 100644 --- a/libavcodec/indeo4.c +++

[libav-devel] [PATCH 08/25] intrax8: Check and propagate errors from ff_intrax8_common_init

2016-03-19 Thread Vittorio Giovara
This allows dropping an afterwards redundant assert. --- libavcodec/intrax8.c | 12 +--- libavcodec/intrax8.h | 3 ++- 2 files changed, 11 insertions(+), 4 deletions(-) Unchanged. diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index bea1b76..0c720b0 100644 ---

[libav-devel] [PATCH 07/25] intrax8: Let x8_vlc_init report errors

2016-03-19 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) Unchanged. diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 3ab2faa..bea1b76 100644 --- a/libavcodec/intrax8.c +++ b/libavcodec/intrax8.c @@ -45,7 +45,7 @@ static VLC j_ac_vlc[2][2][8]; //

[libav-devel] [PATCH 20/25] intrax8: Pass macroblock coordinates to ff_intrax8_decode_picture

2016-03-19 Thread Vittorio Giovara
These values need to be updated with the last macroblock position, so keep them as pointers. --- libavcodec/intrax8.c | 63 +- libavcodec/intrax8.h | 4 +++- libavcodec/vc1_block.c | 3 ++- libavcodec/wmv2dec.c | 3 ++- 4 files changed, 39

[libav-devel] [PATCH 15/25] intrax8: Use a constant buffer instead of a ScratchpadContext

2016-03-19 Thread Vittorio Giovara
The size of the block is fixed (8x8 plus padding). --- libavcodec/intrax8.c | 5 ++--- libavcodec/intrax8.h | 1 + 2 files changed, 3 insertions(+), 3 deletions(-) Modified according to instructions. diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 9da6b41f4..fae9bc8 100644 ---

[libav-devel] [PATCH 14/25] intrax8: Pass the output frame to the decoding function

2016-03-19 Thread Vittorio Giovara
Helps in decoupling this code from mpegvideo. --- libavcodec/intrax8.c | 21 +++-- libavcodec/intrax8.h | 7 +-- libavcodec/vc1_block.c | 5 +++-- libavcodec/wmv2dec.c | 3 ++- 4 files changed, 21 insertions(+), 15 deletions(-) diff --git a/libavcodec/intrax8.c

[libav-devel] [PATCH 17/25] intrax8: Reference the currect AVCodecContext

2016-03-19 Thread Vittorio Giovara
It will be needed in later commits. --- libavcodec/intrax8.c | 4 +++- libavcodec/intrax8.h | 5 - libavcodec/vc1dec.c | 2 +- libavcodec/wmv2dec.c | 2 +- 4 files changed, 9 insertions(+), 4 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 336551f..3365995 100644

[libav-devel] [PATCH 23/25] intrax8: Keep a reference to the decoder blocks

2016-03-19 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 31 --- libavcodec/intrax8.h | 6 ++ libavcodec/vc1dec.c | 1 + libavcodec/wmv2dec.c | 1 + 4 files changed, 24 insertions(+), 15 deletions(-) As with a previous patch, I prefer to reference the existing fields, to make sure that it

[libav-devel] [PATCH 19/25] intrax8: Pass macroblock size to ff_intrax8_common_init

2016-03-19 Thread Vittorio Giovara
Helps in decoupling this code from mpegvideo. --- libavcodec/intrax8.c | 20 +++- libavcodec/intrax8.h | 7 +++ libavcodec/vc1dec.c | 4 +++- libavcodec/wmv2dec.c | 3 ++- 4 files changed, 23 insertions(+), 11 deletions(-) diff --git a/libavcodec/intrax8.c

[libav-devel] [PATCH 25/25] intrax8: Remove mpegvideo dependency

2016-03-19 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 7 +-- libavcodec/intrax8.h | 10 +- libavcodec/vc1dec.c | 3 +-- libavcodec/wmv2dec.c | 2 +- 4 files changed, 4 insertions(+), 18 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index f538d5d..d9fb34d 100644 ---

[libav-devel] [PATCH 24/25] intrax8: Drop MB emulation code

2016-03-19 Thread Vittorio Giovara
This is already performed in init_context_frame(). --- libavcodec/intrax8.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index f67c5e0..f538d5d 100644 --- a/libavcodec/intrax8.c +++ b/libavcodec/intrax8.c @@ -828,8 +828,6 @@ int

[libav-devel] [PATCH 22/25] intrax8: Use the generic horizband function

2016-03-19 Thread Vittorio Giovara
This is assuming that intrax8 has no support for interlacing (hence PICT_FRAME is set and last_frame is NULL). --- libavcodec/intrax8.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) Modified as discussed with Luca. Vittorio diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c

[libav-devel] [PATCH 21/25] intrax8: Carry over lowdelay value in ff_intrax8_decode_picture

2016-03-19 Thread Vittorio Giovara
Helps in decoupling this code from mpegvideo. --- libavcodec/intrax8.c | 3 ++- libavcodec/intrax8.h | 3 ++- libavcodec/vc1_block.c | 2 +- libavcodec/wmv2dec.c | 2 +- 4 files changed, 6 insertions(+), 4 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index

[libav-devel] [PATCH 18/25] intrax8: Add a local BlockDSPContext and initialize it

2016-03-19 Thread Vittorio Giovara
Helps in decoupling this code from mpegvideo. --- configure| 2 +- libavcodec/intrax8.c | 3 ++- libavcodec/intrax8.h | 2 ++ 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 493bbc7..1c2ac71 100755 --- a/configure +++ b/configure @@ -1860,7

[libav-devel] [PATCH 1/2] lavf: Raw G.729 demuxer

2016-03-19 Thread Vittorio Giovara
From: Vladimir Voroshilov Signed-off-by: Vittorio Giovara --- Changelog| 1 + libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/g729dec.c| 105 +++ 4

[libav-devel] [PATCH 2/2] GIF demuxer

2016-03-19 Thread Vittorio Giovara
From: Vitaliy E Sugrobov This demuxer is capable of extracting multiple frames from gif file. In conjunction with gif decoder it implements support for reading animated gifs. Signed-off-by: Vitaliy E Sugrobov Signed-off-by: Vittorio Giovara

[libav-devel] [PATCH 06/25] intrax8: Move documentation from implementation to header files

2016-03-19 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 20 libavcodec/intrax8.h | 22 ++ 2 files changed, 22 insertions(+), 20 deletions(-) Unchanged. diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 851c434..3ab2faa 100644 --- a/libavcodec/intrax8.c +++

[libav-devel] [PATCH 02/25] intrax8: Move error resilience out of intrax8

2016-03-19 Thread Vittorio Giovara
The intrax8 decoding process does not imply any kind of error correction, and the only call present is more related to how mpegvideo works rather than anything else. Therefore have the parent decoders carried out er when actually needed. Drop two variables that were initialized but never used.

[libav-devel] [PATCH 09/25] vc1dec: wmv2dec: Validate ff_intrax8_common_init return value

2016-03-19 Thread Vittorio Giovara
--- libavcodec/vc1dec.c | 8 +--- libavcodec/wmv2dec.c | 4 +--- 2 files changed, 6 insertions(+), 6 deletions(-) Unchanged. diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index 51f5c42..76f3591 100644 --- a/libavcodec/vc1dec.c +++ b/libavcodec/vc1dec.c @@ -314,7 +314,7 @@ static

[libav-devel] [PATCH 03/25] intrax8: Wrap multiline macros in do{}while(0) clauses

2016-03-19 Thread Vittorio Giovara
These macros are treated like functions, the wrapping simplifies error checking and avoids deep nested if in the following commit. --- libavcodec/intrax8.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) Unchanged. Vittorio diff --git a/libavcodec/intrax8.c

[libav-devel] [PATCH 12/25] intrax8: Keep a reference to the context idctdsp

2016-03-19 Thread Vittorio Giovara
Use it instead of the embedded mpegvideo one. Update init function signature to load it directly from the callers. --- configure| 1 + libavcodec/intrax8.c | 14 -- libavcodec/intrax8.h | 6 +- libavcodec/vc1dec.c | 2 +- libavcodec/wmv2dec.c | 2 +- 5 files

[libav-devel] [PATCH 11/25] intrax8: Make x8_init_block_index not use mpegvideo fields

2016-03-19 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 23 +++ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 3993c57..bb2ee8f 100644 --- a/libavcodec/intrax8.c +++ b/libavcodec/intrax8.c @@ -725,22 +725,21 @@ block_placed: } // FIXME

[libav-devel] [PATCH 04/25] intrax8: Split one comment on multiple lines where it applies

2016-03-19 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) Taken out of the k patch. diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 4887643..b951684 100644 --- a/libavcodec/intrax8.c +++ b/libavcodec/intrax8.c @@ -152,11 +152,11 @@ static inline

[libav-devel] [PATCH 10/25] intrax8: Use local destination buffers

2016-03-19 Thread Vittorio Giovara
These buffers are just a way to store frame pointers and be able to modify them without touching the original ones. The two dependent decoders (WMV2 and VC1) do not need special care for these fields: the former does not seem to use the dest buffers, while the latter reinits them every time to

[libav-devel] [PATCH 01/25] fate: Add test for WMV2 with jframes

2016-03-19 Thread Vittorio Giovara
--- tests/fate/microsoft.mak| 3 + tests/ref/fate/wmv8-intrax8 | 475 2 files changed, 478 insertions(+) create mode 100644 tests/ref/fate/wmv8-intrax8 This test breaks on an oracle box with gcc-4.4. I carried out a minimum investigation and

[libav-devel] [PATCH 16/25] intrax8: Keep a reference to the GetBitContext reader

2016-03-19 Thread Vittorio Giovara
Helps in decoupling this code from mpegvideo. --- libavcodec/intrax8.c | 35 --- libavcodec/intrax8.h | 4 +++- libavcodec/vc1_block.c | 2 +- libavcodec/wmv2dec.c | 2 +- 4 files changed, 21 insertions(+), 22 deletions(-) diff --git

[libav-devel] [PATCH 6/8] RFC: avio: Copy URLContext generic options into child URLContexts

2016-03-19 Thread Martin Storsjö
Since all URLContexts have the same AVOptions, such AVOptions will be applied on the outermost context only and removed from the dict, while they probably make sense on all contexts. This makes sure that rw_timeout gets propagated to the innermost URLContext (to make sure it gets passed to the

[libav-devel] [PATCH] indeo4: Rework stream analysis report

2016-03-19 Thread Vittorio Giovara
* Change log level from error to debug * Print report after the first decoded frame, not at the end of decoding * Drop macro guard and use a context variable instead Signed-off-by: Vittorio Giovara --- libavcodec/indeo4.c | 11 +-- libavcodec/ivi.c| 33

Re: [libav-devel] [PATCH 2/2] vda_h264: Use av_buffer to manage buffers

2016-03-19 Thread Vittorio Giovara
On Tue, Mar 4, 2014 at 7:30 PM, Luca Barbato wrote: > From: Sebastien Zwickert > > Fix a buffer leak when seeking occurs. > > Signed-off-by: Luca Barbato > --- > libavcodec/vda_h264.c | 25 - >

Re: [libav-devel] [PATCH] matroskaenc: Don't set language to empty string, use "und"

2016-03-19 Thread Vittorio Giovara
On Wed, Jul 23, 2014 at 7:03 AM, Anton Khirnov wrote: > > On Tue, 22 Jul 2014 09:38:42 +0200, j...@v2v.cc wrote: >> On 07/21/2014 09:35 PM, Anton Khirnov wrote: >> > >> > On Sun, 20 Jul 2014 15:28:36 +0200, j...@v2v.cc wrote: >> >> On 07/20/2014 03:13 PM, Anton Khirnov wrote:

Re: [libav-devel] [PATCH 2/2] mpegts: use av_islang before setting language

2016-03-19 Thread Vittorio Giovara
On Thu, Jul 24, 2014 at 3:55 AM, Jan Gerber wrote: > --- > libavformat/mpegts.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c > index 5d8b08c..fa784ac 100644 > --- a/libavformat/mpegts.c > +++

Re: [libav-devel] [PATCH 1/2] avlanguage: add av_islang

2016-03-19 Thread Vittorio Giovara
On Thu, Jul 24, 2014 at 3:55 AM, Jan Gerber wrote: > --- > libavformat/avlanguage.c | 14 ++ > libavformat/avlanguage.h | 5 + > 2 files changed, 19 insertions(+) > > diff --git a/libavformat/avlanguage.c b/libavformat/avlanguage.c > index e606ef2..a4b916f 100644 >

Re: [libav-devel] [RFC] no longer marking native aac encoder as experimental

2016-03-19 Thread Vittorio Giovara
On Sun, Nov 30, 2014 at 6:06 PM, Luca Barbato wrote: > On 30/11/14 23:43, Andreas Cadhalpun wrote: >> >> On 20.11.2014 17:59, Andreas Cadhalpun wrote: >>> >>> Hi, >>> >>> currently the native aac encoder is marked as experimental, while the >>> libvo_aacenc encoder is not. >>>

Re: [libav-devel] [FFmpeg-devel] [PATCH] aacpsy: avoid norm_fac becoming NaN

2016-03-19 Thread Vittorio Giovara
On Tue, Apr 21, 2015 at 12:50 PM, Andreas Cadhalpun wrote: > On 21.04.2015 02:20, Claudio Freire wrote: >> On Mon, Apr 20, 2015 at 9:13 PM, Michael Niedermayer >> wrote: >>> On Mon, Apr 20, 2015 at 09:07:14PM -0300, Claudio Freire wrote:

Re: [libav-devel] [PATCH] takdec: ensure chan2 is a valid channel index

2016-03-19 Thread Vittorio Giovara
On Tue, Jun 9, 2015 at 8:26 PM, Luca Barbato wrote: > On 10/06/15 00:12, Andreas Cadhalpun wrote: >> If chan2 is not smaller than the number of channels, it can cause >> segmentation faults due to dereferencing a NULL pointer. >> >> Signed-off-by: Andreas Cadhalpun

Re: [libav-devel] [PATCH 5/6] avio: Add avio_read wrapper to simplify error checking

2016-03-19 Thread Vittorio Giovara
On Sun, Jun 7, 2015 at 1:07 PM, Luca Barbato wrote: > On 07/06/15 16:30, Andreas Cadhalpun wrote: >> It seems Luca accidentally pushed an old version of this patch. >> The pushed version still has the ffio_read_size name and Luca's >> comment about the error handling is also

Re: [libav-devel] [PATCH] img2: Provide an option take the path verbatim

2016-03-19 Thread Vittorio Giovara
On Sat, Feb 7, 2015 at 8:22 AM, Luca Barbato wrote: > On 02/02/15 01:36, Luca Barbato wrote: >> There can be valid files path sporting %-sequences or have >> files ending with .V in the the same directory but not related. >> --- >> >> Mainly an rfc and pro-memoria. >> > >

Re: [libav-devel] [PATCH] lavf: add a format flag for separate carriers, and an event for detecting carrier presence.

2016-03-19 Thread Vittorio Giovara
On Mon, Jun 8, 2015 at 5:28 AM, John Högberg wrote: > Luca Barbato wrote: >> Please do. I thought about the alternative to support your scenario (beside >> having a callback API that I'll blog about soonish) and another way to do >> that >> is to have an

Re: [libav-devel] [FFmpeg-devel] [PATCH 2/2] matroskadec: validate audio channels and bitdepth

2016-03-19 Thread Vittorio Giovara
On Tue, Jun 16, 2015 at 3:55 PM, Andreas Cadhalpun wrote: > On 16.06.2015 00:37, Luca Barbato wrote: >> On 16/06/15 00:14, Andreas Cadhalpun wrote: I wonder if the sanity check in the decoder would be enough to not have other problems down the line. >>>

Re: [libav-devel] [PATCH] x86: Add SSSE3_SLOW CPU flag and related convenience macros

2016-03-19 Thread Vittorio Giovara
On Tue, Jan 19, 2016 at 7:55 AM, Diego Biurrun wrote: > --- > > Fixed FAST_SLOW typo. > > doc/APIchanges | 3 +++ > libavutil/cpu.h | 1 + > libavutil/version.h | 2 +- > libavutil/x86/cpu.h | 6 ++ > 4 files changed, 11 insertions(+), 1 deletion(-) > > diff --git

Re: [libav-devel] [PATCH] movenc: check for the error in the AVIOContext and write

2016-03-19 Thread Vittorio Giovara
On Thu, Aug 27, 2015 at 7:20 AM, Luca Barbato wrote: > On 27/08/15 13:10, Martin Storsjö wrote: >> On Thu, 27 Aug 2015, Alexandra Hájková wrote: >> >>> the error message if there is some error. MOV has its header >>> at the end of the file so the output will be corrupted if

Re: [libav-devel] [PATCH] hevc: Parse the encoder info SEI

2016-03-19 Thread Vittorio Giovara
On Sat, Oct 17, 2015 at 6:25 PM, Luca Barbato wrote: > On 16/10/15 19:46, Anton Khirnov wrote: >> >> If the block above got executed, won't it attemp to skip it again? > > > it starts skipping from whenever `i` was before, alternatively I can just > process all the buffer and

Re: [libav-devel] [PATCH] asfdec: hadle invalid Object size properly

2016-03-19 Thread Vittorio Giovara
On Mon, Feb 8, 2016 at 3:12 PM, Vittorio Giovara wrote: > On Tue, Jun 30, 2015 at 5:19 AM, Luca Barbato wrote: >> On 30/06/15 11:35, Alexandra Hájková wrote: >>> --- >>> libavformat/asfdec.c | 12 ++-- >>> 1 file changed, 10 insertions(+),

Re: [libav-devel] [PATCH] libavcodec/util: Fix timebase overflow check

2016-03-19 Thread Vittorio Giovara
On Wed, Jan 27, 2016 at 5:19 PM, Derek Buitenhuis wrote: > On 1/27/2016 10:16 PM, Luca Barbato wrote: >> When it is zero? (isn't there a check before to sanitize the time_base ?) > > I only see a check for audio. What happened to this patch? -- Vittorio

Re: [libav-devel] [PATCH] avconv: Move the reporting in a separate thread

2016-03-19 Thread Vittorio Giovara
On Thu, Feb 4, 2016 at 7:07 AM, Luca Barbato wrote: > On 04/02/16 12:52, Vittorio Giovara wrote: >> is this still needed? > > Yes, I'll send and update on this Friday. > > lu I might have forgotten to check, and couldn't find it in the tree, where is the new version? --

Re: [libav-devel] [PATCH] diracdec: Move strides to bytes, and pointer types to uint8_t.

2016-03-19 Thread Vittorio Giovara
On Sun, Dec 6, 2015 at 8:57 AM, Diego Biurrun wrote: > On Sun, Dec 06, 2015 at 12:36:24PM +, Kieran Kunhya wrote: >> Start templating functions for move to support 10-bit >> Parts of this patch were written by Rostislav Pehlivanov >> --- a/libavcodec/diracdec.c >> +++

Re: [libav-devel] [PATCH] opus_silk: fix out of array read in silk_lsf2lpc

2016-03-19 Thread Vittorio Giovara
On Sat, Mar 19, 2016 at 4:44 PM, Vittorio Giovara wrote: > On Tue, Dec 15, 2015 at 4:41 PM, Andreas Cadhalpun > wrote: >> On 15.12.2015 08:17, Anton Khirnov wrote: >>> Can you share the sample that shows the problem? >> >> I could,

Re: [libav-devel] [PATCH] avio: Add avio_check2

2016-03-19 Thread Vittorio Giovara
On Thu, Feb 4, 2016 at 6:56 AM, Vittorio Giovara wrote: > On Sun, Nov 8, 2015 at 5:12 PM, Luca Barbato wrote: >> On 08/11/15 16:43, Anton Khirnov wrote: >>> Ok, now that sounds like a real use case. But that's really only applies >>> to img2dec

Re: [libav-devel] [PATCH 1/3] x264: Optionally forward the input frame type

2016-03-19 Thread Vittorio Giovara
On Sat, Feb 6, 2016 at 6:02 AM, Luca Barbato wrote: > On 04/02/16 13:09, Vittorio Giovara wrote: >> On Tue, Sep 1, 2015 at 12:50 AM, Luca Barbato wrote: >>> On 31/08/15 20:55, Anton Khirnov wrote: Looks like a hack to me. If you don't want to force

Re: [libav-devel] [PATCH] opus_silk: fix out of array read in silk_lsf2lpc

2016-03-19 Thread Vittorio Giovara
On Tue, Dec 15, 2015 at 4:41 PM, Andreas Cadhalpun wrote: > On 15.12.2015 08:17, Anton Khirnov wrote: >> Can you share the sample that shows the problem? > > I could, but it's of no use for comparing with libopus, because their > decoder errors out in an

Re: [libav-devel] [PATCH] rtsp: Factor out the rtp stream opening

2016-03-19 Thread Vittorio Giovara
On Thu, Feb 4, 2016 at 7:08 AM, Luca Barbato wrote: > On 04/02/16 12:54, Vittorio Giovara wrote: > >> is this still needed? >> > > Yes. so.. any reason it's not pushed? -- Vittorio ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 6/7] avio: Disable incompatible pointer type warnings

2016-03-19 Thread Vittorio Giovara
On Sat, Feb 6, 2016 at 6:09 AM, Luca Barbato wrote: > On 04/02/16 13:04, Vittorio Giovara wrote: >> On Thu, Nov 26, 2015 at 5:17 PM, Luca Barbato wrote: >>> --- >>> libavformat/avio.c | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git

Re: [libav-devel] [PATCH] h264: Use isprint to sanitize the SEI debug message

2016-03-19 Thread Vittorio Giovara
On Sat, Feb 6, 2016 at 3:22 PM, Luca Barbato wrote: > On 06/02/16 19:39, Henrik Gramner wrote: >> On Sat, Feb 6, 2016 at 7:34 PM, Luca Barbato wrote: >>> Give how this function is used it is not really important, its purpose >>> is to not break the

Re: [libav-devel] [PATCH] mpegts: Forward the errors on mpeg4 objects parsing

2016-03-19 Thread Vittorio Giovara
On Tue, Feb 16, 2016 at 8:23 PM, Luca Barbato wrote: > --- > libavformat/mpegts.c | 25 - > 1 file changed, 16 insertions(+), 9 deletions(-) This patch is approved but not applied, what's the hold up? -- Vittorio

[libav-devel] [PATCH 4/8] opt: Add const to av_opt_next

2016-03-19 Thread Martin Storsjö
From: Lukasz Marek Also add const to pointers in static functions within opt.c where possible/necessary. --- Now adding it immediately instead of after the next bump, added a micro bump. --- libavutil/opt.c | 4 ++-- libavutil/opt.h | 2 +- libavutil/version.h

[libav-devel] [PATCH 2/8] avio: Add an option 'rw_timeout'

2016-03-19 Thread Martin Storsjö
From: Andrey Utkin If set non-zero, this limits duration of the retry_transfer_wrapper() loop, thus affecting ffurl_read*(), ffurl_write(). As soon as one single byte is successfully received/transmitted, the timer restarts. This has further changes by Michael

[libav-devel] [PATCH 5/8] opt: Add av_opt_copy()

2016-03-19 Thread Martin Storsjö
From: Michael Niedermayer This includes documentation and other modifications by Lukasz Marek and Martin Storsjö. --- Changed things according to Anton's suggestions. --- doc/APIchanges | 3 +++ libavutil/opt.c | 66

Re: [libav-devel] [PATCH 6/8] RFC: avio: Copy URLContext generic options into child URLContexts

2016-03-19 Thread Martin Storsjö
On Sat, 19 Mar 2016, Martin Storsjö wrote: Since all URLContexts have the same AVOptions, such AVOptions will be applied on the outermost context only and removed from the dict, while they probably make sense on all contexts. This makes sure that rw_timeout gets propagated to the innermost

[libav-devel] [PATCH 7/8] tcp: Use rw_timeout for setting the connect/listen timeouts

2016-03-19 Thread Martin Storsjö
Apply the default value for timeout in code instead of via the avoption, to allow distinguishing the default value from the user not setting anything at all. --- Previously approved. --- libavformat/tcp.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/tcp.c

[libav-devel] [PATCH 3/8] file: Add an option for following a file that is being written

2016-03-19 Thread Martin Storsjö
Using this requires setting the rw_timeout option to make it terminate, alternatively using the interrupt callback (if used via the API). --- Added documentation, added a micro bump. --- doc/protocols.texi| 11 +++ libavformat/file.c| 4 libavformat/version.h | 2 +- 3

[libav-devel] [PATCH 1/8] avio: Apply avoptions on the URLContext itself as well

2016-03-19 Thread Martin Storsjö
Currently the list of avoptions for URLContext is empty though, but such options will be added. --- This one didn't receive any comments in the previous round. --- libavformat/avio.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/avio.c b/libavformat/avio.c index

[libav-devel] [PATCH 8/8] unix: Use rw_timeout for setting the connect timeout

2016-03-19 Thread Martin Storsjö
--- Previously approved. --- libavformat/unix.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/unix.c b/libavformat/unix.c index 6bb677d..647e7e8 100644 --- a/libavformat/unix.c +++ b/libavformat/unix.c @@ -73,6 +73,9 @@ static int unix_open(URLContext *h, const char

[libav-devel] [PATCH 6/8] RFC: avio: Copy URLContext generic options into child URLContexts

2016-03-19 Thread Martin Storsjö
Since all URLContexts have the same AVOptions, such AVOptions will be applied on the outermost context only and removed from the dict, while they probably make sense on all contexts. This makes sure that rw_timeout gets propagated to the innermost URLContext (to make sure it gets passed to the

Re: [libav-devel] [PATCH 5/8] opt: Add av_opt_copy()

2016-03-19 Thread Martin Storsjö
On Sat, 19 Mar 2016, Anton Khirnov wrote: Quoting Martin Storsjö (2016-03-19 20:01:15) On Sat, 19 Mar 2016, Anton Khirnov wrote: > Quoting Martin Storsjö (2016-03-18 13:01:39) >> From: Michael Niedermayer >> >> This includes documentation and other modifications by >>

Re: [libav-devel] [PATCH 5/8] opt: Add av_opt_copy()

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-19 20:01:15) > On Sat, 19 Mar 2016, Anton Khirnov wrote: > > > Quoting Martin Storsjö (2016-03-18 13:01:39) > >> From: Michael Niedermayer > >> > >> This includes documentation and other modifications by > >> Lukasz Marek. > >> --- > >>

Re: [libav-devel] [PATCH 4/8] opt: Add const to av_opt_next in the next major bump

2016-03-19 Thread Martin Storsjö
On Sat, 19 Mar 2016, Anton Khirnov wrote: Quoting Martin Storsjö (2016-03-18 13:01:38) From: Lukasz Marek Also add const to pointers in static functions within opt.c where possible. --- Or can we freely add const here without waiting for a major bump? I don't see

Re: [libav-devel] [PATCH 6/8] RFC: avio: Copy URLContext generic options into child URLContexts

2016-03-19 Thread Martin Storsjö
On Sat, 19 Mar 2016, Anton Khirnov wrote: Quoting Martin Storsjö (2016-03-18 13:01:40) Since all URLContexts have the same AVOptions, such AVOptions will be applied on the outermost context only and removed from the dict, while they probably make sense on all contexts. This makes sure that

Re: [libav-devel] [PATCH 5/8] opt: Add av_opt_copy()

2016-03-19 Thread Martin Storsjö
On Sat, 19 Mar 2016, Anton Khirnov wrote: Quoting Martin Storsjö (2016-03-18 13:01:39) From: Michael Niedermayer This includes documentation and other modifications by Lukasz Marek. --- doc/APIchanges | 3 +++ libavutil/opt.c | 58

Re: [libav-devel] [PATCH 3/8] file: Add an option for following a file that is being written

2016-03-19 Thread Martin Storsjö
On Sat, 19 Mar 2016, Anton Khirnov wrote: Quoting Martin Storsjö (2016-03-18 13:01:37) Using this requires setting the rw_timeout option to make it terminate, alternatively using the interrupt callback (if used via the API). --- libavformat/file.c | 4 1 file changed, 4 insertions(+)

Re: [libav-devel] [PATCH 2/8] avio: Add an option 'rw_timeout'

2016-03-19 Thread Martin Storsjö
On Sat, 19 Mar 2016, Anton Khirnov wrote: Quoting Martin Storsjö (2016-03-18 13:01:36) From: Andrey Utkin If set non-zero, this limits duration of the retry_transfer_wrapper() loop, thus affecting ffurl_read*(), ffurl_write(). As soon as one single byte is

Re: [libav-devel] [PATCH 4/8] opt: Add const to av_opt_next in the next major bump

2016-03-19 Thread Vittorio Giovara
On Sat, Mar 19, 2016 at 11:14 AM, Anton Khirnov wrote: > Quoting Martin Storsjö (2016-03-18 13:01:38) >> From: Lukasz Marek >> >> Also add const to pointers in static functions within opt.c where >> possible. >> --- >> Or can we freely add const here

Re: [libav-devel] [PATCH 2/9] lavf: generic hardware surface upload and download

2016-03-19 Thread Vittorio Giovara
On Tue, Mar 15, 2016 at 6:57 PM, Mark Thompson wrote: > --- > libavfilter/Makefile| 2 + > libavfilter/allfilters.c| 2 + > libavfilter/avfilter.c | 2 + > libavfilter/avfilter.h | 9 ++ > libavfilter/vf_hwdownload.c | 212

Re: [libav-devel] [PATCH 0/9] Generic hw filter support + VAAPI hwcontext and encoders

2016-03-19 Thread Luca Barbato
On 15/03/16 23:55, Mark Thompson wrote: > Hi all, > > Whole series follows again. > > Changes since last time: > > * APIchanges added where appropriate. > > * Memory allocation cleaned up; now valgrind-clean for normal use (might > still be some leaks in error cases). > > * Some

Re: [libav-devel] [PATCH 4/8] opt: Add const to av_opt_next in the next major bump

2016-03-19 Thread Luca Barbato
On 18/03/16 13:01, Martin Storsjö wrote: > From: Lukasz Marek > > Also add const to pointers in static functions within opt.c where > possible. > --- > Or can we freely add const here without waiting for a major bump? > I don't see any case where adding const actually

[libav-devel] [PATCH] aac_adtstoasc_bsf: convert to the new API

2016-03-19 Thread Anton Khirnov
--- libavcodec/aac_adtstoasc_bsf.c | 104 +++-- libavcodec/allcodecs.c | 1 - libavcodec/bitstream_filters.c | 5 ++ 3 files changed, 74 insertions(+), 36 deletions(-) diff --git a/libavcodec/aac_adtstoasc_bsf.c b/libavcodec/aac_adtstoasc_bsf.c

Re: [libav-devel] [PATCH] New BSF API v2

2016-03-19 Thread Anton Khirnov
ping on the last two patches in the set. -- Anton Khirnov ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 02/14] aac_adtstoasc_bsf: convert to the new API

2016-03-19 Thread Anton Khirnov
Quoting Luca Barbato (2016-03-07 09:10:14) > On 07/03/16 08:59, Luca Barbato wrote: > > On 04/03/16 09:15, Anton Khirnov wrote: > >> --- > >> libavcodec/aac_adtstoasc_bsf.c | 95 > >> ++ > >> libavcodec/allcodecs.c | 1 - > >>

Re: [libav-devel] [PATCH 7/8] tcp: Use rw_timeout for setting the connect/listen timeouts

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:41) > Apply the default value for timeout in code instead of via the > avoption, to allow distinguishing the default value from the user > not setting anything at all. > --- > libavformat/tcp.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) >

Re: [libav-devel] [PATCH 8/8] unix: Use rw_timeout for setting the connect timeout

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:42) > --- > libavformat/unix.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/libavformat/unix.c b/libavformat/unix.c > index 6bb677d..647e7e8 100644 > --- a/libavformat/unix.c > +++ b/libavformat/unix.c > @@ -73,6 +73,9 @@ static int

Re: [libav-devel] [PATCH 6/8] RFC: avio: Copy URLContext generic options into child URLContexts

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:40) > Since all URLContexts have the same AVOptions, such AVOptions > will be applied on the outermost context only and removed from the > dict, while they probably make sense on all contexts. > > This makes sure that rw_timeout gets propagated to the

Re: [libav-devel] [PATCH 5/8] opt: Add av_opt_copy()

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:39) > From: Michael Niedermayer > > This includes documentation and other modifications by > Lukasz Marek. > --- > doc/APIchanges | 3 +++ > libavutil/opt.c | 58 > + >

Re: [libav-devel] [PATCH 3/8] file: Add an option for following a file that is being written

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:37) > Using this requires setting the rw_timeout option to make it > terminate, alternatively using the interrupt callback (if used via > the API). > --- > libavformat/file.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/file.c

Re: [libav-devel] [PATCH 4/8] opt: Add const to av_opt_next in the next major bump

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:38) > From: Lukasz Marek > > Also add const to pointers in static functions within opt.c where > possible. > --- > Or can we freely add const here without waiting for a major bump? > I don't see any case where adding const

Re: [libav-devel] [PATCH 2/8] avio: Add an option 'rw_timeout'

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:36) > From: Andrey Utkin > > If set non-zero, this limits duration of the retry_transfer_wrapper() > loop, thus affecting ffurl_read*(), ffurl_write(). As soon as > one single byte is successfully received/transmitted, the

  1   2   >