[FFmpeg-devel] [PATCH] Making process of uuid-xmp faster.

2016-11-08 Thread Chen Meng
--- libavformat/mov.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index f06de06..388cd1f 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -4533,13 +4533,20 @@ static int mov_read_uuid(MOVContext *c, AVIOContext

Re: [FFmpeg-devel] [PATCH v4] avformat/flvenc: add add_keyframe_index option

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 03:17:26PM +0800, Steven Liu wrote: [...] > > diff --git a/tests/ref/fate/flv-add_keyframe_index > > b/tests/ref/fate/flv-add_keyframe_index > > new file mode 100644 > > index 000..8e12244 > > --- /dev/null > > +++ b/tests/ref/fate/flv-add_keyframe_index > > @@ -0,0 +1,1

Re: [FFmpeg-devel] [PATCH v4] avformat/flvenc: add add_keyframe_index option

2016-11-08 Thread Steven Liu
2016-11-08 16:47 GMT+08:00 Michael Niedermayer : > On Tue, Nov 08, 2016 at 03:17:26PM +0800, Steven Liu wrote: > [...] > > > diff --git a/tests/ref/fate/flv-add_keyframe_index > > > b/tests/ref/fate/flv-add_keyframe_index > > > new file mode 100644 > > > index 000..8e12244 > > > --- /dev/null

Re: [FFmpeg-devel] [PATCH v4] avformat/flvenc: add add_keyframe_index option

2016-11-08 Thread Steven Liu
2016-11-08 16:51 GMT+08:00 Steven Liu : > > > 2016-11-08 16:47 GMT+08:00 Michael Niedermayer : > >> On Tue, Nov 08, 2016 at 03:17:26PM +0800, Steven Liu wrote: >> [...] >> > > diff --git a/tests/ref/fate/flv-add_keyframe_index >> > > b/tests/ref/fate/flv-add_keyframe_index >> > > new file mode 100

Re: [FFmpeg-devel] [PATCH v4] avformat/flvenc: add add_keyframe_index option

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 05:48:07PM +0800, Steven Liu wrote: > 2016-11-08 16:51 GMT+08:00 Steven Liu : > > > > > > > 2016-11-08 16:47 GMT+08:00 Michael Niedermayer : > > > >> On Tue, Nov 08, 2016 at 03:17:26PM +0800, Steven Liu wrote: > >> [...] > >> > > diff --git a/tests/ref/fate/flv-add_keyframe

[FFmpeg-devel] [PATCH]lavf/mpegtsenc: Automatically insert dump_extra for asp muxing

2016-11-08 Thread Carl Eugen Hoyos
Hi! Attached patch allows automatic insertion of dump_extra when remuxing mpeg4 asp. Please comment, Carl Eugen From 8ff75d4fa9127f3f02dde9408795f27c8d0e9c2c Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Tue, 8 Nov 2016 12:22:26 +0100 Subject: [PATCH] lavf/mpegtsenc: Automatically inser

[FFmpeg-devel] [PATCH v5] avformat/flvenc: add add_keyframe_index option

2016-11-08 Thread Steven Liu
Add keyframe index metadata Used to facilitate seeking; particularly for HTTP pseudo streaming. 1. read live streaming or file by sequence 2. if use add_keyframe_index option, add a mark flag at the position, use to insert new context at the last step. 3. add the keyframes *offset* and *time

Re: [FFmpeg-devel] [PATCH v4] avformat/flvenc: add add_keyframe_index option

2016-11-08 Thread Steven Liu
2016-11-08 19:07 GMT+08:00 Michael Niedermayer : > On Tue, Nov 08, 2016 at 05:48:07PM +0800, Steven Liu wrote: > > 2016-11-08 16:51 GMT+08:00 Steven Liu : > > > > > > > > > > > 2016-11-08 16:47 GMT+08:00 Michael Niedermayer >: > > > > > >> On Tue, Nov 08, 2016 at 03:17:26PM +0800, Steven Liu wrot

Re: [FFmpeg-devel] [PATCH]lavf/mpegtsenc: Automatically insert dump_extra for asp muxing

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 12:25:00PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch allows automatic insertion of dump_extra when > remuxing mpeg4 asp. > > Please comment, Carl Eugen > mpegtsenc.c |7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > 071687a7d12bb9b78df3601

Re: [FFmpeg-devel] [PATCH] Making process of uuid-xmp faster.

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 04:34:13PM +0800, Chen Meng wrote: > --- > libavformat/mov.c | 15 +++ > 1 file changed, 11 insertions(+), 4 deletions(-) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index f06de06..388cd1f 100644 > --- a/libavformat/mov.c > +++ b/libavformat/mov.c

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/hap: pass texture-compression destination as argument, not in context

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 12:21:52AM +, Tom Butterworth wrote: > This allows a subsequent change to compress directly into the output packet > when possible. > --- > libavcodec/hapenc.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/hapenc.c b/libavcod

[FFmpeg-devel] [PATCH] Add decoding support for the second generation IEEE 1857 video coding standard(AVS2) using uavs2d library(git clones https://github.com/nelvtpkusz/uavs2d.git uavs2d). Support AV

2016-11-08 Thread Zhenyu Wang
--- configure | 4 + libavcodec/Makefile| 1 + libavcodec/allcodecs.c | 2 + libavcodec/avcodec.h | 1 + libavcodec/cavs_parser.c | 9 ++ libavcodec/codec_desc.c| 7 ++ libavcodec/libuavs2d.c | 218 +++

Re: [FFmpeg-devel] [PATCH] libavcodec/tests: Added test for libavcodec/avpacket.c

2016-11-08 Thread Michael Niedermayer
On Mon, Nov 07, 2016 at 08:38:59PM -0800, Thomas Turner wrote: > Signed-off-by: Thomas Turner > --- > libavcodec/Makefile | 3 +- > libavcodec/tests/avpacket.c | 134 > > tests/fate/libavcodec.mak | 5 ++ > 3 files changed, 141 insertion

Re: [FFmpeg-devel] [PATCH] mov: Read multiple stsd from DV

2016-11-08 Thread Michael Niedermayer
On Mon, Nov 07, 2016 at 10:32:05PM -0500, Vittorio Giovara wrote: > Signed-off-by: Vittorio Giovara > --- > Sorry, I can't share the sample or add a fate for this. > Please CC. > Vittorio > > libavformat/mov.c | 2 ++ > 1 file changed, 2 insertions(+) applied thx [..] -- Michael GnuPG fi

Re: [FFmpeg-devel] [PATCH] Add decoding support for the second generation IEEE 1857 video coding standard(AVS2) using uavs2d library(git clones https://github.com/nelvtpkusz/uavs2d.git uavs2d). Suppor

2016-11-08 Thread Steven Liu
2016-11-08 20:57 GMT+08:00 Zhenyu Wang : > --- > configure | 4 + > libavcodec/Makefile| 1 + > libavcodec/allcodecs.c | 2 + > libavcodec/avcodec.h | 1 + > libavcodec/cavs_parser.c | 9 ++ > libavcodec/codec_desc.c| 7 ++ > libavcodec/libuavs

Re: [FFmpeg-devel] [PATCH] Add decoding support for the second generation IEEE 1857 video coding standard(AVS2) using uavs2d library(git clones https://github.com/nelvtpkusz/uavs2d.git uavs2d). Suppor

2016-11-08 Thread Steven Liu
2016-11-08 21:07 GMT+08:00 Steven Liu : > > > 2016-11-08 20:57 GMT+08:00 Zhenyu Wang : > https://ffmpeg.org/developer.html Keep the main commit message short with an extended description below. The commit message should have a short first line in the form of a ‘topic: short description’ as a he

Re: [FFmpeg-devel] [PATCH] Add decoding support for the second generation IEEE 1857 video coding standard(AVS2) using uavs2d library(git clones https://github.com/nelvtpkusz/uavs2d.git uavs2d). Suppor

2016-11-08 Thread Steven Liu
2016-11-08 20:57 GMT+08:00 Zhenyu Wang : > --- > configure | 4 + > libavcodec/Makefile| 1 + > libavcodec/allcodecs.c | 2 + > libavcodec/avcodec.h | 1 + > libavcodec/cavs_parser.c | 9 ++ > libavcodec/codec_desc.c| 7 ++ > libavcodec/libuavs

[FFmpeg-devel] [PATCH 1/2] avcodec/hap: pass texture-compression destination as argument, not in context

2016-11-08 Thread Tom Butterworth
This allows a subsequent change to compress directly into the output packet when possible. --- libavcodec/hapenc.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/libavcodec/hapenc.c b/libavcodec/hapenc.c index 076923b..79b6074 100644 --- a/libavcodec/hapenc.c +++

[FFmpeg-devel] [PATCH 2/2] avcodec/hap: add "compressor" option to Hap encoder to disable secondary compression

2016-11-08 Thread Tom Butterworth
The secondary compression in Hap is optional, this change exposes that option to the user as some use-cases favour higher bitrate files to reduce workload decoding. Adds "none" or "snappy" as options for "compressor". Selecting "none" disregards "chunks" option: chunking is only of benefit decompre

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/hap: pass texture-compression destination as argument, not in context

2016-11-08 Thread Tom Butterworth
On Tue, 8 Nov 2016 at 12:44 Michael Niedermayer wrote: ... > > -static void compress_texture(AVCodecContext *avctx, const AVFrame *f) > > +static void compress_texture(AVCodecContext *avctx, uint8_t *out, const > AVFrame *f) > > passing an output array without array size into a function raises a

Re: [FFmpeg-devel] [PATCH v5] avformat/flvenc: add add_keyframe_index option

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 07:43:49PM +0800, Steven Liu wrote: > Add keyframe index metadata > Used to facilitate seeking; particularly for HTTP pseudo streaming. > 1. read live streaming or file by sequence > 2. if use add_keyframe_index option, add a mark flag at the position, > use to insert

[FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-08 Thread Franklin Phillips
This patch is a fix for ticket #2833. Each subtitle segment is its own WebVTT file and has to demuxed individually. The timing of the subtitles are not perfect but it is the best I could do and it also does not take into account the X-TIMESTAMP-MAP header in the WebVTT files which is needed to

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/hap: pass texture-compression destination as argument, not in context

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 01:30:53PM +, Tom Butterworth wrote: > This allows a subsequent change to compress directly into the output packet > when possible. > --- > libavcodec/hapenc.c | 12 +--- > 1 file changed, 9 insertions(+), 3 deletions(-) LGTM thx [...] -- Michael GnuPG

Re: [FFmpeg-devel] [FFmpeg-cvslog] ffserver: get time_base from AVStream in print_stream_params()

2016-11-08 Thread James Almer
> ffmpeg | branch: master | Reynaldo H. Verdejo Pinochet osg.samsung.com> | Tue Nov 8 00:51:28 2016 -0800| > [1323349befd3f55bef2f9e46168f17a53f8f6d4a] | committer: Michael Niedermayer > > ffserver: get time_base from AVStream in print_stream_params() > > AVStream.codec is deprecated > > Signed-

Re: [FFmpeg-devel] [PATCH] Making process of uuid-xmp faster.

2016-11-08 Thread Chen Meng
Thx for reply.I've tried to add error code in return. 此致——孟辰(阏逢) 阿里云事业群-飞天事业部-视频服务Eml: mengchen.mc@alibaba-inc.comMob: +86-1316-257-5752 | Tel: 021-5255-9888.71048Adr: 上海市浦东新区杨高南路759号陆家嘴世纪金融广场2号楼15层 --From:Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] Making process of uuid-xmp faster.

2016-11-08 Thread Chen Meng
From baee22d71825128f74beb15122cd786f8d89f13f Mon Sep 17 00:00:00 2001 From: Chen Meng  Date: Tue, 8 Nov 2016 22:58:44 +0800 Subject: [PATCH] Take the error code in return. ---  libavformat/mov.c | 30 ++  1 file changed, 14 insertions(+), 16 deletions(-) diff --git a/l

Re: [FFmpeg-devel] [PATCH 2/3] lavfi/loudnorm: add an internal libebur128 library

2016-11-08 Thread Kyle Swanson
On Mon, Nov 7, 2016 at 6:00 PM, Marton Balint wrote: > > On Fri, 4 Nov 2016, Marton Balint wrote: > >> >> On Thu, 3 Nov 2016, Hendrik Leppkes wrote: >> >>> On Mon, Oct 17, 2016 at 5:20 PM, Moritz Barsnick >>> wrote: On Mon, Oct 17, 2016 at 17:09:15 +0200, wm4 wrote: > > Does thi

[FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Also a small cosmetic change to the avx2 idct16 version to make it explicit that one of the arguments to the write-out macros is unused for >=avx2 (it uses pmovzxbw instead of punpcklbw). --- libavcodec/x86/vp9dsp_init.c | 8 +- libavcodec/x86/vp9itxfm.asm | 188

Re: [FFmpeg-devel] [PATCH] avformat/matroskadec: fix DiscardPadding element parsing

2016-11-08 Thread James Almer
On 11/7/2016 9:00 PM, James Zern wrote: > On Sat, Nov 5, 2016 at 1:50 PM, James Almer wrote: >> If the value is negative then it means padding at the start of the packet >> instead of at the end. >> >> Based on a patch by Hendrik Leppkes. >> >> Signed-off-by: James Almer >> --- >> libavformat/ma

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/hap: pass texture-compression destination as argument, not in context

2016-11-08 Thread Tom Butterworth
thanks, pushed On Tue, 8 Nov 2016 at 14:14 Michael Niedermayer wrote: > On Tue, Nov 08, 2016 at 01:30:53PM +, Tom Butterworth wrote: > > This allows a subsequent change to compress directly into the output > packet when possible. > > --- > > libavcodec/hapenc.c | 12 +--- > > 1 file

Re: [FFmpeg-devel] [PATCH]lavf/mpegtsenc: Automatically insert dump_extra for asp muxing

2016-11-08 Thread Fredrik Persson
I will try this, thank you very much! /Fredrik Persson On Tue, Nov 8, 2016 at 1:05 PM, Michael Niedermayer wrote: > On Tue, Nov 08, 2016 at 12:25:00PM +0100, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch allows automatic insertion of dump_extra when > > remuxing mpeg4 asp. > > > > Plea

Re: [FFmpeg-devel] [PATCH] Making process of uuid-xmp faster.

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 11:04:35PM +0800, Chen Meng wrote: > From baee22d71825128f74beb15122cd786f8d89f13f Mon Sep 17 00:00:00 2001 > From: Chen Meng  > Date: Tue, 8 Nov 2016 22:58:44 +0800 > Subject: [PATCH] Take the error code in return. git refuses to apply this Applying: Making process of uui

Re: [FFmpeg-devel] [PATCH] mpegts: prevent division by zero

2016-11-08 Thread Andreas Cadhalpun
On 08.11.2016 00:54, Michael Niedermayer wrote: > On Mon, Nov 07, 2016 at 11:49:52PM +0100, Andreas Cadhalpun wrote: >> Signed-off-by: Andreas Cadhalpun >> --- >> libavformat/mpegts.c | 4 >> 1 file changed, 4 insertions(+) >> >> diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c >> in

Re: [FFmpeg-devel] [FFmpeg-cvslog] ffserver: get time_base from AVStream in print_stream_params()

2016-11-08 Thread Reynaldo H. Verdejo Pinochet
On 11/08/2016 06:29 AM, James Almer wrote: [..] Right now it just returns st->codec->time_base, but after st->codec is deprecated it will be adapted to use st->internal->avctx->time_base. Perfect! Will do. Thanks for the hint James. Bests, -- Reynaldo H. Verdejo Pinochet Open Source Grou

Re: [FFmpeg-devel] [PATCH] matroskadec: fix NULL pointer dereference in webm_dash_manifest_read_header

2016-11-08 Thread Andreas Cadhalpun
On 08.11.2016 01:12, Michael Niedermayer wrote: > On Tue, Nov 08, 2016 at 12:44:56AM +0100, Andreas Cadhalpun wrote: >> The code assumes that s->streams[0] is valid. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavformat/matroskadec.c | 5 + >> 1 file changed, 5 insertions(+) > > shoul

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 11:22:31AM -0500, Ronald S. Bultje wrote: > Also a small cosmetic change to the avx2 idct16 version to make it > explicit that one of the arguments to the write-out macros is unused > for >=avx2 (it uses pmovzxbw instead of punpcklbw). > --- > libavcodec/x86/vp9dsp_init.c |

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Hi, On Tue, Nov 8, 2016 at 2:39 PM, Michael Niedermayer wrote: > On Tue, Nov 08, 2016 at 11:22:31AM -0500, Ronald S. Bultje wrote: > > Also a small cosmetic change to the avx2 idct16 version to make it > > explicit that one of the arguments to the write-out macros is unused > > for >=avx2 (it us

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Hi, On Tue, Nov 8, 2016 at 2:43 PM, Ronald S. Bultje wrote: > Hi, > > On Tue, Nov 8, 2016 at 2:39 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > >> On Tue, Nov 08, 2016 at 11:22:31AM -0500, Ronald S. Bultje wrote: >> > Also a small cosmetic change to the avx2 idct16 version to make

Re: [FFmpeg-devel] [PATCH] mpegts: prevent division by zero

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 07:47:02PM +0100, Andreas Cadhalpun wrote: > On 08.11.2016 00:54, Michael Niedermayer wrote: > > On Mon, Nov 07, 2016 at 11:49:52PM +0100, Andreas Cadhalpun wrote: > >> Signed-off-by: Andreas Cadhalpun > >> --- > >> libavformat/mpegts.c | 4 > >> 1 file changed, 4 ins

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/hap: add "compressor" option to Hap encoder to disable secondary compression

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 01:30:54PM +, Tom Butterworth wrote: > The secondary compression in Hap is optional, this change exposes that option > to > the user as some use-cases favour higher bitrate files to reduce workload > decoding. > Adds "none" or "snappy" as options for "compressor". Selec

Re: [FFmpeg-devel] [PATCH] mpegts: prevent division by zero

2016-11-08 Thread Andreas Cadhalpun
On 08.11.2016 21:09, Michael Niedermayer wrote: > On Tue, Nov 08, 2016 at 07:47:02PM +0100, Andreas Cadhalpun wrote: >> On 08.11.2016 00:54, Michael Niedermayer wrote: >>> On Mon, Nov 07, 2016 at 11:49:52PM +0100, Andreas Cadhalpun wrote: Signed-off-by: Andreas Cadhalpun --- libavf

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/hap: add "compressor" option to Hap encoder to disable secondary compression

2016-11-08 Thread Martin Vignali
> > Adds "none" or "snappy" as options for "compressor". Selecting "none" > disregards > > "chunks" option: chunking is only of benefit decompressing Snappy. > > --- Maybe can be useful, to have a log, if chunk_count != 1 and compression is none, to show to the user, that the chunk count option w

[FFmpeg-devel] [PATCH] feature: xcbgrab single window

2016-11-08 Thread Ing . Radomír Polách
Allows to grab only a single window by using syntax: ffmpeg -f x11grab -r 15 -i ":0/0x0580001b/parent2" ... The syntax for input is "display/grab_window_id/focus_window_id". If focus_window_id is omitted it is set as grab_window_id. There are special constants for focus_window_id: - this: grab_wind

Re: [FFmpeg-devel] [PATCH] mpegts: prevent division by zero

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 09:38:49PM +0100, Andreas Cadhalpun wrote: > On 08.11.2016 21:09, Michael Niedermayer wrote: > > On Tue, Nov 08, 2016 at 07:47:02PM +0100, Andreas Cadhalpun wrote: > >> On 08.11.2016 00:54, Michael Niedermayer wrote: > >>> On Mon, Nov 07, 2016 at 11:49:52PM +0100, Andreas Ca

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/hap: add "compressor" option to Hap encoder to disable secondary compression

2016-11-08 Thread Tom Butterworth
Thanks for your comments Martin - > > Adds "none" or "snappy" as options for "compressor". Selecting "none" > > disregards > > > "chunks" option: chunking is only of benefit decompressing Snappy. > > > --- > > > Maybe can be useful, to have a log, > if chunk_count != 1 and compression is none, to

[FFmpeg-devel] [PATCH] cmdutils: remove duplicate windows.h include

2016-11-08 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- cmdutils.c | 1 - 1 file changed, 1 deletion(-) diff --git a/cmdutils.c b/cmdutils.c index 44fe64c..2373dbf 100644 --- a/cmdutils.c +++ b/cmdutils.c @@ -231,7 +231,6 @@ static const OptionDef *find_option(const OptionDef *po, const char *name) * by defau

Re: [FFmpeg-devel] [PATCH] mpegts: prevent division by zero

2016-11-08 Thread Andreas Cadhalpun
On 08.11.2016 22:12, Michael Niedermayer wrote: > On Tue, Nov 08, 2016 at 09:38:49PM +0100, Andreas Cadhalpun wrote: >> On 08.11.2016 21:09, Michael Niedermayer wrote: >>> On Tue, Nov 08, 2016 at 07:47:02PM +0100, Andreas Cadhalpun wrote: On 08.11.2016 00:54, Michael Niedermayer wrote: > O

[FFmpeg-devel] [PATCH] dvbsubdec: fix division by zero in compute_default_clut

2016-11-08 Thread Andreas Cadhalpun
Since the loop goes down to i = 0 the number of iterations (variable count) is the original i, not i - 1. In the case of i originally being 1, this miscalculation resulted in a division by zero. This problem was introduced in commit 4b90dcb8493552c17a811c8b1e6538dae4061f9d. Signed-off-by: Andreas

[FFmpeg-devel] [PATCH 3/3] fate: Add h264 and hevc extradata reload tests

2016-11-08 Thread Vittorio Giovara
--- Samples available at - https://www.dropbox.com/s/gkswnubzy6e6rgg/h264-extradata-reload-multi-stsd.mov?dl=0 - https://www.dropbox.com/s/djfvoqgu2z9zrqu/hevc-extradata-reload-multi-stsd.mov?dl=0 They need to be renamed without the codec tag and placed in the appropriate directory in fate. Pleas

[FFmpeg-devel] [PATCH 1/3] hevc: Allow parsing external extradata buffers

2016-11-08 Thread Vittorio Giovara
--- As mentioned in the discussion. Please CC. Vittorio libavcodec/hevc.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c index 29e0d49..02fd606 100644 --- a/libavcodec/hevc.c +++ b/libavcodec/hevc.c @@ -2973,17 +2973,15 @@ st

[FFmpeg-devel] [PATCH 2/3] hevc: Support extradata changes

2016-11-08 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- Applied review. Please CC. Vittorio libavcodec/hevc.c | 10 ++ libavformat/mov.c | 4 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c index 02fd606..4417f79 100644 --- a/libavcodec/hevc.c +++

Re: [FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata.

2016-11-08 Thread Moritz Barsnick
On Mon, Nov 07, 2016 at 16:30:02 -0800, Zhenni Huang wrote: > -avio_rb32(pb); /* vendor */ > +video_vendor_id = avio_rb32(pb); /* vendor */ > avio_rb32(pb); /* temporal quality */ > avio_rb32(pb); /* spatial quality */ > > +/* set video_vendor_id */ > +video_vendor_id_bu

[FFmpeg-devel] [PATCH] icodec: correctly check avio_read return value

2016-11-08 Thread Andreas Cadhalpun
It can read less than the requested amount, in which case buf contains uninitialized data, causing problems like segmentation faults later on. Signed-off-by: Andreas Cadhalpun --- libavformat/icodec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/icodec.c b/

[FFmpeg-devel] [PATCH 1/2] icodec: fix leaking pkt on error

2016-11-08 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- libavformat/icodec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/icodec.c b/libavformat/icodec.c index aad1416..becbc0f 100644 --- a/libavformat/icodec.c +++ b/libavformat/icodec.c @@ -174,8 +174,10 @@ static int read_pac

[FFmpeg-devel] [PATCH 2/2] iocodec: add ico_read_close to fix leaking ico->images

2016-11-08 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- libavformat/icodec.c | 8 1 file changed, 8 insertions(+) diff --git a/libavformat/icodec.c b/libavformat/icodec.c index becbc0f..1d8e383 100644 --- a/libavformat/icodec.c +++ b/libavformat/icodec.c @@ -199,6 +199,13 @@ static int read_packet(AVForma

Re: [FFmpeg-devel] [PATCH 2/2] iocodec: add ico_read_close to fix leaking ico->images

2016-11-08 Thread Moritz Barsnick
On Wed, Nov 09, 2016 at 00:00:09 +0100, Andreas Cadhalpun wrote: > [PATCH 2/2] iocodec: add ico_read_close to fix leaking ico->images ^ icodec M. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffm

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread compn
On Tue, 8 Nov 2016 14:44:32 -0500 "Ronald S. Bultje" wrote: > >> x86-32 linux gets stomach ache from this: > >> > >> YASMlibavcodec/x86/vp9itxfm.o > >> src/libavcodec/x86/vp9itxfm.asm:2150: error: instruction expected > >> after label > >> src/libavcodec/x86/vp9itxfm.asm:2151: error: instruct

Re: [FFmpeg-devel] [PATCH 2/2] iocodec: add ico_read_close to fix leaking ico->images

2016-11-08 Thread Andreas Cadhalpun
On 09.11.2016 00:28, Moritz Barsnick wrote: > On Wed, Nov 09, 2016 at 00:00:09 +0100, Andreas Cadhalpun wrote: >> [PATCH 2/2] iocodec: add ico_read_close to fix leaking ico->images > ^ icodec Fixed locally. Best regards, Andreas ___ ffm

[FFmpeg-devel] [PATCH] escape124: reject codebook size 0

2016-11-08 Thread Andreas Cadhalpun
It causes a cb_depth of 32, leading to assertion failures in get_bits. Signed-off-by: Andreas Cadhalpun --- libavcodec/escape124.c | 4 1 file changed, 4 insertions(+) diff --git a/libavcodec/escape124.c b/libavcodec/escape124.c index b872b3a..c3174ce 100644 --- a/libavcodec/escape124.c ++

[FFmpeg-devel] [PATCH] pnmdec: make sure v is capped by maxval

2016-11-08 Thread Andreas Cadhalpun
Otherwise put_bits can be called with a value that doesn't fit in the sample_len, causing an assertion failure. --- libavcodec/pnmdec.c | 4 1 file changed, 4 insertions(+) diff --git a/libavcodec/pnmdec.c b/libavcodec/pnmdec.c index ca97cc3..0381ea6 100644 --- a/libavcodec/pnmdec.c +++ b/li

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Hi, On Tue, Nov 8, 2016 at 6:33 PM, compn wrote: > On Tue, 8 Nov 2016 14:44:32 -0500 > "Ronald S. Bultje" wrote: > > > >> x86-32 linux gets stomach ache from this: > > >> > > >> YASMlibavcodec/x86/vp9itxfm.o > > >> src/libavcodec/x86/vp9itxfm.asm:2150: error: instruction expected > > >> aft

Re: [FFmpeg-devel] [PATCH 3/3] fate: Add h264 and hevc extradata reload tests

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 05:03:28PM -0500, Vittorio Giovara wrote: > --- > Samples available at > - > https://www.dropbox.com/s/gkswnubzy6e6rgg/h264-extradata-reload-multi-stsd.mov?dl=0 > - > https://www.dropbox.com/s/djfvoqgu2z9zrqu/hevc-extradata-reload-multi-stsd.mov?dl=0 > They need to be rena

[FFmpeg-devel] [PATCH] MAINTAINERS: Add myself to flvenc

2016-11-08 Thread Steven Liu
Signed-off-by: Steven Liu --- MAINTAINERS |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 43ccec7..7bb9cc4 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -396,7 +396,8 @@ Muxers/Demuxers: epafdec.c Paul B Ma

Re: [FFmpeg-devel] [PATCH] dvbsubdec: fix division by zero in compute_default_clut

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 10:36:01PM +0100, Andreas Cadhalpun wrote: > Since the loop goes down to i = 0 the number of iterations (variable > count) is the original i, not i - 1. > In the case of i originally being 1, this miscalculation resulted in > a division by zero. > > This problem was introdu

Re: [FFmpeg-devel] [PATCH] icodec: correctly check avio_read return value

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 11:36:58PM +0100, Andreas Cadhalpun wrote: > It can read less than the requested amount, in which case buf contains > uninitialized data, causing problems like segmentation faults later on. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/icodec.c | 4 ++-- > 1 fi

Re: [FFmpeg-devel] [PATCH] Making process of uuid-xmp faster.

2016-11-08 Thread Chen Meng
Sorry for that, I've taken code back into up-date master branch again.---  libavformat/mov.c | 34 --  1 file changed, 20 insertions(+), 14 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index ca978c2..c358d17 100644 --- a/libavformat/mov.c +++ b/lib

Re: [FFmpeg-devel] [PATCH 3/3] fate: Add h264 and hevc extradata reload tests

2016-11-08 Thread Vittorio Giovara
On Tue, Nov 8, 2016 at 7:15 PM, Michael Niedermayer wrote: > On Tue, Nov 08, 2016 at 05:03:28PM -0500, Vittorio Giovara wrote: >> --- >> Samples available at >> - >> https://www.dropbox.com/s/gkswnubzy6e6rgg/h264-extradata-reload-multi-stsd.mov?dl=0 >> - >> https://www.dropbox.com/s/djfvoqgu2z9z

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/hap: add "compressor" option to Hap encoder to disable secondary compression

2016-11-08 Thread Martin Vignali
> > > > Maybe can be useful, to have a log, > > if chunk_count != 1 and compression is none, to show to the user, that > the > > chunk count option will be override by 1. > > > > The existing info log in hap_init() will print eg "6 chunks requested but 1 > used." as is already done when the user's