Re: [FFmpeg-devel] [PATCH 01/13] Rename mipsdspr1 to mipsdsp

2015-11-17 Thread Nedeljko Babic
I applied all the patches and it worked fine. But, there is a problem with the first patch indeed. There is one 'fi' command added that looks misplaced. It looks like patch 12 fixes problem caused with patch 1 by adding appropriate 'if'. -Nedeljko __

Re: [FFmpeg-devel] [PATCH] configure: Add option to use gmp

2015-11-17 Thread Matt Oliver
On 15 November 2015 at 01:09, Ricardo Constantino wrote: > Enables RTMP(T)E support with SChannel > > (Not sure if this is the right way to enable this) > > Signed-off-by: Ricardo Constantino > --- > configure | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/configu

Re: [FFmpeg-devel] UDP constant bitrate feature (cbr)

2015-11-17 Thread Zach Swena
Hi Pavel, I can confirm that there is a problem with the UDP packet engine in FFmpeg. FFmpeg has excessive jitter in it's UDP streaming output to the point where hardware decoders can't handle it. My solution was to buffer to disk and have my own program read and send the datagrams via a very ti

Re: [FFmpeg-devel] [PATCH] avformat/mp3dec, rmdec: check return value of ffio_ensure_seekback

2015-11-17 Thread Ganesh Ajjanagadde
On Mon, Nov 16, 2015 at 8:12 AM, Ganesh Ajjanagadde wrote: > On Mon, Nov 16, 2015 at 7:53 AM, Hendrik Leppkes wrote: >> On Mon, Nov 16, 2015 at 1:52 PM, Ganesh Ajjanagadde wrote: >>> On Mon, Nov 16, 2015 at 3:27 AM, wm4 wrote: On Sun, 15 Nov 2015 17:56:22 -0500 Ganesh Ajjanagadde wro

[FFmpeg-devel] [PATCHv2] avformat/mp3dec, rmdec: check return value of ffio_ensure_seekback

2015-11-17 Thread Ganesh Ajjanagadde
ffio_ensure_seekback can fail due to e.g ENOMEM. This return value is checked here and a diagnostic is logged. All usage of ffio_ensure_seekback in the codebase now has the return value checked. Signed-off-by: Ganesh Ajjanagadde --- libavformat/mp3dec.c | 6 -- libavformat/rmdec.c | 3 ++-

Re: [FFmpeg-devel] [PATCH] avutil/libm: correct isnan, isinf compat hacks

2015-11-17 Thread Ganesh Ajjanagadde
On Mon, Nov 16, 2015 at 7:50 AM, Ganesh Ajjanagadde wrote: > On Mon, Nov 16, 2015 at 6:55 AM, Michael Niedermayer > wrote: >> On Sat, Nov 14, 2015 at 08:05:59PM -0500, Ganesh Ajjanagadde wrote: >>> isnan and isinf are actually macros as per the standard. In particular, >>> the existing implementa

[FFmpeg-devel] [PATCHv2] avutil/libm: correct isnan, isinf compat hacks

2015-11-17 Thread Ganesh Ajjanagadde
isnan and isinf are actually macros as per the standard. In particular, the existing implementation has incorrect signature. Furthermore, this results in undefined behavior for e.g double values outside float range as per the standard. This patch corrects the undefined behavior for all usage withi

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/faandct: use typedef instead of #define

2015-11-17 Thread Ganesh Ajjanagadde
On Sun, Nov 15, 2015 at 8:43 PM, Ganesh Ajjanagadde wrote: > On Sun, Nov 15, 2015 at 8:33 PM, Michael Niedermayer > wrote: >> On Sun, Nov 15, 2015 at 10:33:10AM -0500, Ganesh Ajjanagadde wrote: >>> On Fri, Nov 13, 2015 at 12:05 PM, Ganesh Ajjanagadde >>> wrote: >>> > On Fri, Nov 13, 2015 at 11:

Re: [FFmpeg-devel] [PATCH] avformat/rtpdec_mpa_robust: change assignment to equality test in conditional

2015-11-17 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 9:50 AM, Ganesh Ajjanagadde wrote: > On Wed, Oct 28, 2015 at 10:19 PM, Michael Niedermayer > wrote: >> On Tue, Oct 27, 2015 at 07:58:03PM -0400, Ganesh Ajjanagadde wrote: >>> In the spirit of commit 8199908fdf9b3797cceaea9d1e2fc09d02ef7309, likely >>> typo originally. >>>

