Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Accept MXF D-10 with 49.999840 Mbit/sec

2015-06-01 Thread tim nicholson
On 01/06/15 20:35, Michael Niedermayer wrote: > This is the maximum rate possible based on the frame size limit of MXF D-10 > > Signed-off-by: Michael Niedermayer > --- > libavformat/mxfenc.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mxfenc.c b/liba

[FFmpeg-devel] colorspace.h

2015-06-01 Thread Chris
Hello - I have found what I believe to be some questionable code in colorspace.h. I have modified the code to what I believe is correct but am unable to compile ffmpeg myself on Windows to test my modifications, despite spending many hours trying to do so and reading several how-to's. We can d

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-06-01 Thread Carl Eugen Hoyos
Anshul gmail.com> writes: > can you provide me link of carl file you said it was > smallest, I would make that test case again with > carl file. http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/192583/focus=192944 http://ffmpeg.org/pipermail/ffmpeg-devel/2015-May/172810.html Didn't you g

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-06-01 Thread Anshul
On 05/31/2015 06:53 PM, Michael Niedermayer wrote: On Sun, May 31, 2015 at 01:59:44PM +0200, Clément Bœsch wrote: On Tue, May 05, 2015 at 08:37:03PM +0530, Anshul wrote: On 05/05/2015 08:33 PM, Anshul wrote: On 05/04/2015 08:50 PM, Michael Niedermayer wrote: this is 31mb, that does not se

Re: [FFmpeg-devel] [PATCH] avcodec: loongson3 optimized h264chroma put and avg with mmi

2015-06-01 Thread 周晓勇
abort the one before and re-patch on the latest master. --- >From c35e1e18f446fc15233ab42f52077d57681b27fa Mon Sep 17 00:00:00 2001 From: ZhouXiaoyong Date: Fri, 17 Apr 2015 17:09:06 +0800 Subject: [PATCH] avcodec: loongson3 optimized h264chroma put and avg with mmi Signed-off-by: ZhouXiaoyong

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mov: increase current_sample after reading, to avoid discontinuous sample

2015-06-01 Thread Zhang Rui
2015-06-02 7:26 GMT+08:00 Michael Niedermayer : > On Thu, May 21, 2015 at 12:31:21PM +0800, Zhang Rui wrote: >> Discontinuous sample could cause corrupted image if next video frame >> is non-key frame. >> >> Mostly, avio_seek() fails on I/O error for http/ftp/... stream. >> In my opinion, retry is

Re: [FFmpeg-devel] [PATCH] avcodec: loongson3 optimized h264chroma put and avg with mmi

2015-06-01 Thread 周晓勇
sorry about the mistake when pasting to email. --- >From 1b0f0c0d3b3eb017c7a79daaba89f9fc677381c4 Mon Sep 17 00:00:00 2001 From: ZhouXiaoyong Date: Fri, 17 Apr 2015 17:09:06 +0800 Subject: [PATCH] avcodec: loongson3 optimized h264chroma put and avg with mmi Signed-off-by: ZhouXiaoyong --- liba

Re: [FFmpeg-devel] [PATCH] lavf/matroskaenc: Write Block Keyframe correctly

2015-06-01 Thread James Almer
On 01/06/15 9:07 PM, Vignesh Venkatasubramanian wrote: > Per matroska Block Structure [1], for keyframes 0th bit of the flag > should not be set (unlike SimpleBlocks). For Blocks, keyframes is > inferred by the absence of ReferenceBlock element (as done by > matroskadec). This CL writes the flag co

Re: [FFmpeg-devel] [PATCH]Double sar for tinterlace modes merge and pad

2015-06-01 Thread Michael Niedermayer
On Tue, Jun 02, 2015 at 01:48:54AM +0200, Carl Eugen Hoyos wrote: > On Tuesday 02 June 2015 01:24:35 am Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes aspect for the tinterlace modes merge and pad. > > Tested with the sample from ticket #4328. > > New patch attached that works with f

Re: [FFmpeg-devel] [PATCH] avcodec/mips: MSA (MIPS-SIMD-Arch) optimizations for HEVC bi mc functions

2015-06-01 Thread Michael Niedermayer
On Fri, May 29, 2015 at 12:20:26PM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > This patch adds MSA (MIPS-SIMD-Arch) optimizations for HEVC bi mc functions > (qpel as well as epel) in new file hevc_mc_bi_msa.c > Adds new generic macros (needed for this patch) in > libavutil/

Re: [FFmpeg-devel] [PATCH]Double sar for tinterlace modes merge and pad

