[libav-devel] [PATCH] mov: Implement support for multiple sample description tables

2016-06-10 Thread Vittorio Giovara
Store data from each stsd in a separate extradata buffer, keep track of the stsc index for read and seek operations, switch buffers when the index differs. Decoder is notified with an AV_PKT_DATA_NEW_EXTRADATA packet side data. Since H264 supports this notification, and can be reset midstream, ena

Re: [libav-devel] [PATCH 3/3] mov: Implement support for multiple sample description tables

2016-06-10 Thread Vittorio Giovara
On Fri, Jun 10, 2016 at 2:32 PM, Martin Storsjö wrote: > On Thu, 9 Jun 2016, Martin Storsjö wrote: > >> On Mon, 6 Jun 2016, Vittorio Giovara wrote: >> >>> On Mon, Jun 6, 2016 at 2:57 PM, Martin Storsjö wrote: On Wed, 1 Jun 2016, Vittorio Giovara wrote: > Store data from each st

Re: [libav-devel] [PATCH 3/5] mov: Implement support for multiple sample description tables

2016-06-10 Thread Vittorio Giovara
On Fri, Jun 10, 2016 at 1:23 PM, Anton Khirnov wrote: > Quoting Vittorio Giovara (2016-06-10 16:52:39) >> On Fri, Jun 10, 2016 at 4:56 AM, Anton Khirnov wrote: >> > Quoting Vittorio Giovara (2016-06-10 00:55:18) >> >> Store data from each stsd in a separate extradata buffer, keep track of >> >> t

Re: [libav-devel] [PATCH 3/3] mov: Implement support for multiple sample description tables

2016-06-10 Thread Martin Storsjö
On Thu, 9 Jun 2016, Martin Storsjö wrote: On Mon, 6 Jun 2016, Vittorio Giovara wrote: On Mon, Jun 6, 2016 at 2:57 PM, Martin Storsjö wrote: On Wed, 1 Jun 2016, Vittorio Giovara wrote: Store data from each stsd in a separate extradata buffer, keep track of the stsc index for read and seek o

Re: [libav-devel] [PATCH 3/5] mov: Implement support for multiple sample description tables

2016-06-10 Thread Anton Khirnov
Quoting Vittorio Giovara (2016-06-10 16:52:39) > On Fri, Jun 10, 2016 at 4:56 AM, Anton Khirnov wrote: > > Quoting Vittorio Giovara (2016-06-10 00:55:18) > >> Store data from each stsd in a separate extradata buffer, keep track of > >> the stsc index for read and seek operations, switch buffers wh

Re: [libav-devel] [PATCH 077/115] dv: Convert to the new bitreader

2016-06-10 Thread Luca Barbato
On 09/06/16 19:24, Vittorio Giovara wrote: > maybe these two functions could be dv_ prefixed to avoid clashing with > the main bitstream API? I considered moving them to the main header since other codecs might enjoy to unget bits, luckily seems that's not the case so yes, it could be made dv-only

Re: [libav-devel] [PATCH 4/5] mov: Support prores with multiple stsd

2016-06-10 Thread Vittorio Giovara
On Fri, Jun 10, 2016 at 4:56 AM, Anton Khirnov wrote: > Quoting Vittorio Giovara (2016-06-10 00:55:19) >> This function needs to return false, or data in the additional tables >> will be skipped, and the decoder will not be able to decode frames >> associated with them. >> >> --- >> libavformat/m

Re: [libav-devel] [PATCH 3/5] mov: Implement support for multiple sample description tables

2016-06-10 Thread Vittorio Giovara
On Fri, Jun 10, 2016 at 4:56 AM, Anton Khirnov wrote: > Quoting Vittorio Giovara (2016-06-10 00:55:18) >> Store data from each stsd in a separate extradata buffer, keep track of >> the stsc index for read and seek operations, switch buffers when the >> index differs. Decoder is notified with an AV

Re: [libav-devel] [PATCH 002/115] Add a new bitstream reader to replace get_bits

2016-06-10 Thread Luca Barbato
On 10/06/16 09:41, Anton Khirnov wrote: > Quoting Diego Biurrun (2016-06-09 17:11:44) >> From: Alexandra Hájková >> >> The new bit reader features a simpler API and an implementation without >> stacks of nested macros. Performance is the same or better, especially >> for mezzanine formats and 64-b

Re: [libav-devel] [PATCH 4/5] mov: Support prores with multiple stsd

2016-06-10 Thread Anton Khirnov
Quoting Vittorio Giovara (2016-06-10 00:55:19) > This function needs to return false, or data in the additional tables > will be skipped, and the decoder will not be able to decode frames > associated with them. > > --- > libavformat/mov.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git

Re: [libav-devel] [PATCH 3/5] mov: Implement support for multiple sample description tables

2016-06-10 Thread Anton Khirnov
Quoting Vittorio Giovara (2016-06-10 00:55:18) > Store data from each stsd in a separate extradata buffer, keep track of > the stsc index for read and seek operations, switch buffers when the > index differs. Decoder is notified with an AV_PKT_DATA_NEW_EXTRADATA > packet side data. > > Since H264

Re: [libav-devel] [PATCH] vaapi_encode: Add driver-specific behaviour for i965 param buffers

2016-06-10 Thread Mark Thompson
On 10/06/16 08:06, Anton Khirnov wrote: > Quoting Mark Thompson (2016-06-10 01:25:58) >> Adds a general mechanism for setting driver quirks, and detects the >> driver name to set them. >> >> When the i965 driver is being used, destroy all parameter buffers >> explicitly after use despite passing th

Re: [libav-devel] [PATCH 004/115] eamad: Convert to the new bitstream reader

2016-06-10 Thread Anton Khirnov
Quoting Anton Khirnov (2016-06-10 10:16:00) > Quoting Diego Biurrun (2016-06-09 17:11:46) > > From: Alexandra Hájková > > > > --- > > libavcodec/eamad.c | 43 --- > > 1 file changed, 20 insertions(+), 23 deletions(-) > > > > diff --git a/libavcodec/eamad.

Re: [libav-devel] [PATCH 004/115] eamad: Convert to the new bitstream reader

2016-06-10 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:46) > From: Alexandra Hájková > > --- > libavcodec/eamad.c | 43 --- > 1 file changed, 20 insertions(+), 23 deletions(-) > > diff --git a/libavcodec/eamad.c b/libavcodec/eamad.c > index 3e8d4fd..c174ffd 100644 > ---

Re: [libav-devel] [PATCH 003/115] cllc: Convert to the new bitstream reader

2016-06-10 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:45) > From: Alexandra Hájková > > --- > libavcodec/cllc.c | 88 > +++ > 1 file changed, 36 insertions(+), 52 deletions(-) > > diff --git a/libavcodec/cllc.c b/libavcodec/cllc.c > index cdbed74..bac2b73

Re: [libav-devel] [PATCH 002/115] Add a new bitstream reader to replace get_bits

2016-06-10 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:44) > From: Alexandra Hájková > > The new bit reader features a simpler API and an implementation without > stacks of nested macros. Performance is the same or better, especially > for mezzanine formats and 64-bit platforms. > --- > Changelog

Re: [libav-devel] [PATCH] vaapi_encode: Add driver-specific behaviour for i965 param buffers

2016-06-10 Thread Anton Khirnov
Quoting Mark Thompson (2016-06-10 01:25:58) > Adds a general mechanism for setting driver quirks, and detects the > driver name to set them. > > When the i965 driver is being used, destroy all parameter buffers > explicitly after use despite passing them to vaRenderPicture(). > --- > An alternativ