[libav-devel] [PATCH 2/5] tiffenc: Check zlib support for deflate option during initialization

This converts a misleading error about an option not found to a more meaningful one. --- libavcodec/tiffenc.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavcodec/tiffenc.c b/libavcodec/tiffenc.c index 7c23ee2..4940399 100644 --- a/libavcodec/tiffenc.c +++

[libav-devel] [PATCH 3/5] tiffenc: Check av_pix_fmt_desc_get() return value

--- libavcodec/tiffenc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/tiffenc.c b/libavcodec/tiffenc.c index 4940399..cf6e262 100644 --- a/libavcodec/tiffenc.c +++ b/libavcodec/tiffenc.c @@ -254,7 +254,9 @@ static int encode_frame(AVCodecContext *avctx,

[libav-devel] [PATCH 1/5] targaenc: Move size check to initialization function

--- libavcodec/targaenc.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/libavcodec/targaenc.c b/libavcodec/targaenc.c index 204ecfe..f0cee38 100644 --- a/libavcodec/targaenc.c +++ b/libavcodec/targaenc.c @@ -89,10 +89,6 @@ static int

[libav-devel] [PATCH 5/5] rle: Convert into a proper dsp module

--- configure | 4 libavcodec/Makefile | 7 --- libavcodec/rle.c | 17 - libavcodec/rle.h | 50 +++--- libavcodec/sgienc.c | 15 ++- libavcodec/targaenc.c | 17 +

[libav-devel] [PATCH 4/5] rle: K formatting cosmetics

--- libavcodec/rle.c | 34 -- libavcodec/rle.h | 9 + 2 files changed, 25 insertions(+), 18 deletions(-) diff --git a/libavcodec/rle.c b/libavcodec/rle.c index 8a2d922..6c8bf27 100644 --- a/libavcodec/rle.c +++ b/libavcodec/rle.c @@ -18,26 +18,29 @@ *

Re: [libav-devel] [PATCH 08/11] qsvdec: use the same mfxFrameInfo for allocating frames that was passed to DECODE_Init

On Sat, Jun 25, 2016 at 10:27 AM, Anton Khirnov wrote: > Stop duplicating this information. > --- > libavcodec/qsvdec.c | 15 ++- > libavcodec/qsvdec.h | 1 + > 2 files changed, 7 insertions(+), 9 deletions(-) probably ok -- Vittorio

Re: [libav-devel] [PATCH 01/11] pixfmt: add P010 pixel format

On Sat, Jun 25, 2016 at 10:27 AM, Anton Khirnov wrote: > From: Hendrik Leppkes > > P010 is the 10-bit variant of NV12 (planar luma, packed chroma), using two > bytes per component to store 10-bit data plus 6-bit zeroes in the LSBs. > > Signed-off-by: Anton

Re: [libav-devel] [PATCH 1/4] pixdesc: Add aliases to SMPTE color properties

On Tue, Jun 21, 2016 at 10:22 AM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > Drop ST from names and symbols, it does not add anything distinctive or > descriptive. > --- > libavcodec/options_table.h | 9 ++--- > libavcodec/version.h | 2 +- > libavu

Re: [libav-devel] [PATCH 02/11] swscale: add P010 input support

On Sat, Jun 25, 2016 at 10:27 AM, Anton Khirnov wrote: > From: Hendrik Leppkes > > Signed-off-by: Anton Khirnov > --- > libswscale/input.c| 52 > +++ > libswscale/swscale_unscaled.c

Re: [libav-devel] [PATCH] Add an OpenH264 decoder

On Thu, Jun 23, 2016 at 3:03 PM, Martin Storsjö <mar...@martin.st> wrote: > On Thu, 23 Jun 2016, Vittorio Giovara wrote: > >> On Thu, Jun 23, 2016 at 7:06 AM, Martin Storsjö <mar...@martin.st> wrote: >>> >>> While it is less featureful (and slower) tha

Re: [libav-devel] [PATCH] bitstream: Add bitstream_apply_sign() helper function

On Thu, Jun 23, 2016 at 9:54 AM, Diego Biurrun wrote: > This reads a sign bit from the bitstream and flips the sign of the > provided value accordingly. > --- > > .. as suggested by Anton .. > > libavcodec/bitstream.h | 7 +++ > 1 file changed, 7 insertions(+) > > diff

Re: [libav-devel] [PATCH 2/2] msmpeg4: Remove commented-out debug logging code

On Thu, Jun 23, 2016 at 8:46 AM, Diego Biurrun wrote: > --- > > This does not seem worth the ifdef soup to me. > > libavcodec/msmpeg4dec.c | 25 - > 1 file changed, 25 deletions(-) > > diff --git a/libavcodec/msmpeg4dec.c b/libavcodec/msmpeg4dec.c >

Re: [libav-devel] [PATCH 1/2] msmpeg4: Remove some broken, commented-out cruft

On Thu, Jun 23, 2016 at 8:46 AM, Diego Biurrun wrote: > --- > > This must have stopped compiling in 2002... > > libavcodec/msmpeg4dec.c | 26 -- > 1 file changed, 26 deletions(-) > > diff --git a/libavcodec/msmpeg4dec.c b/libavcodec/msmpeg4dec.c > index

Re: [libav-devel] [PATCH] Add an OpenH264 decoder

On Thu, Jun 23, 2016 at 7:06 AM, Martin Storsjö wrote: > While it is less featureful (and slower) than the built-in H264 > decoder, one could potentially want to use it to take advantage > of the cisco patent license offer. > --- > I got a user explicitly requesting this

Re: [libav-devel] [PATCH 03/10] avconv: buffer the packets written while the muxer is not initialized

On Tue, Jun 21, 2016 at 1:50 PM, Anton Khirnov wrote: > --- > avconv.c| 76 > +++-- > avconv.h| 7 ++ > avconv_opt.c| 10 > doc/avconv.texi | 9 +++ > 4 files changed, 84 insertions(+),

Re: [libav-devel] [PATCH 01/10] lavfi: set the link hwframes context before configuring the dst input

On Tue, Jun 21, 2016 at 1:50 PM, Anton Khirnov wrote: > The destination filter might expect the hw frames context to be already > set (this is the case e.g. for hwdownload). > --- > libavfilter/avfilter.c | 16 > 1 file changed, 8 insertions(+), 8 deletions(-)

Re: [libav-devel] [PATCH] Simplify some sign detection operations with an intermediate variable

On Tue, Jun 21, 2016 at 11:02 AM, Diego Biurrun wrote: > --- > > I've pulled this out instead of doing it as part of the new bitstream reader > port. > > libavcodec/eamad.c | 4 +++- > libavcodec/mdec.c | 4 +++- > libavcodec/mpeg12.c| 5 +++-- >

[libav-devel] [PATCH 1/4] pixdesc: Add aliases to SMPTE color properties

Drop ST from names and symbols, it does not add anything distinctive or descriptive. --- libavcodec/options_table.h | 9 ++--- libavcodec/version.h | 2 +- libavutil/pixdesc.c| 6 +++--- libavutil/pixfmt.h | 9 ++--- libavutil/version.h| 2 +- 5 files

[libav-devel] [PATCH 3/4] options_table: Add aliases for color properties

All option names now match the ones provided by the av_color_*_name(). --- libavcodec/options_table.h | 32 libavcodec/version.h | 2 +- 2 files changed, 25 insertions(+), 9 deletions(-) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h

[libav-devel] [PATCH 4/4] pixdesc: Add documentation about the main use-case of color name APIs

--- doc/APIchanges | 6 ++ libavutil/pixdesc.h | 9 + 2 files changed, 15 insertions(+) diff --git a/doc/APIchanges b/doc/APIchanges index de0213f..712f0b6 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -13,6 +13,12 @@ libavutil: 2015-08-28 API changes, most recent

[libav-devel] [PATCH 2/4] pixdesc: Fix AVCOL_TRC_BT2020_12 name

--- libavutil/pixdesc.c | 2 +- libavutil/version.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index f43f2a8..6d9e38d 100644 --- a/libavutil/pixdesc.c +++ b/libavutil/pixdesc.c @@ -1633,7 +1633,7 @@ static const char

Re: [libav-devel] [PATCH 2/2] truemotion2rt: Convert to the new bitstream reader

On Tue, Jun 21, 2016 at 8:42 AM, Diego Biurrun wrote: > --- > libavcodec/truemotion2rt.c | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/truemotion2rt.c b/libavcodec/truemotion2rt.c > index 4d398fb..666111c 100644 > ---

Re: [libav-devel] [PATCH 1/2] magicyuv: Convert to the new bitstream reader

On Tue, Jun 21, 2016 at 8:42 AM, Diego Biurrun wrote: > --- > libavcodec/magicyuv.c | 33 + > 1 file changed, 17 insertions(+), 16 deletions(-) > > diff --git a/libavcodec/magicyuv.c b/libavcodec/magicyuv.c > index 310ead4..8dec6bb 100644 > ---

[libav-devel] [PATCH] truemotion2rt: Use ff_set_dimensions

--- libavcodec/truemotion2rt.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/libavcodec/truemotion2rt.c b/libavcodec/truemotion2rt.c index 4d398fb..ed7fb68 100644 --- a/libavcodec/truemotion2rt.c +++ b/libavcodec/truemotion2rt.c @@ -63,7 +63,8 @@ static int

Re: [libav-devel] [PATCH 4/5] options_table: Add aliases for color properties

On Wed, Jun 15, 2016 at 2:54 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 15/06/16 20:25, Vittorio Giovara wrote: >> All option names now match the ones provided by the av_color_*_name(). >> --- >> Needs lavc micro version bump >>

[libav-devel] [PATCH] pixdesc: Add documentation about the main use-case of color name APIs

--- More documentation about this change as requested. Vittorio doc/APIchanges | 6 ++ libavutil/pixdesc.h | 8 2 files changed, 14 insertions(+) diff --git a/doc/APIchanges b/doc/APIchanges index de0213f..050c897 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -13,6

Re: [libav-devel] [PATCH] Add MagicYUV decoder

On Sun, Jun 19, 2016 at 9:33 AM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 19/06/16 00:23, Vittorio Giovara wrote: >> From: Paul B Mahol <one...@gmail.com> >> >> Signed-off-by: Paul B Mahol <one...@gmail.com> >> Signed-off-by: Vittorio Giovara <

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Thu, Jun 16, 2016 at 1:27 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > On Thu, Jun 16, 2016 at 1:09 PM, Luca Barbato <lu_z...@gentoo.org> wrote: >> On 16/06/16 18:56, Vittorio Giovara wrote: >>> The symbols with the unfortunate name were add

Re: [libav-devel] [PATCH] Add MagicYUV decoder

On Fri, Jun 17, 2016 at 1:45 PM, Diego Biurrun <di...@biurrun.de> wrote: > On Fri, Jun 17, 2016 at 12:10:31PM -0400, Vittorio Giovara wrote: >> On Fri, Jun 17, 2016 at 10:20 AM, Diego Biurrun <di...@biurrun.de> wrote: >> > On Mon, Jun 06, 2016 at 12:31:56PM

Re: [libav-devel] [PATCH] ffv1: Error out on unsupported format

On Thu, Jun 16, 2016 at 10:44 AM, Luca Barbato wrote: > From: Jerome Martinez > > Transparency is supported only by YUV and within specific bit depths. > --- > > Eventually had a look, added a similar block for the other colorspace as well. > >

[libav-devel] [PATCH] Add MagicYUV decoder

From: Paul B Mahol <one...@gmail.com> Signed-off-by: Paul B Mahol <one...@gmail.com> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- In this version - correct handling of raw slices - added size checks - any additional review comment Vittorio Changelog

[libav-devel] [PATCH] Add TrueMotion 2.0 Real Time decoder

From: Paul B Mahol <one...@gmail.com> Signed-off-by: Paul B Mahol <one...@gmail.com> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Addressed Luca's and Diego's comments. Vittorio Changelog | 1 + doc/general.texi | 2 + l

Re: [libav-devel] [PATCH] Add MagicYUV decoder

On Fri, Jun 17, 2016 at 8:32 AM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2016-06-06 18:31:56) >> From: Paul B Mahol <one...@gmail.com> >> >> +s->slice_height = bytestream2_get_le32(); >> +if ((s->slice_height &l

Re: [libav-devel] [PATCH] Add MagicYUV decoder

On Fri, Jun 17, 2016 at 10:20 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Mon, Jun 06, 2016 at 12:31:56PM -0400, Vittorio Giovara wrote: >> --- a/doc/general.texi >> +++ b/doc/general.texi >> @@ -672,6 +672,7 @@ following image formats are supported: >> @it

Re: [libav-devel] [PATCH] mov: handle files which do not have stsc data

On Thu, Jun 16, 2016 at 3:19 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 16/06/16 21:17, Vittorio Giovara wrote: >> --- >> libavformat/mov.c | 28 +++- >> 1 file changed, 15 insertions(+), 13 deletions(-) >> >> To be squa

[libav-devel] [PATCH] mov: handle files which do not have stsc data

--- libavformat/mov.c | 28 +++- 1 file changed, 15 insertions(+), 13 deletions(-) To be squashed in the mov patch (3/5) Vittorio diff --git a/libavformat/mov.c b/libavformat/mov.c index be09373..fe1f264 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -3674,19

Re: [libav-devel] [PATCH] ffv1: Drop version 2

On Thu, Jun 16, 2016 at 10:04 AM, Luca Barbato wrote: > The version is deemed non-standardizable and the current draft considers > it reserved. > > Do not produce files with that version. > --- > libavcodec/ffv1enc.c | 79 >

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Thu, Jun 16, 2016 at 1:09 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 16/06/16 18:56, Vittorio Giovara wrote: >> The symbols with the unfortunate name were added in 4a66422 (17 Sep >> 2015), they were not part of the 11 release, nor is the API in >> question

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

On Wed, Jun 15, 2016 at 3:24 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > 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. Deco

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Thu, Jun 16, 2016 at 4:31 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Wed, Jun 15, 2016 at 03:08:57PM -0400, Vittorio Giovara wrote: >> On Wed, Jun 15, 2016 at 3:00 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: >> > On Wed, Jun 15, 2016 at 8:53 PM, Lu

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 5:31 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > On Wed, Jun 15, 2016 at 11:06 PM, Vittorio Giovara > <vittorio.giov...@gmail.com> wrote: >> On Wed, Jun 15, 2016 at 3:49 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: >>> On

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 3:49 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > On Wed, Jun 15, 2016 at 9:08 PM, Vittorio Giovara > <vittorio.giov...@gmail.com> wrote: >> On Wed, Jun 15, 2016 at 3:00 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: >>> On Wed

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

, enable this feature only for multiple avcC's. All other stsd types (such as hvc1 and hev1) need decoder-side changes, so they are left disabled for now. This is implemented only in non-fragmented MOVs. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Squashed 5/5 in this one,

Re: [libav-devel] [PATCH 3/5] options_table: Drop the ST attribute from recent smpte color properties

On Wed, Jun 15, 2016 at 2:37 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > On Wed, Jun 15, 2016 at 8:25 PM, Vittorio Giovara > <vittorio.giov...@gmail.com> wrote: >> It does not add anything distinctive or descriptive. >> --- > > The name of the specific

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 3:00 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > On Wed, Jun 15, 2016 at 8:53 PM, Luca Barbato <lu_z...@gentoo.org> wrote: >> On 15/06/16 20:25, Vittorio Giovara wrote: >>> This allows to directly use av_color_*_name() return value

[libav-devel] [PATCH 1/5] pixfmt: Add ARIB STD-B76 color transfer characteristic

: http://phenix.it-sudparis.eu/jct/doc_end_user/current_document.php?id=10481 And also with latest proposal for color format in mkv: https://mailarchive.ietf.org/arch/search/?email_list=cellar=1=Colour+Format+proposal Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Adoped the sa

[libav-devel] [PATCH 5/5] pixdesc: Add new SMPTE 431, 432, 2085 color properties

Appeared in H.264 2016/02. --- Needs lavc micro version bump. Vittorio libavcodec/options_table.h | 3 +++ libavutil/pixdesc.c| 3 +++ libavutil/pixfmt.h | 3 +++ libavutil/version.h| 2 +- 4 files changed, 10 insertions(+), 1 deletion(-) diff --git

[libav-devel] [PATCH 3/5] options_table: Drop the ST attribute from recent smpte color properties

It does not add anything distinctive or descriptive. --- libavcodec/options_table.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h index 5c788ed..8c07022 100644 --- a/libavcodec/options_table.h +++

[libav-devel] [PATCH 4/5] options_table: Add aliases for color properties

All option names now match the ones provided by the av_color_*_name(). --- Needs lavc micro version bump libavcodec/options_table.h | 27 ++- 1 file changed, 22 insertions(+), 5 deletions(-) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h index

[libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

This allows to directly use av_color_*_name() return value as command line parameters for the encoder. --- libavutil/pixdesc.c | 8 libavutil/version.h | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index 209d107..216452c

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

On Thu, Jun 9, 2016 at 11:13 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/dirac.c | 89 > +++--- > 1 file changed, 45 insertions(+), 44 deletions(-) seems ok -- Vittorio

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

On Thu, Jun 9, 2016 at 11:12 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/exr.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) ok -- Vittorio ___ libav-devel

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

On Thu, Jun 9, 2016 at 11:12 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/hq_hqa.c | 48 > 1 file changed, 24 insertions(+), 24 deletions(-) > i think it's ok -- Vittorio

Re: [libav-devel] [PATCH 042/115] hqx: Convert to the new bitstream header

On Thu, Jun 9, 2016 at 11:12 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/hqx.c | 64 > > libavcodec/hqx.h | 5 +++-- > 2 files changed, 35 insertions(+), 34

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

On Thu, Jun 9, 2016 at 11:12 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavformat/mov.c| 10 ++ > libavformat/movenc.c | 41 + > 2 files changed, 27 insertions(+), 24 deletions(-) >

Re: [libav-devel] [PATCH 113/115] Remove legacy get_bits headers

On Thu, Jun 9, 2016 at 11:13 AM, Diego Biurrun wrote: > --- > libavcodec/get_bits.h | 531 > libavcodec/golomb_legacy.h | 589 > - > libavcodec/unary_legacy.h | 56 - > 3 files

Re: [libav-devel] [PATCH] h264dsp: Change type of stride parameters to ptrdiff_t

On Mon, Jun 13, 2016 at 1:29 PM, Diego Biurrun wrote: > This avoids SIMD-optimized functions having to sign-extend their > line size argument manually to be able to do pointer arithmetic. > --- ok if oracle is happy -- Vittorio ___

Re: [libav-devel] [PATCH] avdevice: Ignore timefilter test program

On Mon, Jun 13, 2016 at 11:25 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Mon, Jun 13, 2016 at 10:50:40AM -0400, Vittorio Giovara wrote: >> On Mon, Jun 13, 2016 at 8:54 AM, Diego Biurrun <di...@biurrun.de> wrote: >> > --- /dev/null >> > +++ b/libav

Re: [libav-devel] [PATCH] avdevice: Ignore timefilter test program

On Mon, Jun 13, 2016 at 8:54 AM, Diego Biurrun wrote: > --- > libavdevice/tests/.gitignore | 1 + > 1 file changed, 1 insertion(+) > create mode 100644 libavdevice/tests/.gitignore > > diff --git a/libavdevice/tests/.gitignore b/libavdevice/tests/.gitignore > new file mode

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

, enable this feature only for multiple avcC's. All other stsd types (such as hvc1 and hev1) need decoder-side changes, so they are left disabled for now. This is implemented only in non-fragmented MOVs. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- In this iteration: - d

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

On Fri, Jun 10, 2016 at 2:32 PM, Martin Storsjö <mar...@martin.st> 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ö <mar...@martin.st> wrote: >>

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

On Fri, Jun 10, 2016 at 1:23 PM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2016-06-10 16:52:39) >> On Fri, Jun 10, 2016 at 4:56 AM, Anton Khirnov <an...@khirnov.net> wrote: >> > Quoting Vittorio Giovara (2016-06-10 00:55:18) >> >&g

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

On Fri, Jun 10, 2016 at 4:56 AM, Anton Khirnov <an...@khirnov.net> 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 &g

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

On Fri, Jun 10, 2016 at 4:56 AM, Anton Khirnov <an...@khirnov.net> 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 >

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

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 a/libavformat/mov.c b/libavformat/mov.c index 9e2d0e2..9a7a0bc

[libav-devel] [PATCH 5/5] mov: Validate stsc indexes are within stsd bounds

--- Already approved. Vittorio libavformat/mov.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9a7a0bc..f09c614 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1954,6 +1954,8 @@ static int mov_read_stsc(MOVContext *c, AVIOContext

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

, enable this feature only for multiple avcC's. All other stsd types (such as hvc1 and hev1) need decoder-side changes, so they are left disabled for now. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Only modified to avoid caching a zero-sized extradata. Vittorio libav

[libav-devel] [PATCH 2/5] h264: Support AV_PKT_DATA_NEW_EXTRADATA

--- Applied Anton's change. Vittorio libavcodec/h264.c | 13 + 1 file changed, 13 insertions(+) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 224ba2f..2dba261 100644 --- a/libavcodec/h264.c +++ b/libavcodec/h264.c @@ -1004,6 +1004,8 @@ static int

[libav-devel] [PATCH 1/5] lavc: Document AV_PKT_DATA_NEW_EXTRADATA

--- Already approved. libavcodec/avcodec.h | 8 1 file changed, 8 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index f33b7e5..01a6286 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -1187,6 +1187,14 @@ typedef struct AVCPBProperties { */ enum

[libav-devel] [PATCH] avpacket: Error out when creating 0-sized side data

This mimics the behaviour of other av_*_new_side_data(). This is not caught by the malloc check, since AV_INPUT_BUFFER_PADDING_SIZE is always added to the allocated size. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- This is assuming that packet side data cannot be used

Re: [libav-devel] [PATCH] avpacket: Error out when creating 0-sized side data

On Thu, Jun 9, 2016 at 6:31 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > This mimics the behaviour of other av_*_new_side_data(). > This is not caught by the malloc check, since AV_INPUT_BUFFER_PADDING_SIZE > is always added to the allocated size. > > Signed-off

[libav-devel] [PATCH] avpacket: Error out when creating 0-sized side data

This mimics the behaviour of other av_*_new_side_data(). This is not caught by the malloc check, since AV_INPUT_BUFFER_PADDING_SIZE is always added to the allocated size. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- This is assuming that packet side data cannot be used

Re: [libav-devel] [PATCH 10/28] h264: factor out setting frame properties / side data

On Thu, Jun 9, 2016 at 4:29 AM, Anton Khirnov wrote: > Right now this code is mixed with selecting the next output frame. Move > it to a separate function called from h264_field_start(), which is a > more appropriate place for this. > --- ok -- Vittorio

Re: [libav-devel] [PATCH 2/3] h264: Support AV_PKT_DATA_NEW_EXTRADATA

On Thu, Jun 9, 2016 at 4:46 PM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2016-06-09 22:42:30) >> On Thu, Jun 9, 2016 at 4:36 PM, Anton Khirnov <an...@khirnov.net> wrote: >> > Quoting Vittorio Giovara (2016-06-02 02:00:03) >> >&

Re: [libav-devel] [PATCH 2/3] h264: Support AV_PKT_DATA_NEW_EXTRADATA

On Thu, Jun 9, 2016 at 4:36 PM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2016-06-02 02:00:03) >> --- >> I left the context extradata unchanged, and I process the new extradata >> directly. This fixed the crash at decoder_close. >> Vitto

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

On Thu, Jun 9, 2016 at 11:12 AM, Diego Biurrun wrote: > From: Luca Barbato > > --- > libavcodec/dvdec.c | 122 > ++--- > 1 file changed, 69 insertions(+), 53 deletions(-) > > diff --git a/libavcodec/dvdec.c

Re: [libav-devel] [PATCH 02/28] h264: move the block starting a new field out of slice_header_parse()

On Thu, Jun 9, 2016 at 4:29 AM, Anton Khirnov wrote: > There is no bitstream parsing in that block and messing with > decoder-global state is not something that belongs into header parsing. > > Nothing else in this function depends on the value of current_slice, > except for

Re: [libav-devel] [PATCH 01/28] h264: pass a H2645NAL to slice header decoding

On Thu, Jun 9, 2016 at 4:29 AM, Anton Khirnov wrote: > Replace the decoder-global nal_unit_type/nal_ref_idc variables with the > per-NAL ones. The decoder-global ones still cannot be removed because > they are used by hwaccels. > --- > libavcodec/h264.c | 2 +- >

Re: [libav-devel] [PATCH 15/28] h264: tighten the valid range for ref_frame_count

On Thu, Jun 9, 2016 at 4:54 AM, Diego Biurrun wrote: > On Thu, Jun 09, 2016 at 10:30:01AM +0200, Anton Khirnov wrote: >> This field (which the spec calls max_num_ref_frames) must be less or >> equal to MaxDpbFrames, which is at most 16. > > less than > > Diego lgtm with that

Re: [libav-devel] [PATCH 13/28] svq3: stop using H264Picture

On Thu, Jun 9, 2016 at 4:29 AM, Anton Khirnov wrote: > The SVQ3 decoder has been decoupled from the H.264 decoder, so it can > now use its own data type. > --- > libavcodec/svq3.c | 34 -- > 1 file changed, 24 insertions(+), 10 deletions(-)

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

On Mon, Jun 6, 2016 at 2:57 PM, Martin Storsjö <mar...@martin.st> 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 operations, switch buffers when the &

Re: [libav-devel] [PATCH] mov: Validate stsc indexes are within stsd bounds

On Mon, Jun 6, 2016 at 1:26 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 06/06/16 18:47, Vittorio Giovara wrote: >> --- >> libavformat/mov.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/libavformat/mov.c b/libavformat

[libav-devel] [PATCH] mov: Validate stsc indexes are within stsd bounds

--- libavformat/mov.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index fceef5b..385bd1d 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1952,6 +1952,8 @@ static int mov_read_stsc(MOVContext *c, AVIOContext *pb, MOVAtom atom)

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

On Fri, Jun 3, 2016 at 5:24 AM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 02/06/16 02:00, Vittorio Giovara wrote: >> 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 >

[libav-devel] [PATCH] Add MagicYUV decoder

From: Paul B Mahol <one...@gmail.com> Signed-off-by: Paul B Mahol <one...@gmail.com> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Applied Diego's comments. Vittorio Changelog | 1 + configure | 1 + doc/general.texi|

Re: [libav-devel] [PATCH 2/2] fate: Move texture-based decoder tests to a separate file

On Sat, Jun 4, 2016 at 4:55 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Fri, Jun 03, 2016 at 02:05:27PM -0400, Vittorio Giovara wrote: >> --- >> tests/Makefile | 1 + >> tests/fate/video.mak | 43 --- >> 2

Re: [libav-devel] [PATCH] Return the actual size of the data in AVPacket when decoding VP9 superframe

On Sat, Jun 4, 2016 at 7:28 AM, Luca Barbato wrote: > On 04/06/16 13:22, y...@dinauz.org wrote: >> From: typx >> >> According to avcodec.h, avcodec_decode_video2 should return the number of >> bytes used if a frame was decoded. >> The current implementation

[libav-devel] [PATCH 5/5] fate: Add tests for MagicYUV

--- Also these ones could maybe better be in magicyuv.mak? Vittorio tests/fate/video.mak | 19 +++ tests/ref/fate/magicyuv-rgb| 2 ++ tests/ref/fate/magicyuv-rgba | 2 ++ tests/ref/fate/magicyuv-y400i | 2 ++ tests/ref/fate/magicyuv-y420 | 2 ++

[libav-devel] [PATCH 3/5] fate: Move Duck Truemotion 1 and 2 tests to vpx.mak

--- Alternatively I can move them to duck.mak, but they seem closely related. Vittorio tests/fate/video.mak | 12 tests/fate/vpx.mak | 12 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/fate/video.mak b/tests/fate/video.mak index

[libav-devel] [PATCH 1/5] Add MagicYUV decoder

From: Paul B Mahol <one...@gmail.com> Signed-off-by: Paul B Mahol <one...@gmail.com> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Needs version bump. Vittorio Changelog | 1 + configure | 1 + doc/general.texi|

[libav-devel] [PATCH 4/5] fate: Add TrueMotion 2 RT tests

--- tests/fate/vpx.mak| 12 tests/ref/fate/truemotion2rt-high | 2 ++ tests/ref/fate/truemotion2rt-low | 2 ++ tests/ref/fate/truemotion2rt-mid | 2 ++ 4 files changed, 63 insertions(+) create mode 100644 tests/ref/fate/truemotion2rt-high create mode 100644

[libav-devel] [PATCH 2/5] Add TrueMotion 2.0 Real Time decoder

From: Paul B Mahol <one...@gmail.com> Signed-off-by: Paul B Mahol <one...@gmail.com> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Needs version bump. Vittorio Changelog | 1 + doc/general.texi | 2 + libavcodec/Mak

[libav-devel] [PATCH] fate: Move Canopus decoder tests to a separate file

--- Resend preserving the entries order. Vittorio tests/Makefile| 1 + tests/fate/canopus.mak| 34 ++ tests/fate/lossless-video.mak | 12 tests/fate/video.mak | 21 - 4 files changed, 35

[libav-devel] [PATCH 2/2] fate: Move texture-based decoder tests to a separate file

--- tests/Makefile | 1 + tests/fate/video.mak | 43 --- 2 files changed, 1 insertion(+), 43 deletions(-) diff --git a/tests/Makefile b/tests/Makefile index 36a3a72..5059176 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -108,6 +108,7 @@

[libav-devel] [PATCH 1/2] fate: Move Canopus decoder tests to a separate file

--- tests/Makefile| 1 + tests/fate/canopus.mak| 33 + tests/fate/lossless-video.mak | 12 tests/fate/video.mak | 21 - 4 files changed, 34 insertions(+), 33 deletions(-) create mode 100644

Re: [libav-devel] [PATCH] avconv: Do not copyextradata if not empty

Wrong title, "avconv: Do not copy extradata if source buffer is empty". On Fri, Jun 3, 2016 at 12:14 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > Fixes clang-usan runtime error "null pointer passed as argument 2, > which is declared to never b

[libav-devel] [PATCH] avconv: Do not copyextradata if not empty

Fixes clang-usan runtime error "null pointer passed as argument 2, which is declared to never be null" while streamcopying. --- avconv.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/avconv.c b/avconv.c index d863103..2e3bde9 100644 --- a/avconv.c +++

[libav-devel] [PATCH 1/3] lavc: Document AV_PKT_DATA_NEW_EXTRADATA

--- I removed the code example to avoid any possible confusion with extradata handling. Vittorio libavcodec/avcodec.h | 8 1 file changed, 8 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index f33b7e5..01a6286 100644 --- a/libavcodec/avcodec.h +++

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

, enable this feature only for multiple avcC's. All other stsd types (such as hvc1 and hev1) need decoder-side changes, so they are left disabled for now. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- I slightly simplified the stsc/stsd index tracking so that it's less waste

[libav-devel] [PATCH 2/3] h264: Support AV_PKT_DATA_NEW_EXTRADATA

--- I left the context extradata unchanged, and I process the new extradata directly. This fixed the crash at decoder_close. Vittorio libavcodec/h264.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 224ba2f..869b528 100644 ---

<    4   5   6   7   8   9   10   11   12   13   >