2015-06-01 Thread Carl Eugen Hoyos
On Tuesday 02 June 2015 01:24:35 am Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes aspect for the tinterlace modes merge and pad. > Tested with the sample from ticket #4328. New patch attached that works with ffplay and ffmpeg. Carl Eugen diff --git a/libavfilter/vf_tinterlace.c b/libavfi

Re: [FFmpeg-devel] [PATCH][GSoC] lavf/http: Parse and set HTTP method when listening on HTTP(S)

2015-06-01 Thread Stephan Holljes
On Mon, Jun 1, 2015 at 10:52 PM, Nicolas George wrote: > Le tridi 13 prairial, an CCXXIII, Stephan Holljes a écrit : >> After looking at the ffserver code, I'm wondering if maintaining a >> list of clients as HTTPContext or custom wrapper structs would make >> sense. >> For the time being it would

Re: [FFmpeg-devel] [PATCH]Double sar for tinterlace modes merge and pad

2015-06-01 Thread Michael Niedermayer
On Tue, Jun 02, 2015 at 01:24:35AM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes aspect for the tinterlace modes merge and pad. > Tested with the sample from ticket #4328. > > Please comment, Carl Eugen you also have to update the outlink sample_aspect_ratio [...] -- Michael

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mov: increase current_sample after reading, to avoid discontinuous sample

