Re: [FFmpeg-devel] GSoC '15 Introduction

2015-03-09 Thread Roger Pack
On 3/8/15, compn te...@mi.rr.com wrote: On Sat, 7 Mar 2015 20:24:16 +0200 Ilinca Andrei andrei.ilinc...@gmail.com wrote: Hello! welcome! *Why directshow digital video capture ?* Simply because It sounds really interesting and entertaining. I have some experience

Re: [FFmpeg-devel] [PATCH 5/8] ffmdec: break infinite resync loop

2015-03-09 Thread Andreas Cadhalpun
On 09.03.2015 13:59, Michael Niedermayer wrote: i was thinking more about limiting the backward seek before resync to the last resync position +1 if there was a previous resync so that resync which moves forward could not end before. This would avoid the failure and allow the demuxer to

Re: [FFmpeg-devel] [PATCH] avcodec/options_table: remove extradata_size from the AVOptions table

2015-03-09 Thread Andreas Cadhalpun
On 09.03.2015 03:42, Michael Niedermayer wrote: allowing access to the size but not the extradata itself is not useful and could lead to potential problems if writing happens through this field Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/options_table.h |1 - 1

Re: [FFmpeg-devel] [PATCH 7/8] ffmdec: don't return AVERROR(EAGAIN) from ffm_is_avail_data

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 12:33:38PM +0100, Andreas Cadhalpun wrote: On 09.03.2015 03:56, Michael Niedermayer wrote: On Mon, Mar 09, 2015 at 12:05:01AM +0100, Andreas Cadhalpun wrote: Hi, if AVERROR(EAGAIN) is returned from ffm_is_avail_data it always causes an infinite EAGAIN loop. it

Re: [FFmpeg-devel] [PATCH] avformat/srtenc: avoid segmentation fault when transcoding from SSA to SRT

2015-03-09 Thread Gilles Chanteperdrix
On Sun, Mar 08, 2015 at 10:57:54AM +0100, Nicolas George wrote: L'octidi 18 ventôse, an CCXXIII, Clement Boesch a écrit : Try adding tags with no text maybe. You may also try ASS drawing mode, but FFmpeg probably doesn't do cray stuff about it. I tried various combinations, including empty

Re: [FFmpeg-devel] [PATCH 7/8] ffmdec: don't return AVERROR(EAGAIN) from ffm_is_avail_data