Re: [FFmpeg-devel] [PATCH 01/13] Rename mipsdspr1 to mipsdsp

2015-11-17 Thread Michael Niedermayer
On Mon, Nov 16, 2015 at 02:57:07PM +, Vicente Olivert Riera wrote: [...] > index 9a736ce..0461a97 100755 > --- a/configure > +++ b/configure > @@ -380,7 +380,7 @@ Optimization options (experts only): >--disable-yasm disable use of nasm/yasm assembly >--disable-mips32r5 d

[FFmpeg-devel] [PATCH] avformat/dashenc: Add framerate to dash manifest

2015-11-17 Thread Bryan Huh
DASH manifest should have framerate specified as an attribute in the AdaptationSet element and Representation elements. Though ISO/IEC 23009-1:2014 doesn't seem to define frameRate as a required attribute, it is at least optional, and DASH-IF IOP 3.0 seems to require it. See section 3.2.4 of http:/

Re: [FFmpeg-devel] 6c6ac9cb "avutil/x86/intmath: Use tzcnt in place of bsf."

2015-11-17 Thread Hans Wennborg
Hi Matt, Thanks for the quick reply. On Mon, Nov 16, 2015 at 11:02 PM, Matt Oliver wrote: > On 17 November 2015 at 12:12, Hans Wennborg wrote: >> This commit [1] is causing problems when compiling with Clang on Windows: >> >> ..\..\third_party\ffmpeg\libavutil/x86/intmath.h(53,33) : error: >>

Re: [FFmpeg-devel] [PATCH] avformat: add IVR demuxer

2015-11-17 Thread Paul B Mahol
On 11/16/15, Paul B Mahol wrote: > On 11/16/15, Michael Niedermayer wrote: >> On Mon, Nov 16, 2015 at 11:25:33AM +0100, Paul B Mahol wrote: >>> 2n version attached >> >>> Makefile |1 >>> allformats.c |1 >>> rmdec.c | 303 >>> +++

Re: [FFmpeg-devel] [PATCH] avformat/genh: Fix tools/probetest failure

2015-11-17 Thread Paul B Mahol
On 11/17/15, Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavformat/genh.c |2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/genh.c b/libavformat/genh.c > index 260e320..d8723d5 100644 > --- a/libavformat/genh.c

Re: [FFmpeg-devel] [PATCH 06/13] MIPS i6400 core do has an FPU, so not disable it

2015-11-17 Thread Ronald S. Bultje
Hi, On Tue, Nov 17, 2015 at 11:40 AM, Vicente Olivert Riera < vincent.ri...@imgtec.com> wrote: > Dear Moritz Barsnick, > > On 11/17/2015 04:20 PM, Moritz Barsnick wrote: > > On Tue, Nov 17, 2015 at 15:07:56 +, Vicente Olivert Riera wrote: > >> file. Do you mean the patch subject instead of pa

Re: [FFmpeg-devel] [PATCH 06/13] MIPS i6400 core do has an FPU, so not disable it

2015-11-17 Thread Vicente Olivert Riera
Dear Moritz Barsnick, On 11/17/2015 04:20 PM, Moritz Barsnick wrote: > On Tue, Nov 17, 2015 at 15:07:56 +, Vicente Olivert Riera wrote: >> file. Do you mean the patch subject instead of patch name? If so, what's >> the error? > > Indeed, the grammar: > >> [PATCH 06/13] MIPS i6400 core do has

Re: [FFmpeg-devel] [PATCH] ffprobe: do not exit in case the demuxer returns AVERROR(EAGAIN)

2015-11-17 Thread Stefano Sabatini
On date Tuesday 2015-11-17 17:12:46 +0100, wm4 encoded: > On Tue, 17 Nov 2015 17:00:29 +0100 > Stefano Sabatini wrote: [...] > > No, just in case ret == AVERROR(EAGAIN), in all other failure cases it > > will exit the loop immediately. > > > > BTW, the same problem affects some API usage examples

Re: [FFmpeg-devel] [PATCH 06/13] MIPS i6400 core do has an FPU, so not disable it

2015-11-17 Thread Moritz Barsnick
On Tue, Nov 17, 2015 at 15:07:56 +, Vicente Olivert Riera wrote: > file. Do you mean the patch subject instead of patch name? If so, what's > the error? Indeed, the grammar: > [PATCH 06/13] MIPS i6400 core do has an FPU, so not disable it -> MIPS i6400 core has an FPU, so not disable it or w

Re: [FFmpeg-devel] [PATCH] ffprobe: do not exit in case the demuxer returns AVERROR(EAGAIN)