2015-06-01 Thread Michael Niedermayer
On Thu, May 21, 2015 at 12:31:21PM +0800, Zhang Rui wrote: > Discontinuous sample could cause corrupted image if next video frame > is non-key frame. > > Mostly, avio_seek() fails on I/O error for http/ftp/... stream. > In my opinion, retry is better than skip. > > > -/* must be done just bef

[FFmpeg-devel] [PATCH]Double sar for tinterlace modes merge and pad

2015-06-01 Thread Carl Eugen Hoyos
Hi! Attached patch fixes aspect for the tinterlace modes merge and pad. Tested with the sample from ticket #4328. Please comment, Carl Eugen diff --git a/libavfilter/vf_tinterlace.c b/libavfilter/vf_tinterlace.c index 096a614..12ab4f6 100644 --- a/libavfilter/vf_tinterlace.c +++ b/libavfilter/vf

Re: [FFmpeg-devel] [PATCH][GSoC] lavf/http: Parse and set HTTP method when listening on HTTP(S)

2015-06-01 Thread Nicolas George
Le tridi 13 prairial, an CCXXIII, Stephan Holljes a écrit : > After looking at the ffserver code, I'm wondering if maintaining a > list of clients as HTTPContext or custom wrapper structs would make > sense. > For the time being it would only contain the singular client context > which can be accep

Re: [FFmpeg-devel] [PATCH] configure: don't enable any tls protocol if network is disabled

2015-06-01 Thread James Almer
On 01/06/15 5:45 PM, Michael Niedermayer wrote: > On Mon, Jun 01, 2015 at 04:47:55PM -0300, James Almer wrote: >> This was a regression introduced with >> 4a006b9eb7e7d736fd00e6045b5612978ef6404b. >> >> Signed-off-by: James Almer >> --- >> configure | 3 +++ >> 1 file changed, 3 insertions(+) >

Re: [FFmpeg-devel] [PATCH] configure: don't enable any tls protocol if network is disabled

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 04:47:55PM -0300, James Almer wrote: > This was a regression introduced with > 4a006b9eb7e7d736fd00e6045b5612978ef6404b. > > Signed-off-by: James Almer > --- > configure | 3 +++ > 1 file changed, 3 insertions(+) LGTM thanks [...] -- Michael GnuPG fingerprint: 9F

[FFmpeg-devel] [PATCH] configure: don't enable any tls protocol if network is disabled

2015-06-01 Thread James Almer
This was a regression introduced with 4a006b9eb7e7d736fd00e6045b5612978ef6404b. Signed-off-by: James Almer --- configure | 3 +++ 1 file changed, 3 insertions(+) diff --git a/configure b/configure index aabcdc2..8460bc6 100755 --- a/configure +++ b/configure @@ -2652,6 +2652,9 @@ tcp_protocol_s

[FFmpeg-devel] [PATCH] avformat/mxfenc: Accept MXF D-10 with 49.999840 Mbit/sec

2015-06-01 Thread Michael Niedermayer
This is the maximum rate possible based on the frame size limit of MXF D-10 Signed-off-by: Michael Niedermayer --- libavformat/mxfenc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index e0ae14e..7e41c5c 100644 --- a/libavform

[FFmpeg-devel] [PATCH] avcodec/utils: Fix null pointer dereference with user specific codec_id

2015-06-01 Thread Michael Niedermayer
Fixes CID1302837 Signed-off-by: Michael Niedermayer --- libavcodec/utils.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 8695cb9..7e3f92a 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -1616,6 +1616,7 @@ int a

Re: [FFmpeg-devel] [PATCH] avcodec/utils: Warn if a codec is opened with a codec id without codec descriptor

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 08:30:33PM +0200, wm4 wrote: > On Mon, 1 Jun 2015 20:27:11 +0200 > Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/utils.c |2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/libavcodec/utils.c b/libavcodec/uti

Re: [FFmpeg-devel] [PATCH] avcodec/utils: Warn if a codec is opened with a codec id without codec descriptor

2015-06-01 Thread wm4
On Mon, 1 Jun 2015 20:27:11 +0200 Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/utils.c |2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index 7e3f92a..22e826f 100644 > --- a/libavcodec/utils.c > +++ b

Re: [FFmpeg-devel] [PATCH] avcodec/utils: Warn if a codec is opened with a codec id without codec descriptor

2015-06-01 Thread James Almer
On 01/06/15 3:27 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/utils.c |2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index 7e3f92a..22e826f 100644 > --- a/libavcodec/utils.c > +++ b/libavcodec/util

[FFmpeg-devel] [PATCH] avcodec/utils: Warn if a codec is opened with a codec id without codec descriptor

2015-06-01 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/utils.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 7e3f92a..22e826f 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -1453,6 +1453,8 @@ int attribute_align_arg avcodec_open2(AV

Re: [FFmpeg-devel] [PATCH][GSoC] lavf/http: Parse and set HTTP method when listening on HTTP(S)

2015-06-01 Thread Stephan Holljes
On Sat, May 30, 2015 at 10:13 AM, Nicolas George wrote: > Le decadi 10 prairial, an CCXXIII, Stephan Holljes a écrit : >> Should method be reassigned or should the set value be kept? As far as >> I can tell the only way to set method is by specifying it as an option >> in the command line. >> I do

Re: [FFmpeg-devel] [PATCH] doc/formats.texi: document max_interlave_delta muxing option

2015-06-01 Thread Stefano Sabatini
On date Monday 2015-06-01 17:38:37 +0200, Michael Niedermayer encoded: > On Mon, Jun 01, 2015 at 05:16:50PM +0200, Stefano Sabatini wrote: > > The option was added in commit d9ae1031f5edbd25c8526b4cb51aba66d3bee931 > > but it was never document. > > --- > > doc/formats.texi | 19 ++

Re: [FFmpeg-devel] [PATCH] doc/formats.texi: document max_interlave_delta muxing option

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 05:16:50PM +0200, Stefano Sabatini wrote: > The option was added in commit d9ae1031f5edbd25c8526b4cb51aba66d3bee931 > but it was never document. > --- > doc/formats.texi | 19 +++ > 1 file changed, 19 insertions(+) sounds correct to me [...] -- Michael

Re: [FFmpeg-devel] [PATCH] vp9: clamp final zero MV if find_ref_mvs() found no suitable candidates.

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 11:17:44AM -0400, Ronald S. Bultje wrote: > This may actually result in a non-zero MV. Fixes ticket 4583. > --- > libavcodec/vp9.c | 1 + > 1 file changed, 1 insertion(+) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No

[FFmpeg-devel] [PATCH] vp9: clamp final zero MV if find_ref_mvs() found no suitable candidates.

2015-06-01 Thread Ronald S. Bultje
This may actually result in a non-zero MV. Fixes ticket 4583. --- libavcodec/vp9.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 7f28b26..244eb0c 100644 --- a/libavcodec/vp9.c +++ b/libavcodec/vp9.c @@ -1272,6 +1272,7 @@ static void find_ref_mvs(VP9C

[FFmpeg-devel] [PATCH] doc/formats.texi: document max_interlave_delta muxing option

2015-06-01 Thread Stefano Sabatini
The option was added in commit d9ae1031f5edbd25c8526b4cb51aba66d3bee931 but it was never document. --- doc/formats.texi | 19 +++ 1 file changed, 19 insertions(+) diff --git a/doc/formats.texi b/doc/formats.texi index aa39d4b..617cda5 100644 --- a/doc/formats.texi +++ b/doc/format

Re: [FFmpeg-devel] [PATCH 13/14] tests: Add aac_fixed decoder test

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 01:46:32PM +, Nedeljko Babic wrote: > >On Fri, May 29, 2015 at 03:46:32PM +0200, Nedeljko Babic wrote: > >> Signed-off-by: Nedeljko Babic > >> --- > >> tests/fate/aac.mak | 58 > >> +- > >> 1 file changed, 57 inserti

Re: [FFmpeg-devel] [PATCH 02/14] libavutil: Add new fixed dsp functions.

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 01:23:53PM +, Nedeljko Babic wrote: > > Added functions needed for implementation of fixed point aac dec. [...] > >> + > >> AVFixedDSPContext * avpriv_alloc_fixed_dsp(int bit_exact) > >> { > >> AVFixedDSPContext * fdsp = av_malloc(sizeof(AVFixedDSPContext)); > >>

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Correct klv_fill_key

2015-06-01 Thread tim nicholson
On 01/06/15 15:01, Michael Niedermayer wrote: > On Mon, Jun 01, 2015 at 07:11:38AM +0100, tim nicholson wrote: >> On 29/05/15 23:26, Michael Niedermayer wrote: >>> See SMPTE 377-1-2009 6.3.3 KLV Fill Items >>> >>> Signed-off-by: Michael Niedermayer >>> --- >>> libavformat/mxfenc.c|

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Correct klv_fill_key

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 07:11:38AM +0100, tim nicholson wrote: > On 29/05/15 23:26, Michael Niedermayer wrote: > > See SMPTE 377-1-2009 6.3.3 KLV Fill Items > > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/mxfenc.c|2 +- > > tests/ref/lavf/mxf |

Re: [FFmpeg-devel] [PATCH 13/14] tests: Add aac_fixed decoder test

2015-06-01 Thread Nedeljko Babic
>On Fri, May 29, 2015 at 03:46:32PM +0200, Nedeljko Babic wrote: >> Signed-off-by: Nedeljko Babic >> --- >> tests/fate/aac.mak | 58 >> +- >> 1 file changed, 57 insertions(+), 1 deletion(-) > >These tests do not all pass: > >tests/data/fate/aac

Re: [FFmpeg-devel] [libav-devel] [PATCH] libopenjpegdec: check existence of image component data

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 12:27:22AM +0200, Andreas Cadhalpun wrote: > On 31.05.2015 11:11, Luca Barbato wrote: > > On 30/05/15 20:00, Andreas Cadhalpun wrote: > >> +memset(&ctx->event_mgr, 0, sizeof(opj_event_mgr_t)); > > > > usually the size is sizeof(*actualpoint) > > Yes that's better, patc

Re: [FFmpeg-devel] [PATCH 02/14] libavutil: Add new fixed dsp functions.

2015-06-01 Thread Nedeljko Babic
> Added functions needed for implementation of fixed point aac dec. >> >> Signed-off-by: Nedeljko Babic >> --- >> libavutil/fixed_dsp.c | 60 >> +++ >> libavutil/fixed_dsp.h | 53 + >> 2 files changed, 1

Re: [FFmpeg-devel] [PATCH 14/14] Edit documentation and versioning

2015-06-01 Thread Nedeljko Babic
>> @@ -57,7 +57,7 @@ >> >> #define LIBAVUTIL_VERSION_MAJOR 54 >> #define LIBAVUTIL_VERSION_MINOR 26 >> -#define LIBAVUTIL_VERSION_MICRO 100 >> +#define LIBAVUTIL_VERSION_MICRO 101 > >a minor bump is probably more correct for addition of a new decoder Ok. I'll change this. Thanks, -Nedeljko

Re: [FFmpeg-devel] [PATCH] libopenjpegenc: add NULL check for img before accessing it

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 12:57:25AM +0200, Andreas Cadhalpun wrote: > If opj_image_create fails to allocate an image it returns NULL, which > causes a segmentation fault at 'img->x0 = 0'. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/libopenjpegenc.c | 3 +++ > 1 file changed, 3 inserti

Re: [FFmpeg-devel] [PATCH] avcodec: loongson3 optimized h264chroma put and avg with mmi

2015-06-01 Thread Michael Niedermayer
On Mon, Jun 01, 2015 at 11:29:31AM +0800, 周晓勇 wrote: > From 671813a26715d9d2c1b0ef9da9e4da458b709e50 Mon Sep 17 00:00:00 2001 > From: ZhouXiaoyong > Date: Fri, 17 Apr 2015 17:09:06 +0800 > Subject: [PATCH] avcodec: loongson3 optimized h264chroma put and avg with mmi [...] > +av_cold void ff_h264c