Re: [FFmpeg-devel] [PATCH 5/5] dnxhddec: replace some 0s by DNXHD_VARIABLE

2015-10-04 Thread tim nicholson
On 02/10/15 20:00, Christophe Gisquet wrote: > A series of 0 in a table can be confusing, and the corresponding checks > strange, so using a macro instead of that magic is more readable. > --- > libavcodec/dnxhddata.c | 10 +- > libavcodec/dnxhddata.h | 3 +++ > libavcodec/dnxhddec.c |

[FFmpeg-devel] [PATCH] avutil/attributes: add av_warn_unused_result

2015-10-04 Thread Ganesh Ajjanagadde
GCC 3.4 introduced an attribute warn_unused_result to warn when a programmer discards the return value. Applying this judiciously across the codebase can help in fixing a lot of problems. At a high level, functions which return error codes should always be checked. More concretely, consider the fu

Re: [FFmpeg-devel] [PATCH] avcodec/pngdec: initialize "foreground_alpha"

2015-10-04 Thread Ganesh Ajjanagadde
On Sun, Oct 4, 2015 at 11:08 PM, Michael Niedermayer wrote: > On Sun, Oct 04, 2015 at 10:39:26PM -0400, Ganesh Ajjanagadde wrote: >> On Sun, Oct 4, 2015 at 10:16 PM, Michael Niedermayer >> wrote: >> > On Sun, Oct 04, 2015 at 09:21:55PM -0400, Ganesh Ajjanagadde wrote: >> >> Fixes CID 1322359, CI

Re: [FFmpeg-devel] [PATCH] avcodec/pngdec: initialize "foreground_alpha"

2015-10-04 Thread Michael Niedermayer
On Sun, Oct 04, 2015 at 10:39:26PM -0400, Ganesh Ajjanagadde wrote: > On Sun, Oct 4, 2015 at 10:16 PM, Michael Niedermayer wrote: > > On Sun, Oct 04, 2015 at 09:21:55PM -0400, Ganesh Ajjanagadde wrote: > >> Fixes CID 1322359, CID 1322358. > >> > >> Signed-off-by: Ganesh Ajjanagadde > >> --- > >>

Re: [FFmpeg-devel] [PATCH] avcodec/pngdec: initialize "foreground_alpha"

2015-10-04 Thread Ganesh Ajjanagadde
On Sun, Oct 4, 2015 at 10:16 PM, Michael Niedermayer wrote: > On Sun, Oct 04, 2015 at 09:21:55PM -0400, Ganesh Ajjanagadde wrote: >> Fixes CID 1322359, CID 1322358. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/pngdec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >

Re: [FFmpeg-devel] [PATCH] avcodec/pngdec: initialize "foreground_alpha"

2015-10-04 Thread Michael Niedermayer
On Sun, Oct 04, 2015 at 09:21:55PM -0400, Ganesh Ajjanagadde wrote: > Fixes CID 1322359, CID 1322358. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/pngdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c > index d180

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/alacdec: split off decorrelate_stereo and append_extra_bits as alacdsp

2015-10-04 Thread Michael Niedermayer
On Sun, Oct 04, 2015 at 11:51:58PM +, Timothy Gu wrote: > On Sun, Oct 4, 2015 at 2:21 PM Michael Niedermayer wrote: > > > there is a spec ? > > > > Don't think so for the codec itself. > > > > i only know of soe reference source code and i looked yesterday > > and it usesed the full 8 bits

Re: [FFmpeg-devel] [PATCH]vc1 Autodetection

2015-10-04 Thread Michael Niedermayer
On Sun, Oct 04, 2015 at 11:47:59PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached is a patch implementing vc-1 autodetection. > > Please review, Carl Eugen > Makefile |2 - > rawdec.c |4 --- > vc1dec.c | 71 > ++ > ve

[FFmpeg-devel] [PATCH] avcodec/pngdec: initialize "foreground_alpha"

2015-10-04 Thread Ganesh Ajjanagadde
Fixes CID 1322359, CID 1322358. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/pngdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index d180141..fe5 100644 --- a/libavcodec/pngdec.c +++ b/libavcodec/pngdec.c @@ -1006,7 +10

