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

2015-06-07 Thread John Högberg
Luca Barbato wrote: > I was waiting for Anton, I'm more and more convinced that the feature is > important and I'm planning to fit it in the avformat API that will match > the avcodec API I detailed here[1]. > > I'm not sure about the name, but they can be changed later. > > lu > > > [1]: https://

Re: [libav-devel] [PATCH] Introduce a TextureDSP module

2015-06-07 Thread Vittorio Giovara
On Sat, Jun 6, 2015 at 4:27 AM, James Almer wrote: > On 02/06/15 8:09 AM, Vittorio Giovara wrote: >> +/* Alpha compression function */ >> +static void compress_alpha(uint8_t *dst, ptrdiff_t stride, const uint8_t >> *block) >> +{ >> +int i, j; >> +int dist, bias, dist4, dist2, bits, mask;

[libav-devel] [PATCH] configure: Support MSVC 2015

2015-06-07 Thread Luca Barbato
The C runtime C99 compatibility had been improved a lot and it now rejects some of the compatibility defines provided for the older versions. Many thanks to Ray for the time spent testing. Bug-Id: 864 CC: libav-sta...@libav.org --- Martin does it sound ok for you? configure| 22 +++

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

2015-06-07 Thread Luca Barbato
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 not addressed. Good thing it is an internal function and thanks for spotting!

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

2015-06-07 Thread Luca Barbato
On 07/06/15 16:37, Andreas Cadhalpun wrote: > On 01.06.2015 09:13, Luca Barbato wrote: >> On 01/06/15 00:37, Andreas Cadhalpun wrote: >>> Since I don't expect that many people would start using that function before >>> a release, I think delaying this doesn't do much good. But then, it doesn't >>>

Re: [libav-devel] [PATCH] vp8: check for too large dimensions

2015-06-07 Thread Ronald S. Bultje
Hi, On Sun, Jun 7, 2015 at 10:05 AM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > +#define MARGIN (16 << 2) > +#define MAX_MB_SIZE (((INT16_MAX - MARGIN) >> 6) + 1) > So this is roughly 9 bits. > +if (s->avctx->coded_width > MAX_MB_SIZE * 16 || > +s->avctx->coded

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

2015-06-07 Thread Andreas Cadhalpun
On 01.06.2015 09:13, Luca Barbato wrote: > On 01/06/15 00:37, Andreas Cadhalpun wrote: >> Since I don't expect that many people would start using that function before >> a release, I think delaying this doesn't do much good. But then, it doesn't >> do much harm either, for precisely the same reason

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

2015-06-07 Thread Andreas Cadhalpun
On 26.05.2015 16:41, Vittorio Giovara wrote: > On Tue, May 26, 2015 at 3:24 PM, wm4 wrote: >> On Tue, 26 May 2015 14:24:38 +0100 >> Vittorio Giovara wrote: >>> +int ffio_read_size(AVIOContext *s, unsigned char *buf, int size) >>> +{ >>> +int ret = avio_read(s, buf, size); >>> +if (ret !=

[libav-devel] [PATCH] vp8: check for too large dimensions

2015-06-07 Thread Andreas Cadhalpun
If the dimensions are too large, s->mb_width or s->mb_height can become too large, leading to an int16_t overflow of s->mv_max.{x,y}. Signed-off-by: Andreas Cadhalpun --- libavcodec/vp8.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/libavcodec/vp8.c b/libavcodec/

Re: [libav-devel] [PATCH] configure: Support MSVC 2015

2015-06-07 Thread Luca Barbato
On 06/06/15 20:58, Luca Barbato wrote: > On 05/06/15 19:48, Luca Barbato wrote: >> The C runtime C99 compatibility had been improved a lot and it now >> rejects some of the compatibility defines provided for the older >> versions. >> >> Many thanks to Ray for the time spent testing. >> >> Bug-Id: 8