Re: [FFmpeg-devel] [PATCH] fate: add mpdecimate test.

2015-11-10 Thread Nicolas George
Le decadi 20 brumaire, an CCXXIV, Hendrik Leppkes a écrit : > Some mingw32 systems seem to suffer the same fate: > http://fate.ffmpeg.org/report.cgi?time=20151110064021=x86_64-debian-mingw32-gcc-4.6 Thanks for the heads up, I will try to reproduce and understand (the filter does not touch

[FFmpeg-devel] [PATCH] configure: don't let check_inline_asm override explicitly-disabled options

2015-11-10 Thread Rodger Combs
--- configure | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/configure b/configure index f770534..6063281 100755 --- a/configure +++ b/configure @@ -900,8 +900,7 @@ check_inline_asm(){ name="$1" code="$2" shift 2 -disable $name -check_cc "$@"

[FFmpeg-devel] [PATCHv2] avformat: implement SChannel SSP TLS protocol

2015-11-10 Thread Hendrik Leppkes
This implementation does not support TLS listen sockets and loading CA/Certs from files. The Windows API does not support loading PEM certs, and would either require a manual loader or instead be limited to loading Windows PFX certificates TLS listen sockets would have to be implemented quite

Re: [FFmpeg-devel] [PATCH] configure: don't let check_inline_asm override explicitly-disabled options

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 08:54:11AM -0600, Rodger Combs wrote: > --- > configure | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/configure b/configure > index f770534..6063281 100755 > --- a/configure > +++ b/configure > @@ -900,8 +900,7 @@ check_inline_asm(){ >

[FFmpeg-devel] [PATCH] configure: don't let check_inline_asm override explicitly-disabled options

2015-11-10 Thread Rodger Combs
This also prevents check_inline_asm from explicitly disabling failed tests, since otherwise you couldn't chain multiple tests with ||. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index f770534..8361b33 100755 --- a/configure +++

Re: [FFmpeg-devel] [PATCH] avutil/x86/intmath: Disable use of tzcnt on older intel compilers.

2015-11-10 Thread Michael Niedermayer
On Wed, Nov 11, 2015 at 12:21:24AM +1100, Matt Oliver wrote: > It would appear that ICC 12.0.4 doesnt have a definition of the tzcnt > intrinsics which brakes compilation. Current FATE tests show that atleast > version 12.1.6 has the required intrinsic however im not certain at what > version

Re: [FFmpeg-devel] [PATCHv2] avformat: implement SChannel SSP TLS protocol

2015-11-10 Thread Hendrik Leppkes
On Tue, Nov 10, 2015 at 5:07 PM, Hendrik Leppkes wrote: > This implementation does not support TLS listen sockets and loading > CA/Certs from files. > > The Windows API does not support loading PEM certs, and would either > require a manual loader or instead be limited to

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/aacsbr: Use FLOAT_0

2015-11-10 Thread Hendrik Leppkes
On Tue, Nov 10, 2015 at 1:51 PM, Michael Niedermayer wrote: > On Tue, Nov 10, 2015 at 11:29:43AM +0100, Hendrik Leppkes wrote: >> This change has broken building on VS2012. >> >> http://fate.ffmpeg.org/report.cgi?time=20151110073635=x86_32-msvc11-windows-native >> >>

[FFmpeg-devel] [PATCH] brstm: reject negative sample rate

2015-11-10 Thread Andreas Cadhalpun
A negative sample rate causes assertion failures in av_rescale_rnd. Signed-off-by: Andreas Cadhalpun --- libavformat/brstm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/brstm.c b/libavformat/brstm.c index 4ac9881..bbdbcef

Re: [FFmpeg-devel] [PATCH v2] avcodec/gifdec: skip data lzw consumed

2015-11-10 Thread Paul B Mahol
On 11/10/15, Ni Hui wrote: > this updated patch fix the return code of avcodec_decode_video2 for gif > decoding > > ff_lzw_decode_tail() now returns the consumed bytes in lzw decompress > and gif frame data buffer is skipped properly > gifdec is the only user of

[FFmpeg-devel] [PATCH] avformat/mov: Add option to ignore chapters during parsing

2015-11-10 Thread Bryan Huh
Chapter-indexing can be expensive since chapters may be interspersed throughout the entire file and may require many seeks - especially costly when consuming a video over a remote protocol like http. Furthermore it is often unnecessary, especially when only trying to get video info (e.g. via

Re: [FFmpeg-devel] [PATCH] brstm: reject negative sample rate

2015-11-10 Thread Paul B Mahol
On 11/10/15, Andreas Cadhalpun wrote: > A negative sample rate causes assertion failures in av_rescale_rnd. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/brstm.c | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [FFmpeg-devel] [PATCH] avutil/softfloat: use abort() instead of av_assert0(0)

2015-11-10 Thread James Almer
On 11/10/2015 9:51 AM, Michael Niedermayer wrote: > On Mon, Nov 09, 2015 at 11:17:52PM -0300, James Almer wrote: >> Fixes compilation of host tool aacps_fixed_tablegen. >> >> Signed-off-by: James Almer >> --- >> See >>

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: implement wrapped_avframe decoder

2015-11-10 Thread Paul B Mahol
On 11/10/15, Michael Niedermayer wrote: > On Tue, Nov 10, 2015 at 04:54:17PM +0700, Muhammad Faiz wrote: >> On Mon, Nov 9, 2015 at 11:22 PM, wm4 wrote: >> > On Mon, 9 Nov 2015 08:03:54 -0800 >> > Muhammad Faiz wrote: >> > >> >>

[FFmpeg-devel] [PATCH] lavf/mov: fix parsing QuickTime meta after the first track

2015-11-10 Thread Rodger Combs
I'm not entirely sure why found_hdlr_mdta existed to begin with, so cc-ing Tinglin Liu (who originally wrote the patch) and Derek Buitenhuis (who signed off on it) hoping for some background. If these checks actually do have a purpose, then the `type == MKTAG('m','d','t','a')` check should be

Re: [FFmpeg-devel] [PATCH] brstm: reject negative sample rate

2015-11-10 Thread Andreas Cadhalpun
On 10.11.2015 20:44, Paul B Mahol wrote: > On 11/10/15, Andreas Cadhalpun wrote: >> A negative sample rate causes assertion failures in av_rescale_rnd. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavformat/brstm.c | 2 +-

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: implement wrapped_avframe decoder

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 03:15:50PM +0100, wm4 wrote: > On Tue, 10 Nov 2015 14:31:26 +0100 > Michael Niedermayer wrote: > > > On Tue, Nov 10, 2015 at 04:54:17PM +0700, Muhammad Faiz wrote: > > > On Mon, Nov 9, 2015 at 11:22 PM, wm4 wrote: > > > >

[FFmpeg-devel] [PATCH] qsvenc: Add adaptive_i and adaptive_b toggles

2015-11-10 Thread Will Kelleher
Signed-off-by: Will Kelleher --- libavcodec/qsvenc.c | 2 ++ libavcodec/qsvenc.h | 2 ++ libavcodec/qsvenc_h264.c | 2 ++ 3 files changed, 6 insertions(+) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index df1f777..5e6ace1 100644 ---

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: implement wrapped_avframe decoder

2015-11-10 Thread wm4
On Tue, 10 Nov 2015 22:42:31 +0100 Michael Niedermayer wrote: > On Tue, Nov 10, 2015 at 03:15:50PM +0100, wm4 wrote: > > On Tue, 10 Nov 2015 14:31:26 +0100 > > Michael Niedermayer wrote: > > > > > On Tue, Nov 10, 2015 at 04:54:17PM +0700,

[FFmpeg-devel] [PATCH 1/2] ffmpeg: set muxer packet duration based on framerate only for CFR

2015-11-10 Thread Michael Niedermayer
From: Michael Niedermayer a set ost->frame_rate does not imply CFR in ffmpeg The changed fate tests had all wrong packet durations (like 1/1000 or 1/9) There might be more cases in which is_cfr could be set Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] avformat/mov: Add option to ignore chapters during parsing

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 11:11:26AM -0800, Bryan Huh wrote: > Chapter-indexing can be expensive since chapters may be interspersed > throughout the entire file and may require many seeks - especially > costly when consuming a video over a remote protocol like http. > Furthermore it is often

[FFmpeg-devel] [PATCH 2/4] dds: validate compressed source buffer size

2015-11-10 Thread Andreas Cadhalpun
A too small buffer will cause segfaults somewhere below decompress_texture_thread. Signed-off-by: Andreas Cadhalpun --- libavcodec/dds.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavcodec/dds.c b/libavcodec/dds.c index 324e665..c918cf0

[FFmpeg-devel] [PATCH 3/4] dds: make sure pallete frame buffer exists before use

2015-11-10 Thread Andreas Cadhalpun
Otherwise it causes a NULL pointer dereference of frame->data[1]. Signed-off-by: Andreas Cadhalpun --- libavcodec/dds.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/dds.c b/libavcodec/dds.c index c918cf0..fe36709 100644 ---

[FFmpeg-devel] [PATCH 4/4] dds: add missing newline to log messages

2015-11-10 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- libavcodec/dds.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/dds.c b/libavcodec/dds.c index fe36709..4d68b33 100644 --- a/libavcodec/dds.c +++ b/libavcodec/dds.c @@ -599,14 +599,14 @@

[FFmpeg-devel] [PATCH] dvdsubdec: validate offset2 similar to offset1

2015-11-10 Thread Andreas Cadhalpun
If it is negative, it causes segmentation faults in decode_rle. Signed-off-by: Andreas Cadhalpun --- libavcodec/dvdsubdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/dvdsubdec.c b/libavcodec/dvdsubdec.c index

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Only store user comment related tags when needed

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 01:01:44PM +, tim nicholson wrote: > On 10/11/15 12:28, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > Also support disabling them as they seem to cause problems to some > > Users. They are also not allowed in IRT D-10 thus

Re: [FFmpeg-devel] [PATCH] dvdsubdec: validate offset2 similar to offset1

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 11:30:05PM +0100, Andreas Cadhalpun wrote: > If it is negative, it causes segmentation faults in decode_rle. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/dvdsubdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

[FFmpeg-devel] [PATCH 2/2] ffmpeg: Print a warning if a pkt duration is already set before using the frame rate

2015-11-10 Thread Michael Niedermayer
From: Michael Niedermayer I didnt find any case that triggers this but if it gets triggered it needs to be investigated Signed-off-by: Michael Niedermayer --- ffmpeg.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/ffmpeg.c b/ffmpeg.c

Re: [FFmpeg-devel] [PATCH v2] avcodec/gifdec: skip data lzw consumed

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 07:45:09PM +0100, Paul B Mahol wrote: > On 11/10/15, Ni Hui wrote: > > this updated patch fix the return code of avcodec_decode_video2 for gif > > decoding > > > > ff_lzw_decode_tail() now returns the consumed bytes in lzw decompress > > and gif

[FFmpeg-devel] [PATCH 1/4] dds: validate source buffer size before copying

2015-11-10 Thread Andreas Cadhalpun
If it is too small av_image_copy_plane segfaults. Signed-off-by: Andreas Cadhalpun --- libavcodec/dds.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavcodec/dds.c b/libavcodec/dds.c index a604d56..324e665 100644 --- a/libavcodec/dds.c +++

[FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-10 Thread Clément Bœsch
From: Clément Bœsch --- So here is a first prototype of a higher level API following an asynchronous model to (for now) simply use the current synchronous API. I suggest to look at libavutil/async.h (sorry no doxy yet) and doc/examples/async_demuxing_decoding.c for an

Re: [FFmpeg-devel] [PATCH] avformat/wavdec: allow data chunk to be before fmt/xma2 chunk

2015-11-10 Thread Paul B Mahol
On 11/9/15, Michael Niedermayer wrote: > On Sun, Nov 08, 2015 at 12:34:21PM +0100, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> libavformat/wavdec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git

[FFmpeg-devel] [PATCH 1/2] avformat/wavdec: parse XMA2 tag

2015-11-10 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavformat/wavdec.c | 58 +--- 1 file changed, 55 insertions(+), 3 deletions(-) diff --git a/libavformat/wavdec.c b/libavformat/wavdec.c index ef24e16..4190fc6 100644 --- a/libavformat/wavdec.c

[FFmpeg-devel] [PATCH 2/2] avformat/wavdec: allow data chunk to be before fmt/xma2 chunk

2015-11-10 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavformat/wavdec.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavformat/wavdec.c b/libavformat/wavdec.c index 4190fc6..d95596f 100644 --- a/libavformat/wavdec.c +++ b/libavformat/wavdec.c @@ -379,7 +379,7 @@

Re: [FFmpeg-devel] [PATCH 3/3] web/documentation: Simplify Components Documentation links

2015-11-10 Thread Alexander Strasser
Am 6. September 2014 19:23:25 MESZ, schrieb "Clément Bœsch" : >On Sat, Sep 06, 2014 at 09:12:47AM -0800, Lou Logan wrote: >> On Sat, Sep 6, 2014, at 04:37 AM, db0company wrote: >> > --- >> > src/documentation | 18 +- >> > 1 files changed, 9 insertions(+), 9

Re: [FFmpeg-devel] [PATCH 2/2] avdevice/lavfi: use wrapped_avframe

2015-11-10 Thread Muhammad Faiz
On Mon, Nov 9, 2015 at 11:05 PM, Muhammad Faiz wrote: > fix ticket #4985 > avoid memory copying from AVFrame to rawvideo > > benchmark: > time ffmpeg -f lavfi -i "amovie=audio.mp3, showcqt" -f null -y /dev/null > old: > real1m3.766s > user1m3.371s > sys

[FFmpeg-devel] [PATCH] avformat/mov: Add option to ignore chapters during parsing

2015-11-10 Thread Bryan Huh
Chapter-indexing can be expensive since chapters may be interspersed throughout the entire file and may require many seeks - especially costly when consuming a video over a remote protocol like http. Furthermore it is often unnecessary, especially when only trying to get video info (e.g. via

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: implement wrapped_avframe decoder

2015-11-10 Thread Muhammad Faiz
On Mon, Nov 9, 2015 at 11:22 PM, wm4 wrote: > On Mon, 9 Nov 2015 08:03:54 -0800 > Muhammad Faiz wrote: > >> From 4dcbda2e585404d2d79d5afcdc13fcb699f6f158 Mon Sep 17 00:00:00 2001 >> From: Muhammad Faiz >> Date: Mon, 9 Nov 2015 15:55:13

Re: [FFmpeg-devel] [PATCH] avformat/mov: Add option to ignore chapters during parsing

2015-11-10 Thread Moritz Barsnick
On Tue, Nov 10, 2015 at 01:12:29 -0800, Bryan Huh wrote: > @@ -5038,6 +5041,8 @@ static const AVOption mov_options[] = { > 0, 1, FLAGS}, > {"ignore_editlist", "", OFFSET(ignore_editlist), AV_OPT_TYPE_INT, {.i64 > = 0}, > 0, 1, FLAGS}, > +{"ignore_chapters", "",

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/aacsbr: Use FLOAT_0

2015-11-10 Thread Hendrik Leppkes
This change has broken building on VS2012. http://fate.ffmpeg.org/report.cgi?time=20151110073635=x86_32-msvc11-windows-native libavcodec/aacsbr_fixed.c(400) : error C2440: 'initializing' : cannot convert from 'const SoftFloat' to 'int32_t' - Hendrik

Re: [FFmpeg-devel] [PATCH] fate: add mpdecimate test.

2015-11-10 Thread Hendrik Leppkes
On Tue, Nov 10, 2015 at 11:32 AM, Hendrik Leppkes wrote: > On Sat, Nov 7, 2015 at 4:12 PM, Nicolas George wrote: >> Le sextidi 6 brumaire, an CCXXIV, Michael Niedermayer a écrit : >>> tested on linux32, 64, mingw32, 64 and mips >>> works fine, LGTM >> >>

Re: [FFmpeg-devel] [PATCH 3/3] web/documentation: Simplify Components Documentation links

2015-11-10 Thread Alexander Strasser
Sorry, please ignore my previous mail! A quirk in my mail setup tricked me into thinking this was new mail. After looking at current documentation link layout I still think we could still avoid the double links (ffmpeg, ffmpeg-full) and just show one entry per tool always linking to full.

Re: [FFmpeg-devel] [PATCH] fate: add mpdecimate test.

2015-11-10 Thread Hendrik Leppkes
On Sat, Nov 7, 2015 at 4:12 PM, Nicolas George wrote: > Le sextidi 6 brumaire, an CCXXIV, Michael Niedermayer a écrit : >> tested on linux32, 64, mingw32, 64 and mips >> works fine, LGTM > > Pushed, thanks. > > Regards, > The test appears to be broken on VS2013

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: implement wrapped_avframe decoder

2015-11-10 Thread wm4
On Tue, 10 Nov 2015 16:54:17 +0700 Muhammad Faiz wrote: > On Mon, Nov 9, 2015 at 11:22 PM, wm4 wrote: > > On Mon, 9 Nov 2015 08:03:54 -0800 > > Muhammad Faiz wrote: > > > >> From 4dcbda2e585404d2d79d5afcdc13fcb699f6f158 Mon Sep 17

[FFmpeg-devel] [PATCH] avcodec/gifdec: skip data lzw consumed

2015-11-10 Thread Ni Hui
this commit fix the return code value of avcodec_decode_video2 for gif decoding, which should be the consumed data length. --- libavcodec/gifdec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/gifdec.c b/libavcodec/gifdec.c index 9f2e6eb..5bcb176 100644 ---

[FFmpeg-devel] [PATCH] avformat/mxfenc: Only store user comment related tags when needed

2015-11-10 Thread Michael Niedermayer
From: Michael Niedermayer Also support disabling them as they seem to cause problems to some Users. They are also not allowed in IRT D-10 thus the default for mxf_d10 is not to write them This also decreases the filesize when no user comment are stored Signed-off-by:

Re: [FFmpeg-devel] [PATCH] avutil/softfloat: use abort() instead of av_assert0(0)

2015-11-10 Thread Michael Niedermayer
On Mon, Nov 09, 2015 at 11:17:52PM -0300, James Almer wrote: > Fixes compilation of host tool aacps_fixed_tablegen. > > Signed-off-by: James Almer > --- > See > http://fate.ffmpeg.org/report.cgi?time=20151108011316=x86_64-openbsd5.6-gcc4.2-conf2 > > libavutil/softfloat.h |

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Only store user comment related tags when needed

2015-11-10 Thread tim nicholson
On 10/11/15 12:28, Michael Niedermayer wrote: > From: Michael Niedermayer > > Also support disabling them as they seem to cause problems to some > Users. They are also not allowed in IRT D-10 thus the default for > mxf_d10 is not to write them > > This also decreases the

Re: [FFmpeg-devel] [PATCH] avcodec/gifdec: skip data lzw consumed

2015-11-10 Thread nihui
At 2015-11-10 06:41:47, "Michael Niedermayer" wrote: >On Sat, Nov 07, 2015 at 10:23:17PM +0800, Ni Hui wrote: >> fix the return code value of avcodec_decode_video2 for gif decoding, which >> should be the consumed data length. >> >> --- >> libavcodec/gifdec.c | 2 ++ >>

Re: [FFmpeg-devel] [PATCH] avcodec/gifdec: skip data lzw consumed

2015-11-10 Thread Paul B Mahol
On 11/10/15, nihui wrote: > > At 2015-11-10 06:41:47, "Michael Niedermayer" > wrote: >>On Sat, Nov 07, 2015 at 10:23:17PM +0800, Ni Hui wrote: >>> fix the return code value of avcodec_decode_video2 for gif decoding, >>> which should be the consumed

Re: [FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-10 Thread Hendrik Leppkes
On Wed, Nov 11, 2015 at 1:27 AM, Clément Bœsch wrote: > From: Clément Bœsch > > --- > > So here is a first prototype of a higher level API following an > asynchronous model to (for now) simply use the current synchronous API. > > I suggest to look at

[FFmpeg-devel] [PATCH v2] avcodec/gifdec: skip data lzw consumed

2015-11-10 Thread Ni Hui
this updated patch fix the return code of avcodec_decode_video2 for gif decoding ff_lzw_decode_tail() now returns the consumed bytes in lzw decompress and gif frame data buffer is skipped properly gifdec is the only user of ff_lzw_decode_tail() my usecase tested and the return code problem got

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Only store user comment related tags when needed

2015-11-10 Thread Michael Niedermayer
On Mon, Nov 09, 2015 at 07:38:17PM +0100, Marton Balint wrote: > > On Mon, 9 Nov 2015, Michael Niedermayer wrote: > > >From: Michael Niedermayer > > > >Also support disabling them as they seem to cause problems to some > >Users. > > > >Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Only store user comment related tags when needed

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 07:37:28AM +, tim nicholson wrote: > On 09/11/15 17:56, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > Also support disabling them as they seem to cause problems to some > > Users. > > > > [..] > > -for (i = 0; i <

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/aacsbr: Use FLOAT_0

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 11:29:43AM +0100, Hendrik Leppkes wrote: > This change has broken building on VS2012. > > http://fate.ffmpeg.org/report.cgi?time=20151110073635=x86_32-msvc11-windows-native > > libavcodec/aacsbr_fixed.c(400) : error C2440: 'initializing' : cannot > convert from 'const

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: implement wrapped_avframe decoder

2015-11-10 Thread wm4
On Tue, 10 Nov 2015 14:31:26 +0100 Michael Niedermayer wrote: > On Tue, Nov 10, 2015 at 04:54:17PM +0700, Muhammad Faiz wrote: > > On Mon, Nov 9, 2015 at 11:22 PM, wm4 wrote: > > > On Mon, 9 Nov 2015 08:03:54 -0800 > > > Muhammad Faiz

Re: [FFmpeg-devel] [PATCH] dvdsubdec: validate offset2 similar to offset1

2015-11-10 Thread Andreas Cadhalpun
On 11.11.2015 00:20, Michael Niedermayer wrote: > On Tue, Nov 10, 2015 at 11:30:05PM +0100, Andreas Cadhalpun wrote: >> If it is negative, it causes segmentation faults in decode_rle. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/dvdsubdec.c | 2 +-

Re: [FFmpeg-devel] [PATCH] XMA1 and XMA2 stereo decoders

2015-11-10 Thread Michael Niedermayer
On Wed, Nov 04, 2015 at 02:18:17PM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol [...] > diff --git a/libavformat/riffdec.c b/libavformat/riffdec.c > index 26779e1..5022312 100644 > --- a/libavformat/riffdec.c > +++ b/libavformat/riffdec.c > @@ -99,10 +99,12 @@ int

[FFmpeg-devel] [PATCH 1/2] lavu/utils: add av_parse_iso8601_tz; use it in time parsing functions

2015-11-10 Thread Rodger Combs
--- libavformat/utils.c| 20 libavutil/parseutils.c | 43 +-- libavutil/parseutils.h | 18 +++--- 3 files changed, 68 insertions(+), 13 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index

Re: [FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-10 Thread Zhang Rui
2015-11-11 8:27 GMT+08:00 Clément Bœsch : > From: Clément Bœsch > > --- > > So here is a first prototype of a higher level API following an > asynchronous model to (for now) simply use the current synchronous API. > > I suggest to look at libavutil/async.h

[FFmpeg-devel] [PATCH 2/2] lavf/mov: strip com.apple.quicktime prefix in meta; parse creation date

2015-11-10 Thread Rodger Combs
--- libavformat/mov.c | 16 1 file changed, 16 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 8fa0189..f4eabce 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -41,6 +41,7 @@ #include "libavutil/aes.h" #include "libavutil/sha.h" #include

[FFmpeg-devel] [PATCH] avutil/x86/intmath: Disable use of tzcnt on older intel compilers.

2015-11-10 Thread Matt Oliver
It would appear that ICC 12.0.4 doesnt have a definition of the tzcnt intrinsics which brakes compilation. Current FATE tests show that atleast version 12.1.6 has the required intrinsic however im not certain at what version number it was introduced (I could not find any relevant info in the intel

Re: [FFmpeg-devel] [PATCH 1/2] avformat/wavdec: parse XMA2 tag

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 09:20:27AM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavformat/wavdec.c | 58 > +--- > 1 file changed, 55 insertions(+), 3 deletions(-) LGTM thanks [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 2/2] avformat/wavdec: allow data chunk to be before fmt/xma2 chunk

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 09:20:28AM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavformat/wavdec.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) should be ok thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: implement wrapped_avframe decoder

2015-11-10 Thread Michael Niedermayer
On Tue, Nov 10, 2015 at 04:54:17PM +0700, Muhammad Faiz wrote: > On Mon, Nov 9, 2015 at 11:22 PM, wm4 wrote: > > On Mon, 9 Nov 2015 08:03:54 -0800 > > Muhammad Faiz wrote: > > > >> From 4dcbda2e585404d2d79d5afcdc13fcb699f6f158 Mon Sep 17 00:00:00 2001 >

[FFmpeg-devel] [PATCH] all: use predefined mathematics macros

2015-11-10 Thread Ganesh Ajjanagadde
This uses M_SQRT2, M_PI, and M_E instead of the actual literals. Benefits include: 1. Reduced scope for copy/paste errors and improved readability. 2. Consistency across the codebase. 3. Actually fixes an incorrect sqrt(2) in avcodec/ppc. 4. Greater precision in avcodec/ac3. Patch tested with