Re: [FFmpeg-devel] [PATCHv2] avcodec/dvenc: silence -Wdiscarded-qualifiers

2015-10-04 Thread Ganesh Ajjanagadde
On Sun, Sep 20, 2015 at 9:09 AM, Ganesh Ajjanagadde wrote: > s->frame is non-const, so this casts them explicitly. > This suppresses -Wdiscarded-qualifiers seen in e.g > http://fate.ffmpeg.org/log.cgi?time=20150919100330&log=compile&slot=x86_64-archlinux-gcc-enableshared. > > Signed-off-by: Ganesh

Re: [FFmpeg-devel] [PATCHv2] ffplay: more robust mutex creation

2015-10-04 Thread Ganesh Ajjanagadde
On Sun, Oct 4, 2015 at 6:47 PM, Marton Balint wrote: > > > On Sun, 4 Oct 2015, Ganesh Ajjanagadde wrote: > >> On Sun, Oct 4, 2015 at 12:47 PM, Marton Balint wrote: >>> >>> >>> >>> On Sun, 4 Oct 2015, Ganesh Ajjanagadde wrote: >>> SDL_CreateMutex can fail: https://wiki.libsdl.org/SDL_Cre

[FFmpeg-devel] [PATCH 3/3] ffplay: use correct context for av_log

2015-10-04 Thread Ganesh Ajjanagadde
Recent commits c3e8de1c248f8c742dd9e61a0c71ee56bba22c28 and 8dc6e92c3dc67a85026f3010045c7a28b1c0adc8 used av_log incorrectly. This fixes such usage. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ffplay.c b/ffplay.c index

[FFmpeg-devel] [PATCH 2/3] ffplay: more robust condition variable creation

2015-10-04 Thread Ganesh Ajjanagadde
SDL_CreateCond can fail: https://wiki.libsdl.org/SDL_CreateCond. This patch makes creation more robust in one instance. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index 8c9f6b1..cf0640e 100644 --- a/f

[FFmpeg-devel] [PATCH 1/3] ffplay: log SDL error messages

2015-10-04 Thread Ganesh Ajjanagadde
This logs the SDL error messages on failure of creation of SDL_CreateMutex, SDL_CreateCond, and SDL_CreateThread. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/ffplay.c b/ffplay.c index 3223ceb..8c9f6b1 100644 --

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/alacdec: split off decorrelate_stereo and append_extra_bits as alacdsp

2015-10-04 Thread Timothy Gu
On Sun, Oct 4, 2015 at 2:21 PM Michael Niedermayer wrote: > there is a spec ? > Don't think so for the codec itself. > i only know of soe reference source code and i looked yesterday > and it usesed the full 8 bits unless i missed something > https://alac.macosforge.org/trac/browser/trunk/co

[FFmpeg-devel] [PATCH] checkasm: add alacdsp tests

2015-10-04 Thread James Almer
Signed-off-by: James Almer --- tests/checkasm/Makefile | 1 + tests/checkasm/alacdsp.c | 119 ++ tests/checkasm/checkasm.c | 3 ++ tests/checkasm/checkasm.h | 1 + 4 files changed, 124 insertions(+) create mode 100644 tests/checkasm/alacdsp.c

Re: [FFmpeg-devel] [PATCHv2] ffplay: more robust mutex creation

2015-10-04 Thread Marton Balint
On Sun, 4 Oct 2015, Ganesh Ajjanagadde wrote: On Sun, Oct 4, 2015 at 12:47 PM, Marton Balint wrote: On Sun, 4 Oct 2015, Ganesh Ajjanagadde wrote: SDL_CreateMutex can fail: https://wiki.libsdl.org/SDL_CreateMutex. This patch makes creation more robust in one instance. Signed-off-by: Gane

Re: [FFmpeg-devel] [PATCH] concatdec: fix metadata memleak on error