2015-11-17 Thread wm4
On Tue, 17 Nov 2015 17:00:29 +0100 Stefano Sabatini wrote: > On date Tuesday 2015-11-17 15:20:32 +0100, wm4 encoded: > > On Tue, 17 Nov 2015 15:13:02 +0100 > > Stefano Sabatini wrote: > > > > > Fix demuxing of files for which the demuxer returns AVERROR(EAGAIN) at > > > some point. Also retur

Re: [FFmpeg-devel] [PATCH] avformat/mlpdec: consider all valid mlp access units when probing

2015-11-17 Thread Michael Niedermayer
On Tue, Nov 17, 2015 at 12:28:58AM +0100, Hendrik Leppkes wrote: > Fixes probing of truehd/mlp files with a lot of frames in between the > major sync frames. The spec allows a distance of up to 128 frames in > between major sync frames, which leads to the probing code not reaching > the desired sco

Re: [FFmpeg-devel] [PATCHv2] configure+libm.h: add hypot emulation

2015-11-17 Thread Hendrik Leppkes
On Tue, Nov 17, 2015 at 5:00 PM, Ganesh Ajjanagadde wrote: > On Sun, Nov 15, 2015 at 11:59 AM, Ganesh Ajjanagadde wrote: >> On Sun, Nov 15, 2015 at 11:34 AM, Michael Niedermayer >> wrote: >>> On Sun, Nov 15, 2015 at 11:01:58AM -0500, Ganesh Ajjanagadde wrote: On Sun, Nov 15, 2015 at 10:56 A

[FFmpeg-devel] [PATCH] avformat/genh: Fix tools/probetest failure

2015-11-17 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavformat/genh.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/genh.c b/libavformat/genh.c index 260e320..d8723d5 100644 --- a/libavformat/genh.c +++ b/libavformat/genh.c @@ -32,6 +32,8 @@ static int genh_

[FFmpeg-devel] [PATCH] movenc-test: Pad the packet data start with 0s

2015-11-17 Thread Derek Buitenhuis
This way, it never starts with 0xFFF0, and never trips the ADTS "Detection" code in movenc.c. Signed-off-by: Derek Buitenhuis --- libavformat/movenc-test.c | 4 ++-- tests/ref/fate/movenc | 40 2 files changed, 22 insertions(+), 22 deletions(-) diff

Re: [FFmpeg-devel] [PATCH] ffprobe: do not exit in case the demuxer returns AVERROR(EAGAIN)

2015-11-17 Thread Stefano Sabatini
On date Tuesday 2015-11-17 15:20:32 +0100, wm4 encoded: > On Tue, 17 Nov 2015 15:13:02 +0100 > Stefano Sabatini wrote: > > > Fix demuxing of files for which the demuxer returns AVERROR(EAGAIN) at > > some point. Also returns error code to the caller in case of different > > non EOF error. > > ---

Re: [FFmpeg-devel] [PATCHv2] configure+libm.h: add hypot emulation

2015-11-17 Thread Ganesh Ajjanagadde
On Sun, Nov 15, 2015 at 11:59 AM, Ganesh Ajjanagadde wrote: > On Sun, Nov 15, 2015 at 11:34 AM, Michael Niedermayer > wrote: >> On Sun, Nov 15, 2015 at 11:01:58AM -0500, Ganesh Ajjanagadde wrote: >>> On Sun, Nov 15, 2015 at 10:56 AM, Michael Niedermayer >>> wrote: >>> > On Sun, Nov 15, 2015 at 1

[FFmpeg-devel] [PATCH] avcodec: add OpenJPEG 2.x compatibility

2015-11-17 Thread Michael Bradshaw
Hi, Attached patch adds support for OpenJPEG 2.0/2.1. Please review. Thanks, Michael Bradshaw 0001-avcodec-add-OpenJPEG-2.x-compatibility.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/l

Re: [FFmpeg-devel] [PATCH 06/13] MIPS i6400 core do has an FPU, so not disable it

2015-11-17 Thread Vicente Olivert Riera
Dear Nedeljko Babic, On 11/17/2015 03:04 PM, Nedeljko Babic wrote: > It looks to me that there is an error in patch name. the patch has been sent via git send-email, so there is no attachment file. Do you mean the patch subject instead of patch name? If so, what's the error? Regards, Vincent.

Re: [FFmpeg-devel] [PATCH 12/13] Improve detection of MIPS ISAs, FPU and ASEs (DSP, MSA)

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 12/13] Improve detection of MIPS ISAs, FPU and ASEs (DSP, MSA) Signed-off-by:

Re: [FFmpeg-devel] [PATCH 11/13] Remove the MIPS "generic" core in favor of "*"

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 11/13] Remove the MIPS "generic" core in favor of "*" There is no point to ha

Re: [FFmpeg-devel] [PATCH 13/13] Add support for MIPS R6

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 13/13] Add support for MIPS R6 Understanding the mips32r6 and mips64r6 ISAs i

Re: [FFmpeg-devel] [PATCH 08/13] Add mips64r2 architecture variant

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 08/13] Add mips64r2 architecture variant Signed-off-by: Vicente Olivert Riera

Re: [FFmpeg-devel] [PATCH 09/13] Add mips32r6 architecture variant

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 09/13] Add mips32r6 architecture variant Signed-off-by: Vicente Olivert Riera

Re: [FFmpeg-devel] [PATCH 10/13] Put "disable mipsfpu" in a better place for loongson

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 10/13] Put "disable mipsfpu" in a better place for loongson Let's disable the

Re: [FFmpeg-devel] [PATCH 06/13] MIPS i6400 core do has an FPU, so not disable it

2015-11-17 Thread Nedeljko Babic
It looks to me that there is an error in patch name. Other than that LGTM. Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 06/13]

Re: [FFmpeg-devel] [PATCH 05/13] Add MIPS i6400 core optimizations

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 05/13] Add MIPS i6400 core optimizations Signed-off-by: Vicente Olivert Riera

Re: [FFmpeg-devel] [PATCH 04/13] Remove --mips64r6 configure option

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 04/13] Remove --mips64r6 configure option Having a configure option with the

Re: [FFmpeg-devel] [PATCH 07/13] MIPS i6400 core is not mips32r2, so disable mips32r2

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 07/13] MIPS i6400 core is not mips32r2, so disable mips32r2 Signed-off-by: Vi

Re: [FFmpeg-devel] [PATCH 03/13] Add MIPS p5600 core optimizations

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 03/13] Add MIPS p5600 core optimizations Signed-off-by: Vicente Olivert Riera

Re: [FFmpeg-devel] [PATCH 02/13] Remove --mips32r5 configure option

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 02/13] Remove --mips32r5 configure option Having a configure option with the

Re: [FFmpeg-devel] [PATCH 01/13] Rename mipsdspr1 to mipsdsp

2015-11-17 Thread Nedeljko Babic
LGTM Thanks, Nedeljko Od: Vincent Olivert Riera Poslato: 16. novembar 2015 15:57 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; Shivraj Patil; Vincent Olivert Riera Tema: [PATCH 01/13] Rename mipsdspr1 to mipsdsp Signed-off-by: Vicente Olivert Riera ---

Re: [FFmpeg-devel] [PATCH] ffprobe: do not exit in case the demuxer returns AVERROR(EAGAIN)

2015-11-17 Thread wm4
On Tue, 17 Nov 2015 15:13:02 +0100 Stefano Sabatini wrote: > Fix demuxing of files for which the demuxer returns AVERROR(EAGAIN) at > some point. Also returns error code to the caller in case of different > non EOF error. > --- > ffprobe.c | 11 ++- > 1 file changed, 10 insertions(+), 1

[FFmpeg-devel] [PATCH] avcodec/h264, videotoolbox: do not return invalid frames on failure

2015-11-17 Thread wm4
If videotoolbox_common_end_frame failed, then the AVFrame was returned to the API user with the dummy buffer (in AVFrame.buf[0]) still set, and the decode call indicating success. These "half-set" AVFrames with dummy buffer are a videotoolbox specific hack, because the decoder requires an allocate

[FFmpeg-devel] [PATCH] ffprobe: do not exit in case the demuxer returns AVERROR(EAGAIN)

2015-11-17 Thread Stefano Sabatini
Fix demuxing of files for which the demuxer returns AVERROR(EAGAIN) at some point. Also returns error code to the caller in case of different non EOF error. --- ffprobe.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/ffprobe.c b/ffprobe.c index c304a6d..481ff0b 10

Re: [FFmpeg-devel] [PATCH] Added VideoToolbox H.264 encoder.

2015-11-17 Thread wm4
On Fri, 13 Nov 2015 19:44:24 -0500 Rick Kern wrote: > --- > MAINTAINERS|1 + > configure | 14 + > libavcodec/Makefile|1 + > libavcodec/allcodecs.c |1 + > libavcodec/vtenc.c | 1089 > > 5 files cha