2015-03-09 Thread Andreas Cadhalpun
On 09.03.2015 03:56, Michael Niedermayer wrote: On Mon, Mar 09, 2015 at 12:05:01AM +0100, Andreas Cadhalpun wrote: Hi, if AVERROR(EAGAIN) is returned from ffm_is_avail_data it always causes an infinite EAGAIN loop. it should only loop until more data is written into the ffm file (this of

Re: [FFmpeg-devel] [PATCH 1/8] ffmdec: initialize f_cprv, f_stvi and f_stau

2015-03-09 Thread Andreas Cadhalpun
On 09.03.2015 13:20, Lukasz Marek wrote: BTW, did you produced this malformed file using ffmpeg tools or just prevent theoretical case? I fuzzed a file created by ffmpeg. Best regards, Andreas ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-09 Thread Reimar Döffinger
On 9 March 2015 13:34:24 CET, Hendrik Leppkes h.lepp...@gmail.com wrote: On Mon, Mar 9, 2015 at 7:38 AM, Rainer Hochecker fernetme...@online.de wrote: Reimar Döffinger Reimar.Doeffinger at gmx.de writes: Any reason to believe this patch causes it? Because I can't see how it would. Maybe it's

Re: [FFmpeg-devel] [PATCH 5/8] ffmdec: break infinite resync loop

2015-03-09 Thread Andreas Cadhalpun
On 09.03.2015 03:13, Michael Niedermayer wrote: On Mon, Mar 09, 2015 at 12:04:13AM +0100, Andreas Cadhalpun wrote: Hi, some broken files can lead to an endless resync loop, which is avoided by attached patch. Best regards, Andreas ffmdec.c | 10 +- 1 file changed, 9

Re: [FFmpeg-devel] [PATCH 1/8] ffmdec: initialize f_cprv, f_stvi and f_stau

2015-03-09 Thread Lukasz Marek
On 9 March 2015 at 12:41, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: On 09.03.2015 10:53, Lukasz Marek wrote: In fact this is a bit wrong. COMM is guaranteed unless malformed file is parsed. These variables are dedicated to detect doubled sections. This patch allows them to

Re: [FFmpeg-devel] [PATCH 5/8] ffmdec: break infinite resync loop

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 12:17:38PM +0100, Andreas Cadhalpun wrote: On 09.03.2015 03:13, Michael Niedermayer wrote: On Mon, Mar 09, 2015 at 12:04:13AM +0100, Andreas Cadhalpun wrote: Hi, some broken files can lead to an endless resync loop, which is avoided by attached patch. Best

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-09 Thread Hendrik Leppkes
On Mon, Mar 9, 2015 at 7:38 AM, Rainer Hochecker fernetme...@online.de wrote: Reimar Döffinger Reimar.Doeffinger at gmx.de writes: Any reason to believe this patch causes it? Because I can't see how it would. Maybe it's just a bug with DXVA and multithreading in the HEVC code? Can you

Re: [FFmpeg-devel] [PATCH 8/8] ffmdec: don't seek back at EOF

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 12:27:15PM +0100, Andreas Cadhalpun wrote: On 09.03.2015 03:46, Michael Niedermayer wrote: On Mon, Mar 09, 2015 at 12:05:31AM +0100, Andreas Cadhalpun wrote: Hi, I'm not sure what the purpose of this avio_seek was, but it can result in an endless loop. Maybe it

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-09 Thread Hendrik Leppkes
On Mon, Mar 9, 2015 at 1:50 PM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 9 March 2015 13:34:24 CET, Hendrik Leppkes h.lepp...@gmail.com wrote: On Mon, Mar 9, 2015 at 7:38 AM, Rainer Hochecker fernetme...@online.de wrote: Reimar Döffinger Reimar.Doeffinger at gmx.de writes: Any

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 04:53:52PM +0100, Michael Niedermayer wrote: On Mon, Mar 09, 2015 at 04:41:05PM +0100, Michael Niedermayer wrote: On Sun, Mar 08, 2015 at 11:07:33PM -0300, Claudio Freire wrote: On Sun, Mar 8, 2015 at 11:06 PM, Claudio Freire klaussfre...@gmail.com wrote: On

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-09 Thread Hendrik Leppkes
On Mon, Mar 9, 2015 at 3:28 PM, Hendrik Leppkes h.lepp...@gmail.com wrote: On Mon, Mar 9, 2015 at 1:50 PM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 9 March 2015 13:34:24 CET, Hendrik Leppkes h.lepp...@gmail.com wrote: On Mon, Mar 9, 2015 at 7:38 AM, Rainer Hochecker

Re: [FFmpeg-devel] [PATCH 5/8] ffmdec: break infinite resync loop

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 03:14:50PM +0100, Andreas Cadhalpun wrote: On 09.03.2015 13:59, Michael Niedermayer wrote: i was thinking more about limiting the backward seek before resync to the last resync position +1 if there was a previous resync so that resync which moves forward could not end

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Michael Niedermayer
On Sun, Mar 08, 2015 at 11:07:33PM -0300, Claudio Freire wrote: On Sun, Mar 8, 2015 at 11:06 PM, Claudio Freire klaussfre...@gmail.com wrote: On Sun, Mar 8, 2015 at 8:23 AM, Michael Niedermayer michae...@gmx.at wrote: i think its better to leave fuzz at a small value otherwise we would

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Claudio Freire
On Mon, Mar 9, 2015 at 12:41 PM, Michael Niedermayer michae...@gmx.at wrote: On Sun, Mar 08, 2015 at 11:07:33PM -0300, Claudio Freire wrote: On Sun, Mar 8, 2015 at 11:06 PM, Claudio Freire klaussfre...@gmail.com wrote: On Sun, Mar 8, 2015 at 8:23 AM, Michael Niedermayer michae...@gmx.at

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 12:55:43PM -0300, Claudio Freire wrote: On Mon, Mar 9, 2015 at 12:41 PM, Michael Niedermayer michae...@gmx.at wrote: On Sun, Mar 08, 2015 at 11:07:33PM -0300, Claudio Freire wrote: On Sun, Mar 8, 2015 at 11:06 PM, Claudio Freire klaussfre...@gmail.com wrote: On

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 04:41:05PM +0100, Michael Niedermayer wrote: On Sun, Mar 08, 2015 at 11:07:33PM -0300, Claudio Freire wrote: On Sun, Mar 8, 2015 at 11:06 PM, Claudio Freire klaussfre...@gmail.com wrote: On Sun, Mar 8, 2015 at 8:23 AM, Michael Niedermayer michae...@gmx.at

Re: [FFmpeg-devel] [PATCH] avcodec/options_table: remove extradata_size from the AVOptions table

2015-03-09 Thread Lukasz Marek
On 9 March 2015 at 14:28, Michael Niedermayer michae...@gmx.at wrote: On Mon, Mar 09, 2015 at 01:27:58PM +0100, Lukasz Marek wrote: On 9 March 2015 at 12:19, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: On 09.03.2015 03:42, Michael Niedermayer wrote: allowing access

Re: [FFmpeg-devel] [PATCH] lavfi: Add support to process_command in vf_eq.c

2015-03-09 Thread arwa arif
I was going through the code, and I realized that I have made a mistake. I have corrected the code, and attached the corresponding patch. From 2ebd299b55a34914d5549f21d264e8cb7f5f605d Mon Sep 17 00:00:00 2001 From: Arwa Arif arwaarif1...@gmail.com Date: Mon, 9 Mar 2015 19:50:32 +0530 Subject:

Re: [FFmpeg-devel] GSoC '15 Introduction

2015-03-09 Thread Ilinca Andrei
Thank you very much for your kind replies and advices compn, Timo and Roger. I am at the very beginning of the journey , I've never worked with Dshow before but I'm very excited to learn and develop. I will look through all the materials given and start doing stuff as soon as possible. On Mon,

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Michael Niedermayer
On Sun, Mar 08, 2015 at 11:07:33PM -0300, Claudio Freire wrote: On Sun, Mar 8, 2015 at 11:06 PM, Claudio Freire klaussfre...@gmail.com wrote: On Sun, Mar 8, 2015 at 8:23 AM, Michael Niedermayer michae...@gmx.at wrote: i think its better to leave fuzz at a small value otherwise we would

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Claudio Freire
On Mon, Mar 9, 2015 at 1:05 PM, Michael Niedermayer michae...@gmx.at wrote: Head is still using a butterworth filter for the lowpass IIRC. That could very well be both platform and sample rate specific. Sadly, I cannot test mips, but commenting the lowpass could yield useful results. what

Re: [FFmpeg-devel] [PATCH] avcodec/options_table: remove extradata_size from the AVOptions table

2015-03-09 Thread Nicolas George
Le nonidi 19 ventôse, an CCXXIII, Michael Niedermayer a écrit : you miss the problem, this is a demuxer side problem, a attacker can at least crash your application no muxer side change can fix this, the attacker has his own self written muxer that produces a mallicious bitstream (Unrelated

Re: [FFmpeg-devel] [PATCH] avcodec/options_table: remove extradata_size from the AVOptions table

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 03:42:00AM +0100, Michael Niedermayer wrote: allowing access to the size but not the extradata itself is not useful and could lead to potential problems if writing happens through this field Signed-off-by: Michael Niedermayer michae...@gmx.at ---

Re: [FFmpeg-devel] [PATCH] avcodec/hevc_parser: use avpriv_find_start_code in hevc_split()

2015-03-09 Thread zhaoxiu.zeng
在 2015/3/8 20:14, Michael Niedermayer 写道: On Sat, Mar 07, 2015 at 11:47:08PM +0800, zhaoxiu.zeng wrote: From ab12e3081ba987c2e05d819be97cde96952f1c2a Mon Sep 17 00:00:00 2001 From: Zeng Zhaoxiu zhaoxiu.z...@gmail.com Date: Sat, 7 Mar 2015 23:29:46 +0800 Subject: [PATCH 1/1]

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-09 Thread Reimar Döffinger
On 9 March 2015 15:28:48 CET, Hendrik Leppkes h.lepp...@gmail.com wrote: On Mon, Mar 9, 2015 at 1:50 PM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 9 March 2015 13:34:24 CET, Hendrik Leppkes h.lepp...@gmail.com wrote: On Mon, Mar 9, 2015 at 7:38 AM, Rainer Hochecker fernetme...@online.de

Re: [FFmpeg-devel] [PATCH] avcodec/options_table: remove extradata_size from the AVOptions table

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 05:33:46PM +0100, Nicolas George wrote: Le nonidi 19 ventôse, an CCXXIII, Michael Niedermayer a écrit : you miss the problem, this is a demuxer side problem, a attacker can at least crash your application no muxer side change can fix this, the attacker has his own

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 01:31:26PM -0300, Claudio Freire wrote: On Mon, Mar 9, 2015 at 1:26 PM, Michael Niedermayer michae...@gmx.at wrote: doesnt help but this does: -fate-aac-s7350-encode: CMD = enc_dec_pcm adts wav s16le $(REF) -strict -2 -c:a aac -b:a 256k +fate-aac-s7350-encode:

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Claudio Freire
On Mon, Mar 9, 2015 at 1:26 PM, Michael Niedermayer michae...@gmx.at wrote: doesnt help but this does: -fate-aac-s7350-encode: CMD = enc_dec_pcm adts wav s16le $(REF) -strict -2 -c:a aac -b:a 256k +fate-aac-s7350-encode: CMD = enc_dec_pcm adts wav s16le $(REF) -strict -2 -c:a aac -b:a 88k

Re: [FFmpeg-devel] [PATCH] avcodec/hevc_parser: use avpriv_find_start_code in hevc_split()

2015-03-09 Thread Michael Niedermayer
On Tue, Mar 10, 2015 at 12:57:57AM +0800, zhaoxiu.zeng wrote: 在 2015/3/8 20:14, Michael Niedermayer 写道: On Sat, Mar 07, 2015 at 11:47:08PM +0800, zhaoxiu.zeng wrote: From ab12e3081ba987c2e05d819be97cde96952f1c2a Mon Sep 17 00:00:00 2001 From: Zeng Zhaoxiu zhaoxiu.z...@gmail.com Date: Sat,

[FFmpeg-devel] Patch proposal : filter ebur128 : add filter name before each line of summary print

2015-03-09 Thread Martin Vignali
In attach a patch proposal, in order to make easier parse result of ebur128 filter by external scripts command line for testing : ffmpeg -i aFile.wav -filter ebur128=peak=true+sample -f null - Before patch, summary look like this : [Parsed_ebur128_0 @ 0x7fe8b0d0] Summary: Integrated

[FFmpeg-devel] [PATCH] rtpenc_mpegts: set rtp_ctx to NULL on avformat_write_header failure

2015-03-09 Thread Andreas Cadhalpun
Hi, trying to encode e.g. codec bmp with format rtp_mpegts crashes, because the avformat_write_header failure isn't handled correctly. Attached is a patch fixing this. Best regards, Andreas From c0ba3afc828bc6255a1bcf0feb34af3caa892572 Mon Sep 17 00:00:00 2001 From: Andreas Cadhalpun

Re: [FFmpeg-devel] [PATCH] avcodec/wmalossless: use av_clip_intp2

2015-03-09 Thread Clément Bœsch
On Mon, Mar 09, 2015 at 10:01:20PM +0100, Reimar Döffinger wrote: On Sun, Mar 08, 2015 at 10:32:03PM +0100, Michael Niedermayer wrote: On Sat, Mar 07, 2015 at 11:44:10PM +0800, zhaoxiu.zeng wrote: From 47c997fa0623ab94a7a93b2d2e4cc4fa64c85d5f Mon Sep 17 00:00:00 2001 From: Zeng Zhaoxiu

Re: [FFmpeg-devel] [PATCH] asfenc: fix leaking asf-index_ptr on error

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 07:37:56PM +0100, Andreas Cadhalpun wrote: Hi, asfenc leaks asf-index_ptr if asf_write_header1 fails. Attached patch fixes this. Best regards, Andreas asfenc.c |1 + 1 file changed, 1 insertion(+) bd66fe12def4c8cd582c02d65eb85caa8306ceb7

Re: [FFmpeg-devel] [PATCH] avfilter/colormatrix:add slice threading

2015-03-09 Thread Michael Niedermayer
Hi On Mon, Mar 09, 2015 at 10:56:18AM -0700, Yayoi wrote: --- libavfilter/vf_colormatrix.c | 146 --- 1 file changed, 95 insertions(+), 51 deletions(-) [...] @@ -372,12 +402,26 @@ static int filter_frame(AVFilterLink *link, AVFrame *in)

Re: [FFmpeg-devel] [PATCH] pthread: Fix?ff_thread_get_formatissues?when called outside frame decode.

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 06:38:18AM +, Rainer Hochecker wrote: Reimar Döffinger Reimar.Doeffinger at gmx.de writes: Any reason to believe this patch causes it? Because I can't see how it would. Maybe it's just a bug with DXVA and multithreading in the HEVC code? Can you provide

Re: [FFmpeg-devel] [PATCH] avcodec/wmalossless: use av_clip_intp2

2015-03-09 Thread Reimar Döffinger
On Sun, Mar 08, 2015 at 10:32:03PM +0100, Michael Niedermayer wrote: On Sat, Mar 07, 2015 at 11:44:10PM +0800, zhaoxiu.zeng wrote: From 47c997fa0623ab94a7a93b2d2e4cc4fa64c85d5f Mon Sep 17 00:00:00 2001 From: Zeng Zhaoxiu zhaoxiu.z...@gmail.com Date: Sat, 7 Mar 2015 23:26:42 +0800 Subject:

Re: [FFmpeg-devel] [PATCH] avcodec/wmalossless: use av_clip_intp2

2015-03-09 Thread Reimar Döffinger
On Mon, Mar 09, 2015 at 10:05:03PM +0100, Clément Bœsch wrote: On Mon, Mar 09, 2015 at 10:01:20PM +0100, Reimar Döffinger wrote: On Sun, Mar 08, 2015 at 10:32:03PM +0100, Michael Niedermayer wrote: On Sat, Mar 07, 2015 at 11:44:10PM +0800, zhaoxiu.zeng wrote: From

Re: [FFmpeg-devel] [PATCH] avcodec/hevc_parser: use avpriv_find_start_code in hevc_split()

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 09:49:04PM +0100, Reimar Döffinger wrote: On Tue, Mar 10, 2015 at 12:57:57AM +0800, zhaoxiu.zeng wrote: 在 2015/3/8 20:14, Michael Niedermayer 写道: On Sat, Mar 07, 2015 at 11:47:08PM +0800, zhaoxiu.zeng wrote: From ab12e3081ba987c2e05d819be97cde96952f1c2a Mon Sep 17

Re: [FFmpeg-devel] Patch proposal : filter ebur128 : add filter name before each line of summary print

2015-03-09 Thread Clément Bœsch
On Mon, Mar 09, 2015 at 10:28:43PM +0100, Martin Vignali wrote: In attach a patch proposal, in order to make easier parse result of ebur128 filter by external scripts command line for testing : ffmpeg -i aFile.wav -filter ebur128=peak=true+sample -f null - Before patch, summary look like

Re: [FFmpeg-devel] [PATCH] dump: show audio service type

2015-03-09 Thread Nicolas George
L'octidi 18 ventôse, an CCXXIII, Thomas Volkert a écrit : We could add const char *av_get_audio_service_name(enum AVAudioServiceType type) to avutil.h and use this function both in avformat and avfilter. In the short term, that would probably be the best (although I would slightly prefer if the

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Claudio Freire
On Mon, Mar 9, 2015 at 1:38 PM, Michael Niedermayer michae...@gmx.at wrote: On Mon, Mar 09, 2015 at 01:31:26PM -0300, Claudio Freire wrote: On Mon, Mar 9, 2015 at 1:26 PM, Michael Niedermayer michae...@gmx.at wrote: doesnt help but this does: -fate-aac-s7350-encode: CMD = enc_dec_pcm

[FFmpeg-devel] [PATCH] roqvideoenc: set enc-avctx in roq_encode_init

2015-03-09 Thread Andreas Cadhalpun
Hi, trying to encode roqvideo into asf currently crashes, because enc-avctx is not set in roq_encode_end, which calls: av_frame_free(enc-avctx-coded_frame); Best regards, Andreas From 4cd715ab02e25948f695c0a4186dab4b864a5ce3 Mon Sep 17 00:00:00 2001 From: Andreas Cadhalpun

Re: [FFmpeg-devel] [PATCH] lavfi: Add support to process_command in vf_eq.c

2015-03-09 Thread arwa arif
On Fri, Feb 20, 2015 at 5:41 AM, Stefano Sabatini stefa...@gmail.com wrote: On date Thursday 2015-02-19 17:13:15 +0530, Arwa Arif encoded: Updated the patch. From 66a8c9d03995c9e7c6ccc05fb9b20756f51c17f4 Mon Sep 17 00:00:00 2001 From: Arwa Arif arwaarif1...@gmail.com Date: Thu, 19 Feb

Re: [FFmpeg-devel] [PATCH] is_compiled flag not being cleared in av_opencl_uninit

2015-03-09 Thread Srikanth G
Hi Michael, I did the fix and verified compilation and run. Confirmed it works. Here is the patch --- libavutil/opencl.c | 4 1 file changed, 4 insertions(+) diff --git a/libavutil/opencl.c b/libavutil/opencl.c index 36cb6fe..2df5653 100644 --- a/libavutil/opencl.c +++

[FFmpeg-devel] [PATCH] avfilter/colormatrix:add slice threading

2015-03-09 Thread Yayoi
--- fixed the issue and cleaned up the code a bit --- libavfilter/vf_colormatrix.c | 146 +++ 1 file changed, 92 insertions(+), 54 deletions(-) diff --git a/libavfilter/vf_colormatrix.c b/libavfilter/vf_colormatrix.c index daba16e..d0839cc 100644 ---

Re: [FFmpeg-devel] [PATCH]Force __EXTENSIONS__ on Solaris

2015-03-09 Thread Michael Niedermayer
On Thu, Feb 26, 2015 at 05:23:52PM +0100, Carl Eugen Hoyos wrote: Hi! A user reported that the Solaris libc detection has never worked on all installations. Attached patch forces the definition of __EXTENSIONS__ which is needed for network compilation. This fixes the following error:

Re: [FFmpeg-devel] [libav-devel] [PATCH 2/2] rtpenc_mpegts: Set chain-rtp_ctx only after avformat_write_header succeeded

2015-03-09 Thread Andreas Cadhalpun
On 09.03.2015 22:15, Martin Storsjö wrote: By making sure we at each time only have one pointer set, either a local variable or one in the context, we avoid potential double frees in the cleanup routines. If chain-rtp_ctx is set, it is closed by calling avformat_write_trailer, but that shouldn't

Re: [FFmpeg-devel] GSoC '15 Introduction

2015-03-09 Thread Ilinca Andrei
Hi, I'm currently working on* dshow digital video capture* and it's really interesting, even if I' m making slow progress, as I' m not familiar with graphstudionext and DShow principles . I decided to apply for another project as well, the *MPEG-4 ALS encoder** project, *for two reasons, as

Re: [FFmpeg-devel] [libav-devel] [PATCH 1/2] rtpenc_mpegts: Free the right -pb in the error path in the init function

2015-03-09 Thread Andreas Cadhalpun
On 09.03.2015 22:15, Martin Storsjö wrote: This fixes a typo from 8e32b1f096. --- libavformat/rtpenc_mpegts.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rtpenc_mpegts.c b/libavformat/rtpenc_mpegts.c index 8ad446b..8ced6a9 100644 ---

Re: [FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary

2015-03-09 Thread Michael Niedermayer
On Thu, Feb 26, 2015 at 04:28:54PM +0100, Carl Eugen Hoyos wrote: On Thursday 26 February 2015 03:31:39 pm Derek Buitenhuis wrote: On 2/26/2015 2:09 PM, Carl Eugen Hoyos wrote: +snprintf(header, len + 3, %s\r\n, s-headers); This does not necessarily work on windows. The C

Re: [FFmpeg-devel] [libav-devel] [PATCH] rtpenc_mpegts: set rtp_ctx to NULL on avformat_write_header failure

2015-03-09 Thread Andreas Cadhalpun
Hi Martin, On 09.03.2015 22:08, Martin Storsjö wrote: Thanks for the patch, and thanks for finding the issue. You're quite correct that the error handling here is broken, but I'm not able to reproduce it by trying to mux bmp into this muxer - that actually succeeds. I guess I'm using the

Re: [FFmpeg-devel] [PATCH v6] libavformat/mxfenc: write package name metadata

2015-03-09 Thread Michael Niedermayer
On Thu, Mar 05, 2015 at 10:59:11AM -0800, Mark Reid wrote: --- libavformat/mxfenc.c | 97 +-- tests/ref/lavf/mxf| 6 +-- tests/ref/lavf/mxf_d10| 2 +- tests/ref/lavf/mxf_opatom | 2 +- 4 files changed, 91 insertions(+), 16

Re: [FFmpeg-devel] [PATCH] pthread: Fix?ff_thread_get_formatissues?when called outside frame decode.

2015-03-09 Thread Hendrik Leppkes
On Mon, Mar 9, 2015 at 11:06 PM, Michael Niedermayer michae...@gmx.at wrote: On Mon, Mar 09, 2015 at 06:38:18AM +, Rainer Hochecker wrote: Reimar Döffinger Reimar.Doeffinger at gmx.de writes: Any reason to believe this patch causes it? Because I can't see how it would. Maybe it's

[FFmpeg-devel] [PATCH] avfilter/colormatrix:add slice threading

2015-03-09 Thread Yayoi
--- separate a struct variable declaraion and assignment statements. --- libavfilter/vf_colormatrix.c | 146 --- 1 file changed, 94 insertions(+), 52 deletions(-) diff --git a/libavfilter/vf_colormatrix.c b/libavfilter/vf_colormatrix.c index

Re: [FFmpeg-devel] [PATCH] avfilter/colormatrix:add slice threading

2015-03-09 Thread Timothy Gu
On Mon, Mar 9, 2015 at 4:34 PM Yayoi yayoi.u...@gmail.com wrote: @@ -372,12 +396,26 @@ static int filter_frame(AVFilterLink *link, AVFrame *in) calc_coefficients(ctx); +ThreadData td = { +.src = in, +.dst = out, +.c2 = color-yuv_convert[color-mode][0][1],

Re: [FFmpeg-devel] [PATCH] AAC: [PATCH] AAC: Add support for 7350Hz sampling rates

2015-03-09 Thread Claudio Freire
On Mon, Mar 9, 2015 at 3:31 PM, Claudio Freire klaussfre...@gmail.com wrote: On Mon, Mar 9, 2015 at 1:38 PM, Michael Niedermayer michae...@gmx.at wrote: On Mon, Mar 09, 2015 at 01:31:26PM -0300, Claudio Freire wrote: On Mon, Mar 9, 2015 at 1:26 PM, Michael Niedermayer michae...@gmx.at wrote:

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-09 Thread Reimar Döffinger
On 09.03.2015, at 07:38, Rainer Hochecker fernetme...@online.de wrote: Reimar Döffinger Reimar.Doeffinger at gmx.de writes: Any reason to believe this patch causes it? Because I can't see how it would. Maybe it's just a bug with DXVA and multithreading in the HEVC code? Can you provide

[FFmpeg-devel] For x264, should I contribute to videoLan directly

2015-03-09 Thread rongyan
All, x264 is the external library of FFmpeg. If I want to contribute to it, should I contribute to VideoLan directly? Thanks. Rong ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-09 Thread Rainer Hochecker
Reimar Döffinger Reimar.Doeffinger at gmx.de writes: Any reason to believe this patch causes it? Because I can't see how it would. Maybe it's just a bug with DXVA and multithreading in the HEVC code? Can you provide some more information like a stacktrace, possibly using a tool like

Re: [FFmpeg-devel] [PATCH]Autoselect ffv1 coder also for RGB colourpaces

2015-03-09 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: Attached patch makes the behaviour of ffv1 using gprp more similar to yuv for 8 bit. Reported by Christoph Gerstbauer . Please comment, Carl Eugen LGTM Patch applied. Thank you, Carl Eugen

Re: [FFmpeg-devel] [PATCH]Improve video creation examples

2015-03-09 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: ffmpeg.texi |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 3e2ef09ab39df5eb9d54af4ddef424b8b1cc7c26 patchimg2framerate.diff probably ok Patch applied. Thank you, Carl Eugen ___

Re: [FFmpeg-devel] [PATCH]Force __EXTENSIONS__ on Solaris

2015-03-09 Thread Carl Eugen Hoyos
Carl Eugen Hoyos cehoyos at ag.or.at writes: A user reported that the Solaris libc detection has never worked on all installations. This should have been ... since 3a5cbc91. Attached patch forces the definition of __EXTENSIONS__ which is needed for network compilation. This fixes the

Re: [FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary

2015-03-09 Thread Carl Eugen Hoyos
Carl Eugen Hoyos cehoyos at ag.or.at writes: I believe attached patch fixes ticket #3268 in some cases. Ping, should one of the patches get applied? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] For x264, should I contribute to videoLan directly

2015-03-09 Thread Carl Eugen Hoyos
rongyan rongyan236 at foxmail.com writes: x264 is the external library of FFmpeg. If I want to contribute to it, should I contribute to VideoLan directly? Please see https://mailman.videolan.org/listinfo/x264-devel Carl Eugen ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]Improve the intra_dc_precision API doc

2015-03-09 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: The mpegvideo decoder sets intra_dc_precision since forever (!). Please comment, Carl Eugen LGTM Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 1/8] ffmdec: initialize f_cprv, f_stvi and f_stau

2015-03-09 Thread Lukasz Marek
On 09.03.2015 02:45, Michael Niedermayer wrote: On Mon, Mar 09, 2015 at 12:02:55AM +0100, Andreas Cadhalpun wrote: Hi, attached patch fixes 'Conditional jump or move depends on uninitialized variables' valgrind warnings. Best regards, Andreas ffmdec.c |2 +- 1 file changed, 1

Re: [FFmpeg-devel] [PATCH]Clarify documentation for fade duration

2015-03-09 Thread Carl Eugen Hoyos
Carl Eugen Hoyos cehoyos at ag.or.at writes: -If both duration and nb_frames are specified, duration is used. Default is 0. +If both duration and nb_frames are specified, duration is used. Default is 0 +(nb_frames is used by default). I pushed this patch as I believe it improves the

Re: [FFmpeg-devel] [PATCH]Set bits_per_raw_sample for hqx

2015-03-09 Thread Carl Eugen Hoyos
Paul B Mahol onemda at gmail.com writes: Attached patch sets bits_per_raw_sample when decoding hqx. I will push this if nobody objects. How do you know this is correct? I had the feeling the current code cannot be correct but please consider the patch dropped. Carl Eugen