2015-10-04 Thread Marton Balint
On Sun, 4 Oct 2015, Nicolas George wrote: Le duodi 12 vendémiaire, an CCXXIV, Marton Balint a écrit : Fixes Coverity CID 1323077. Signed-off-by: Marton Balint --- libavformat/concatdec.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) Sorry, missed it in my inbox. LGTM to

Re: [FFmpeg-devel] [PATCH] avfilter/vf_stereo3d: add x86 SIMD for anaglyph outputs

2015-10-04 Thread Ronald S. Bultje
Hi, On Sun, Oct 4, 2015 at 3:46 PM, Paul B Mahol wrote: > +.loop: > +movd m10, [ana_matrix_rq+ 0] > +movd m11, [ana_matrix_rq+ 4] > +movd m12, [ana_matrix_rq+ 8] > +movd m13, [ana_matrix_rq+12] >

[FFmpeg-devel] [PATCH]vc1 Autodetection

2015-10-04 Thread Carl Eugen Hoyos
Hi! Attached is a patch implementing vc-1 autodetection. Please review, Carl Eugen diff --git a/libavformat/Makefile b/libavformat/Makefile index 2971912..d118019 100644 --- a/libavformat/Makefile +++ b/libavformat/Makefile @@ -440,7 +440,7 @@ OBJS-$(CONFIG_TTA_DEMUXER) += tta.o ape

Re: [FFmpeg-devel] [PATCH] libavformat/tls_securetransport: fix argument evalulation order UB

2015-10-04 Thread Michael Niedermayer
On Sun, Oct 04, 2015 at 02:37:52PM -0500, Rodger Combs wrote: > --- > libavformat/tls_securetransport.c | 10 ++ > 1 file changed, 6 insertions(+), 4 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically faster algo

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/alacdec: split off decorrelate_stereo and append_extra_bits as alacdsp

2015-10-04 Thread Michael Niedermayer
On Sun, Oct 04, 2015 at 01:34:52PM -0300, James Almer wrote: > On 10/4/2015 7:35 AM, Michael Niedermayer wrote: > > On Sun, Oct 04, 2015 at 01:31:27AM -0300, James Almer wrote: > >>> +if (decorr_left_weight) { > >> > >> So while i was writing a checkasm unit i started looking at the possibl

Re: [FFmpeg-devel] [PATCH] checkasm: add alacdsp tests

2015-10-04 Thread Henrik Gramner
On Sun, Oct 4, 2015 at 8:45 PM, James Almer wrote: > +#define randomize_buffers()\ > +do { \ > +int i; \ > +for (i = 0; i < BUF_SIZE * MAX_CHANNELS; i +

[FFmpeg-devel] [PATCH] avfilter/vf_stereo3d: add x86 SIMD for anaglyph outputs

2015-10-04 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/stereo3d.h | 36 libavfilter/vf_stereo3d.c | 70 --- libavfilter/x86/Makefile | 2 + libavfilter/x86/vf_stereo3d.asm| 172 + libavfilter/x86/vf_stereo3d_init

[FFmpeg-devel] [PATCH] libavformat/tls_securetransport: fix argument evalulation order UB

2015-10-04 Thread Rodger Combs
--- libavformat/tls_securetransport.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/libavformat/tls_securetransport.c b/libavformat/tls_securetransport.c index 73662d7..6ad266a 100644 --- a/libavformat/tls_securetransport.c +++ b/libavformat/tls_securetransport.c

Re: [FFmpeg-devel] [PATCH]select attribute of tee pseudo demuxer may contain multiple stream specifiers

2015-10-04 Thread Bodecs Bela
2015.10.04. 20:06 keltezéssel, Nicolas George írta: Le tridi 13 vendémiaire, an CCXXIV, Bodecs Bela a écrit : git format-patch -n -o /tmp/ --attach origin Thanks. Any particular reason to use --attach? It is not a big issue, just a matter of curiosity. I thought I should use it this way to b

[FFmpeg-devel] [PATCH] checkasm: add alacdsp tests

2015-10-04 Thread James Almer
Signed-off-by: James Almer --- tests/checkasm/Makefile | 1 + tests/checkasm/alacdsp.c | 115 ++ tests/checkasm/checkasm.c | 3 ++ tests/checkasm/checkasm.h | 1 + 4 files changed, 120 insertions(+) create mode 100644 tests/checkasm/alacdsp.c

[FFmpeg-devel] [PATCH] checkasm: add alacdsp tests

2015-10-04 Thread James Almer
Signed-off-by: James Almer --- tests/checkasm/Makefile | 1 + tests/checkasm/alacdsp.c | 114 ++ tests/checkasm/checkasm.c | 3 ++ tests/checkasm/checkasm.h | 1 + 4 files changed, 119 insertions(+) create mode 100644 tests/checkasm/alacdsp.c

Re: [FFmpeg-devel] [PATCH]select attribute of tee pseudo demuxer may contain multiple stream specifiers

2015-10-04 Thread Nicolas George
Le tridi 13 vendémiaire, an CCXXIV, Bodecs Bela a écrit : > git format-patch -n -o /tmp/ --attach origin Thanks. Any particular reason to use --attach? It is not a big issue, just a matter of curiosity. > >>@@ -172,15 +175,26 @@ static int open_slave(AVFormatContext *avf, char > >>*slave, TeeSl

Re: [FFmpeg-devel] [PATCH] concatdec: fix metadata memleak on error

2015-10-04 Thread Nicolas George
Le duodi 12 vendémiaire, an CCXXIV, Marton Balint a écrit : > Fixes Coverity CID 1323077. > > Signed-off-by: Marton Balint > --- > libavformat/concatdec.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) Sorry, missed it in my inbox. LGTM too. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH] concatdec: fix metadata memleak on error

2015-10-04 Thread Ronald S. Bultje
Hi, On Sat, Oct 3, 2015 at 5:59 PM, Marton Balint wrote: > Fixes Coverity CID 1323077. > > Signed-off-by: Marton Balint > --- > libavformat/concatdec.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c > index

Re: [FFmpeg-devel] [PATCH]select attribute of tee pseudo demuxer may contain multiple stream specifiers

2015-10-04 Thread Bodecs Bela
See my interleaved comments below. 2015.10.04. 18:50 keltezéssel, Nicolas George írta: Le decadi 10 vendémiaire, an CCXXIV, Bodecs Bela a écrit : thank you for your feedback, I have altered the patch accordingly. I have enclosed the updated patch file. Please remember not to top-post on these

Re: [FFmpeg-devel] [PATCH] avcodec: remove old vdpau decoder implementation

2015-10-04 Thread compn
On Sun, 4 Oct 2015 09:27:05 -0700 Philip Langdale wrote: > On Sun, 4 Oct 2015 09:25:23 -0400 > compn wrote: > > carl is maintainer of this vdpau code, wm4. > > its right there in MAINTAINERS. > > As am I, for better or worse. > > Perhaps Carl and I can talk about this at the GSoC summit. I sup

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/alacdec: split off decorrelate_stereo and append_extra_bits as alacdsp

2015-10-04 Thread James Almer
On 10/4/2015 8:04 AM, Michael Niedermayer wrote: > On Sun, Oct 04, 2015 at 12:44:56AM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavcodec/Makefile | 2 +- >> libavcodec/alac.c| 65 >> +++- >> libavcodec/alacdsp.c | 57

Re: [FFmpeg-devel] [PATCHv2] ffplay: more robust mutex creation

2015-10-04 Thread Ganesh Ajjanagadde
On Sun, Oct 4, 2015 at 12:47 PM, Marton Balint wrote: > > > On Sun, 4 Oct 2015, Ganesh Ajjanagadde wrote: > >> SDL_CreateMutex can fail: >> https://wiki.libsdl.org/SDL_CreateMutex. >> This patch makes creation more robust in one instance. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> ffplay.c

Re: [FFmpeg-devel] [PATCH]select attribute of tee pseudo demuxer may contain multiple stream specifiers

2015-10-04 Thread Nicolas George
Le decadi 10 vendémiaire, an CCXXIV, Bodecs Bela a écrit : > thank you for your feedback, I have altered the patch accordingly. I have > enclosed the updated patch file. Please remember not to top-post on these mailing-lists. See interleaved comments below. > From bcbd5e3e1a850fef1002d3a63c06fc5

Re: [FFmpeg-devel] [PATCH] avcodec: use HAVE_THREADS header guards to silence -Wunused-function

2015-10-04 Thread Ronald S. Bultje
Hi, On Sat, Oct 3, 2015 at 2:25 PM, Michael Niedermayer wrote: > On Sat, Oct 03, 2015 at 11:13:35AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Thu, Sep 17, 2015 at 7:51 AM, Michael Niedermayer > > wrote: > > > > > On Thu, Sep 17, 2015 at 06:54:37AM -0400, Ganesh Ajjanagadde wrote: > > >

Re: [FFmpeg-devel] [PATCHv2] ffplay: more robust mutex creation

2015-10-04 Thread Marton Balint
On Sun, 4 Oct 2015, Ganesh Ajjanagadde wrote: SDL_CreateMutex can fail: https://wiki.libsdl.org/SDL_CreateMutex. This patch makes creation more robust in one instance. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/ffplay.c b/ffplay.c

Re: [FFmpeg-devel] [PATCH] all: add _DEFAULT_SOURCE locally wherever needed

2015-10-04 Thread Ronald S. Bultje
Hi, On Sun, Oct 4, 2015 at 10:27 AM, Ganesh Ajjanagadde wrote: > On Sat, Oct 3, 2015 at 9:07 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Sat, Oct 3, 2015 at 8:47 AM, wm4 wrote: > > > >> On Sat, 3 Oct 2015 07:41:00 -0500 > >> Ganesh Ajjanagadde wrote: > >> > >> > Glibc 2.20 onwards gener

Re: [FFmpeg-devel] modify ffplay for dumpstream in rtsp?

2015-10-04 Thread Nicolas George
Le duodi 12 vendémiaire, an CCXXIV, compn a écrit : > he wants to play and dump at the same time. like vlc can. > > should this functionality be added to ffmpeg or ffplay? ffmpeg can already display video on the fly, with the opengl, xv and sdl devices. Regards, -- Nicolas George signatur

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/alacdec: split off decorrelate_stereo and append_extra_bits as alacdsp

2015-10-04 Thread James Almer
On 10/4/2015 7:35 AM, Michael Niedermayer wrote: > On Sun, Oct 04, 2015 at 01:31:27AM -0300, James Almer wrote: >>> +if (decorr_left_weight) { >> >> So while i was writing a checkasm unit i started looking at the possible >> values for this, and found out that apparently it's either 0 or 1.

Re: [FFmpeg-devel] [PATCH] ffplay: log SDL error message

2015-10-04 Thread Ganesh Ajjanagadde
On Sun, Oct 4, 2015 at 12:33 PM, Ganesh Ajjanagadde wrote: > This patch logs the SDL error message on failure of SDL_CreateMutex. > > Signed-off-by: Ganesh Ajjanagadde > --- > ffplay.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/ffplay.c b/ffplay.c > index ad8ffd5

[FFmpeg-devel] [PATCH] ffplay: log SDL error message

2015-10-04 Thread Ganesh Ajjanagadde
This patch logs the SDL error message on failure of SDL_CreateMutex. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index ad8ffd5..c00af91 100644 --- a/ffplay.c +++ b/ffplay.c @@ -3734,8 +3734,10 @@ static

[FFmpeg-devel] [PATCH] ffplay: log SDL error message

2015-10-04 Thread Ganesh Ajjanagadde
This patch logs the SDL error message on failure of SDL_CreateThread. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ffplay.c b/ffplay.c index 03befdc..ad8ffd5 100644 --- a/ffplay.c +++ b/ffplay.c @@ -3182,6 +3182,7 @@ static VideoState *stream

Re: [FFmpeg-devel] [PATCH] ffplay: log SDL error message

2015-10-04 Thread Ganesh Ajjanagadde
On Sun, Oct 4, 2015 at 12:16 PM, Ganesh Ajjanagadde wrote: > This patch logs the SDL error message on failure of creation of > SDL_CreateMutex, SDL_CreateCond. > > Signed-off-by: Ganesh Ajjanagadde > --- > ffplay.c | 8 ++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a

Re: [FFmpeg-devel] [PATCH] avcodec: remove old vdpau decoder implementation

2015-10-04 Thread Philip Langdale
On Sun, 4 Oct 2015 09:25:23 -0400 compn wrote: > carl: if this new api works with kodi and mplayer, whats the problem? > > are there still users of this api in 2015? yes or no? > > carl, do you use this api? what for? to test vdpau? I'm curious about this too. Traditionally, mplayer *is* the l

[FFmpeg-devel] [PATCH] ffplay: more robust condition variable creation

2015-10-04 Thread Ganesh Ajjanagadde
SDL_CreateCond can fail: https://wiki.libsdl.org/SDL_CreateCond. This patch makes creation more robust in one instance. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index 39a1cbe..03befdc 100644 --- a/f

[FFmpeg-devel] [PATCH] ffplay: log SDL error message

2015-10-04 Thread Ganesh Ajjanagadde
This patch logs the SDL error message on failure of creation of SDL_CreateMutex, SDL_CreateCond. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index 84bd6ee..39a1cbe 100644 --- a/ffplay.c +++ b/ffpl

[FFmpeg-devel] [PATCHv2] ffplay: more robust mutex creation

2015-10-04 Thread Ganesh Ajjanagadde
SDL_CreateMutex can fail: https://wiki.libsdl.org/SDL_CreateMutex. This patch makes creation more robust in one instance. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/ffplay.c b/ffplay.c index c5a7895..84bd6ee 100644 --- a/ffplay.c +++

[FFmpeg-devel] [PATCH] ffplay: more robust mutex creation

2015-10-04 Thread Ganesh Ajjanagadde
SDL_CreateMutex can fail: https://wiki.libsdl.org/SDL_CreateMutex. This patch makes creation more robust in one instance. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/ffplay.c b/ffplay.c index da0fd3a..eaf26be 100644 --- a/ffplay.c +++

Re: [FFmpeg-devel] [PATCHv2] ffplay: more robust thread creation

2015-10-04 Thread Marton Balint
On Sat, 3 Oct 2015, Ganesh Ajjanagadde wrote: SDL_CreateThread can fail: https://wiki.libsdl.org/SDL_CreateThread. This patch makes thread creation more robust in one instance. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 16 1 file changed, 12 insertions(+), 4 deletions(-

Re: [FFmpeg-devel] [PATCH] Changelog: add note on ffplay dynamic volume control

2015-10-04 Thread Marton Balint
On Sat, 3 Oct 2015, Ganesh Ajjanagadde wrote: Signed-off-by: Ganesh Ajjanagadde --- Changelog | 1 + 1 file changed, 1 insertion(+) diff --git a/Changelog b/Changelog index 4b18d00..2d1c842 100644 --- a/Changelog +++ b/Changelog @@ -15,6 +15,7 @@ version : - maskedmerge filter - Screenpresso

Re: [FFmpeg-devel] modify ffplay for dumpstream in rtsp?

2015-10-04 Thread Marton Balint
On Sat, 3 Oct 2015, compn wrote: On Sat, 3 Oct 2015 10:45:25 +0200 Nicolas George wrote: Le duodi 12 vendémiaire, an CCXXIV, Jan Panteltje a écrit : > I was thinking about adding a command line option to ffplay to > enable the 'dump to stdout' modification, but am taken aback by why > I do n

Re: [FFmpeg-devel] [PATCH] all: add _DEFAULT_SOURCE locally wherever needed

2015-10-04 Thread Ganesh Ajjanagadde
On Sat, Oct 3, 2015 at 9:07 AM, Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 3, 2015 at 8:47 AM, wm4 wrote: > >> On Sat, 3 Oct 2015 07:41:00 -0500 >> Ganesh Ajjanagadde wrote: >> >> > Glibc 2.20 onwards generates a deprecation warning for usage of >> _BSD_SOURCE and _SVID_SOURCE. >> > The solu

Re: [FFmpeg-devel] [PATCH 1/2] checkasm: Fix compilation with --disable-avcodec

2015-10-04 Thread Henrik Gramner
On Sun, Oct 4, 2015 at 3:33 PM, Ronald S. Bultje wrote: > Just push it, I'm not sure I can come up with an airtight mechanism that is > not mega-overdesigned into oblivion... Applied, thanks, ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://

Re: [FFmpeg-devel] [PATCH 1/2] checkasm: Fix compilation with --disable-avcodec

2015-10-04 Thread Ronald S. Bultje
Hi, On Sun, Oct 4, 2015 at 9:31 AM, Henrik Gramner wrote: > On Thu, Oct 1, 2015 at 8:05 PM, Henrik Gramner wrote: > > On Thu, Oct 1, 2015 at 7:57 PM, Ronald S. Bultje > wrote: > >> Does this mean these macros are defined and 1 when AVCODEC=0? > >> > >> That seems like a bug? > > > > Yes, they

Re: [FFmpeg-devel] [PATCH 1/2] checkasm: Fix compilation with --disable-avcodec

2015-10-04 Thread Henrik Gramner
On Thu, Oct 1, 2015 at 8:05 PM, Henrik Gramner wrote: > On Thu, Oct 1, 2015 at 7:57 PM, Ronald S. Bultje wrote: >> Does this mean these macros are defined and 1 when AVCODEC=0? >> >> That seems like a bug? > > Yes, they are., but I have no idea what the intended behavior of the > CONFIG_* variabl

Re: [FFmpeg-devel] [PATCH] avcodec: remove old vdpau decoder implementation

2015-10-04 Thread wm4
On Sun, 4 Oct 2015 09:25:23 -0400 compn wrote: > On Sun, 4 Oct 2015 14:48:25 +0200 > wm4 wrote: > > > On Sun, 4 Oct 2015 11:31:31 + (UTC) > > Carl Eugen Hoyos wrote: > > > > > wm4 googlemail.com> writes: > > > > > > > On Sat, 3 Oct 2015 22:23:21 +0200 > > > > Carl Eugen Hoyos ag.or.at>

Re: [FFmpeg-devel] [PATCH] avcodec: remove old vdpau decoder implementation

2015-10-04 Thread compn
On Sun, 4 Oct 2015 14:48:25 +0200 wm4 wrote: > On Sun, 4 Oct 2015 11:31:31 + (UTC) > Carl Eugen Hoyos wrote: > > > wm4 googlemail.com> writes: > > > > > On Sat, 3 Oct 2015 22:23:21 +0200 > > > Carl Eugen Hoyos ag.or.at> wrote: > > > > > > > On Saturday 03 October 2015 10:05:29 pm wm4 wr

Re: [FFmpeg-devel] [PATCH] avcodec: remove old vdpau decoder implementation

2015-10-04 Thread Ronald S. Bultje
Hi, On Sat, Oct 3, 2015 at 4:23 PM, Carl Eugen Hoyos wrote: > On Saturday 03 October 2015 10:05:29 pm wm4 wrote: > > Ping. Will push in 24 hours or so if nobody complains. > > The reason I am against this is just that users told me > repeatedly (in person) that they switched from the dark > side

Re: [FFmpeg-devel] [PATCH] avcodec: remove old vdpau decoder implementation

2015-10-04 Thread wm4
On Sun, 4 Oct 2015 11:31:31 + (UTC) Carl Eugen Hoyos wrote: > wm4 googlemail.com> writes: > > > On Sat, 3 Oct 2015 22:23:21 +0200 > > Carl Eugen Hoyos ag.or.at> wrote: > > > > > On Saturday 03 October 2015 10:05:29 pm wm4 wrote: > > > > Ping. Will push in 24 hours or so if nobody complain

Re: [FFmpeg-devel] [RFC] avformat/mxfenc: stop encoding if unfilled video packet

2015-10-04 Thread Tomas Härdin
On Mon, 2015-09-28 at 15:11 +0200, Tobias Rapp wrote: > On 19.09.2015 22:49, Tomas Härdin wrote: > > On Wed, 2015-09-16 at 14:33 +0200, Tobias Rapp wrote: > >> Hi, > >> > >> attached patch fixes ticket #4759 but I guess it is a bit too hasty to > >> always abort transcoding if a single frame cannot

Re: [FFmpeg-devel] [PATCH] avformat/flacdec: support fast-seek

2015-10-04 Thread Michael Niedermayer
On Sun, Oct 04, 2015 at 01:50:45PM +0800, Ching-Yi Chan wrote: > 2015-10-04 7:37 GMT+08:00 Michael Niedermayer : > > > On Sat, Oct 03, 2015 at 01:14:26AM +0800, Ching-Yi Chan wrote: > > > > iam not sure changing the format flags is a great idea, i think no > > other demuxer does that > > that said

Re: [FFmpeg-devel] [PATCH] avcodec: remove old vdpau decoder implementation

2015-10-04 Thread Carl Eugen Hoyos
wm4 googlemail.com> writes: > On Sat, 3 Oct 2015 22:23:21 +0200 > Carl Eugen Hoyos ag.or.at> wrote: > > > On Saturday 03 October 2015 10:05:29 pm wm4 wrote: > > > Ping. Will push in 24 hours or so if nobody complains. > > > > The reason I am against this is just that users told me > > repeate

Re: [FFmpeg-devel] [PATCH] avfilter/f_ebur128: add dualmono measurement option

2015-10-04 Thread Clément Bœsch
On Wed, Sep 30, 2015 at 10:35:08AM -0500, Kyle Swanson wrote: > --- > doc/filters.texi| 9 + > libavfilter/f_ebur128.c | 22 ++ > 2 files changed, 31 insertions(+) > Applied with -Treat mono input files as 'dual mono.' If a mono file is intended for playback

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/alacdec: split off decorrelate_stereo and append_extra_bits as alacdsp

2015-10-04 Thread Michael Niedermayer
On Sun, Oct 04, 2015 at 12:44:56AM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/Makefile | 2 +- > libavcodec/alac.c| 65 > +++- > libavcodec/alacdsp.c | 57 + > libavco

Re: [FFmpeg-devel] [PATCH] libavformat/tls_securetransport: silence uninitialized value warning

2015-10-04 Thread Michael Niedermayer
On Sat, Oct 03, 2015 at 05:31:33PM -0500, Rodger Combs wrote: > --- > libavformat/tls_securetransport.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavformat/tls_securetransport.c > b/libavformat/tls_securetransport.c > index 73662d7..cdc7953 100644 > --- a/lib

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/alacdec: split off decorrelate_stereo and append_extra_bits as alacdsp

2015-10-04 Thread Michael Niedermayer
On Sun, Oct 04, 2015 at 01:31:27AM -0300, James Almer wrote: > On 10/4/2015 12:44 AM, James Almer wrote: > > Signed-off-by: James Almer > > --- > > libavcodec/Makefile | 2 +- > > libavcodec/alac.c| 65 > > +++- > > libavcodec/alacdsp.c | 57

Re: [FFmpeg-devel] [PATCH 8/9] dnxhd: add better support for CIDs 1270 to 1274

2015-10-04 Thread Christophe Gisquet
2015-10-03 18:59 GMT+02:00 Christophe Gisquet : [SNIP] There was a blank space in there (I think). Patch updated -- Christophe 0008-dnxhd-add-better-support-for-CIDs-1270-to-1274.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-de

Re: [FFmpeg-devel] [PATCH 1/9] dnxhddec: cleanup frame header parsing

2015-10-04 Thread Christophe Gisquet
Hi, 2015-10-03 19:50 GMT+02:00 Carl Eugen Hoyos : > Looks unintended... Indeed. Patch updated. -- Christophe 0001-dnxhddec-cleanup-frame-header-parsing.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpe