Re: [FFmpeg-devel] [PATCH] ffplay: more robust mutex, condition variable handling

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 9:35 PM, Ganesh Ajjanagadde wrote: > SDL_CreateMutex and SDL_CreateCond can fail: > https://wiki.libsdl.org/SDL_CreateMutex. > This patch makes handling more robust in one instance. > > Signed-off-by: Ganesh Ajjanagadde > --- > ffplay.c | 23 +-- > 1 f

Re: [FFmpeg-devel] pngdec failing with an assertion

2015-09-27 Thread Timothy Gu
El domingo, 27 de septiembre de 2015, Gonzalo escribió: > > > El 27/09/15 a las 22:45, Gonzalo escribió: > >> $ ffplay /media/Linux/anims/wildlife/wildlife.%04d.png >> ffplay version N-75563-g235381e Copyright (c) 2003-2015 the FFmpeg >> developers >> built with gcc 4.8 (Ubuntu 4.8.4-2ubuntu1~1

Re: [FFmpeg-devel] [PATCH v2] checkasm: Fix the function name sorting algorithm

2015-09-27 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 07:55:06PM +0200, Henrik Gramner wrote: > The previous implementation was behaving incorrectly in some corner cases. > --- > The first version of this patch didn't take care of all possible corner cases, > but I believe this one does. > --- > tests/checkasm/checkasm.c | 8 +

Re: [FFmpeg-devel] pngdec failing with an assertion

2015-09-27 Thread Gonzalo
El 27/09/15 a las 22:45, Gonzalo escribió: $ ffplay /media/Linux/anims/wildlife/wildlife.%04d.png ffplay version N-75563-g235381e Copyright (c) 2003-2015 the FFmpeg developers built with gcc 4.8 (Ubuntu 4.8.4-2ubuntu1~14.04) configuration: --enable-gpl --enable-shared --enable-gray --ena

[FFmpeg-devel] pngdec failing with an assertion

2015-09-27 Thread Gonzalo
$ ffplay /media/Linux/anims/wildlife/wildlife.%04d.png ffplay version N-75563-g235381e Copyright (c) 2003-2015 the FFmpeg developers built with gcc 4.8 (Ubuntu 4.8.4-2ubuntu1~14.04) configuration: --enable-gpl --enable-shared --enable-gray --enable-runtime-cpudetect --enable-bzlib --enable-

Re: [FFmpeg-devel] [PATCH] avformat/movenc: suppress -Wstrict-overflow warnings

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 9:18 PM, Michael Niedermayer wrote: > On Sun, Sep 27, 2015 at 01:23:03PM -0400, Ganesh Ajjanagadde wrote: >> On Sun, Sep 27, 2015 at 12:58 PM, Michael Niedermayer >> wrote: >> > On Sat, Sep 26, 2015 at 10:55:26PM -0400, Ganesh Ajjanagadde wrote: >> >> On Sat, Sep 26, 2015

[FFmpeg-devel] [PATCH] ffplay: more robust mutex, condition variable handling

2015-09-27 Thread Ganesh Ajjanagadde
SDL_CreateMutex and SDL_CreateCond can fail: https://wiki.libsdl.org/SDL_CreateMutex. This patch makes handling more robust in one instance. Signed-off-by: Ganesh Ajjanagadde --- ffplay.c | 23 +-- 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/ffplay.c b/ffpl

Re: [FFmpeg-devel] [PATCH] avformat/movenc: suppress -Wstrict-overflow warnings

2015-09-27 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 01:23:03PM -0400, Ganesh Ajjanagadde wrote: > On Sun, Sep 27, 2015 at 12:58 PM, Michael Niedermayer > wrote: > > On Sat, Sep 26, 2015 at 10:55:26PM -0400, Ganesh Ajjanagadde wrote: > >> On Sat, Sep 26, 2015 at 10:32 PM, Ronald S. Bultje > >> wrote: > >> > Hi, > >> > > >>

[FFmpeg-devel] [PATCH] avcodec/pngdec: mark previous_picture as done on end of decode_frame_common()

2015-09-27 Thread Michael Niedermayer
From: Michael Niedermayer Fixes deadlock with threads Found-by: Paul B Mahol Signed-off-by: Michael Niedermayer --- libavcodec/pngdec.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index ee11f12..743144b 100644 --- a/libavcodec/pngdec.c +++

Re: [FFmpeg-devel] [PATCH 4/7] dnxhddec: use unsafe bitstream reader

2015-09-27 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 10:09:05AM +0200, Christophe Gisquet wrote: > Each line is padded by the format, and errors are now reported and > stop the decoding. Around 5% speedup. > --- > libavcodec/dnxhddec.c | 1 + > 1 file changed, 1 insertion(+) applied thanks [...] -- Michael GnuPG finge

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread James Darnley
On 2015-09-27 23:06, Ganesh Ajjanagadde wrote: > On Sun, Sep 27, 2015 at 5:00 PM, James Darnley > wrote: >> On 2015-09-27 17:43, Stefano Sabatini wrote: >>> On date Sunday 2015-09-27 15:28:50 +0200, James Darnley encoded: On 2015-09-27 14:37, Stefano Sabatini wrote: > + > +TNS

Re: [FFmpeg-devel] [PATCHv4] ffplay: add support for interactive volume control

2015-09-27 Thread Marton Balint
On Sun, 27 Sep 2015, Ganesh Ajjanagadde wrote: This is a feature heavily inspired by the mpv player. At the moment, methods for adjusting volume in ffplay are rather clumsy: either one needs to set it system-wide, or one needs to set it via the volume filter. This patch adds key bindings ident

Re: [FFmpeg-devel] [PATCHv2] doc/ffplay, ffplay: add information regarding volume control

2015-09-27 Thread Marton Balint
On Sun, 27 Sep 2015, Stefano Sabatini wrote: On date Sunday 2015-09-27 12:38:34 -0400, Ganesh Ajjanagadde encoded: ffplay now supports dynamic volume control. This documents the supported behavior. Signed-off-by: Ganesh Ajjanagadde --- doc/ffplay.texi | 9 + ffplay.c| 3 +++

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 5:00 PM, James Darnley wrote: > On 2015-09-27 17:43, Stefano Sabatini wrote: >> On date Sunday 2015-09-27 15:28:50 +0200, James Darnley encoded: >>> On 2015-09-27 14:37, Stefano Sabatini wrote: + +TNS still needs some polishing, but has the potential to redu

Re: [FFmpeg-devel] [PATCH]libx264: Disable global header for AVC-Intra encoding

2015-09-27 Thread Michael Niedermayer
On Sat, Sep 26, 2015 at 02:25:04PM +, Thomas Mundt wrote: > This will fix AVC-Intra encoding in .mov container. > Regards,Thomas > libx264.c |4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > 1d153d16cdb505ea33cab12979de3b0e50e3b459 libx264.diff > libavcodec/libx264.c | 4 +++-

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread James Darnley
On 2015-09-27 17:43, Stefano Sabatini wrote: > On date Sunday 2015-09-27 15:28:50 +0200, James Darnley encoded: >> On 2015-09-27 14:37, Stefano Sabatini wrote: >>> + >>> +TNS still needs some polishing, but has the potential to reduce coding >>> +artifacts by applying noise shaping in the

Re: [FFmpeg-devel] [PATCH]h264: Fix ticket #3147 H264 - Wrong field order

2015-09-27 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 10:11:47PM +0100, Thomas Mundt wrote: > IME-Version: 1.0 > Content-Type: text/plain; charset=UTF-8 > Content-Transfer-Encoding: 8bit > > Default field order to top field first when interlaced frame is detected and > pic_struct_present_flag is not set. > Since bottom field

[FFmpeg-devel] [PATCH]h264: Fix ticket #3147 H264 - Wrong field order

2015-09-27 Thread Thomas Mundt
IME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Default field order to top field first when interlaced frame is detected and pic_struct_present_flag is not set. Since bottom field first comes from the old NTSC standard and is not used with HD anymore I th

Re: [FFmpeg-devel] [PATCH 3/7] dnxhddec: check and report bitstream errors

2015-09-27 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 10:09:04AM +0200, Christophe Gisquet wrote: > This only occur when an overrun in coefficient decoding is > detected. > --- > libavcodec/dnxhddec.c | 48 ++-- > 1 file changed, 34 insertions(+), 14 deletions(-) applied it might b

Re: [FFmpeg-devel] [PATCH]h264: Fix ticket #3147 H264 - Wrong field order

2015-09-27 Thread Thomas Mundt
Seems yahoo emails become scrambled and links are not supported. Sorry! I´m on Win7 64Bit using latest media autobuild suite (msys2) from here:https://github.com/jb-alvarado/media-autobuild_suite I started fate with this command after rsync the testfiles:make fate SAMPLES="D:/ffmpeg/fate-suite

[FFmpeg-devel] [PATCH v2] checkasm: Fix the function name sorting algorithm

2015-09-27 Thread Henrik Gramner
The previous implementation was behaving incorrectly in some corner cases. --- The first version of this patch didn't take care of all possible corner cases, but I believe this one does. --- tests/checkasm/checkasm.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tests

Re: [FFmpeg-devel] [PATCH]h264: Fix ticket #3147 H264 - Wrong field order

2015-09-27 Thread Thomas Mundt
Michael Niedermayer schrieb am 18:04 Sonntag, 27.September 2015: >On Sun, Sep 27, 2015 at 03:24:48PM +, Thomas Mundt wrote: >> Fate always stops at vf_mergeplanes with error, with or without my >> patch:[AVFilterGraph @ 00eb4500] No such filter: '' >> Error initializing complex filters. >>

Re: [FFmpeg-devel] [PATCH 3/3] configure: Disable automatic image base calculation

2015-09-27 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 04:59:25PM +0200, Hendrik Leppkes wrote: > On Mon, Sep 21, 2015 at 6:41 AM, Alex Smith wrote: > > From: Alex Smith > > > > There's no reason for it. ASLR will rebase it regardless so "preventing > > collisions" isn't really relevant. This also brings it in line with > >

Re: [FFmpeg-devel] [PATCH] avformat/movenc: suppress -Wstrict-overflow warnings

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 12:58 PM, Michael Niedermayer wrote: > On Sat, Sep 26, 2015 at 10:55:26PM -0400, Ganesh Ajjanagadde wrote: >> On Sat, Sep 26, 2015 at 10:32 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Sat, Sep 26, 2015 at 7:19 PM, Ganesh Ajjanagadde >> > wrote: >> > >> >> On Sat

Re: [FFmpeg-devel] [PATCH] avformat/movenc: suppress -Wstrict-overflow warnings

2015-09-27 Thread Michael Niedermayer
On Sat, Sep 26, 2015 at 10:55:26PM -0400, Ganesh Ajjanagadde wrote: > On Sat, Sep 26, 2015 at 10:32 PM, Ronald S. Bultje wrote: > > Hi, > > > > On Sat, Sep 26, 2015 at 7:19 PM, Ganesh Ajjanagadde > > wrote: > > > >> On Sat, Sep 26, 2015 at 7:11 PM, Michael Niedermayer > >> wrote: > >> > On Fri,

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread Stefano Sabatini
On date Sunday 2015-09-27 18:26:04 +0200, Moritz Barsnick encoded: > In addition to the others: > > > +effort. We're looking forward participating to the next GSoC > > looking forward to participating in the next > > > +At the end of the program, Mariusz provided a sketch of implementati

Re: [FFmpeg-devel] [PATCHv2] doc/ffplay, ffplay: add information regarding volume control

2015-09-27 Thread Stefano Sabatini
On date Sunday 2015-09-27 12:38:34 -0400, Ganesh Ajjanagadde encoded: > ffplay now supports dynamic volume control. This documents the supported > behavior. > > Signed-off-by: Ganesh Ajjanagadde > --- > doc/ffplay.texi | 9 + > ffplay.c| 3 +++ > 2 files changed, 12 insertions(+)

Re: [FFmpeg-devel] [PATCHv4] ffplay: add support for interactive volume control

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 12:06 PM, Stefano Sabatini wrote: > On date Sunday 2015-09-27 10:41:37 -0400, Ganesh Ajjanagadde encoded: >> This is a feature heavily inspired by the mpv player. At the moment, methods >> for adjusting volume in ffplay are rather clumsy: either one needs to set it >> syste

[FFmpeg-devel] [PATCHv2] doc/ffplay, ffplay: add information regarding volume control

2015-09-27 Thread Ganesh Ajjanagadde
ffplay now supports dynamic volume control. This documents the supported behavior. Signed-off-by: Ganesh Ajjanagadde --- doc/ffplay.texi | 9 + ffplay.c| 3 +++ 2 files changed, 12 insertions(+) diff --git a/doc/ffplay.texi b/doc/ffplay.texi index be696c8..2a35c21 100644 --- a/d

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-27 Thread DeHackEd
Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 4 libavcodec/libx264.c | 46 ++ 2 files changed, 50 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 3550bcc..f2d46dc 100644 --- a/doc/enc

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-27 Thread Marton Balint
On Sun, 27 Sep 2015, Clément Bœsch wrote: On Sun, Sep 27, 2015 at 11:57:41AM -0400, Ganesh Ajjanagadde wrote: On Sun, Sep 27, 2015 at 11:40 AM, Clément Bœsch wrote: On Sun, Sep 27, 2015 at 10:12:52AM -0400, Ganesh Ajjanagadde wrote: [...] Yes, I consider the current usage terrible unless I

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 12:16 PM, Clément Bœsch wrote: > On Sun, Sep 27, 2015 at 11:57:41AM -0400, Ganesh Ajjanagadde wrote: >> On Sun, Sep 27, 2015 at 11:40 AM, Clément Bœsch wrote: >> > On Sun, Sep 27, 2015 at 10:12:52AM -0400, Ganesh Ajjanagadde wrote: >> > [...] >> >> Yes, I consider the curr

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread Moritz Barsnick
In addition to the others: > +effort. We're looking forward participating to the next GSoC looking forward to participating in the next > +At the end of the program, Mariusz provided a sketch of implementation of an implementation > +with overlapping formatting (which other formats

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-27 Thread Clément Bœsch
On Sun, Sep 27, 2015 at 11:57:41AM -0400, Ganesh Ajjanagadde wrote: > On Sun, Sep 27, 2015 at 11:40 AM, Clément Bœsch wrote: > > On Sun, Sep 27, 2015 at 10:12:52AM -0400, Ganesh Ajjanagadde wrote: > > [...] > >> Yes, I consider the current usage terrible unless I am missing > >> something. volume

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 11:56 AM, Stefano Sabatini wrote: > On date Sunday 2015-09-27 09:44:33 -0400, Ganesh Ajjanagadde encoded: >> On Sun, Sep 27, 2015 at 8:37 AM, Stefano Sabatini wrote: >> > --- >> > src/index | 234 >> > ++ >> > 1

Re: [FFmpeg-devel] Getting timestamp from transport stream

2015-09-27 Thread Stefano Sabatini
On date Saturday 2015-09-26 12:48:33 -0700, Sabur Hassan Baidya encoded: > Hi, > > I am a newbie to ffmpeg. I created a transport stream from encoded mp4 file > using ffmpeg command. Now I am trying to send the packets of the transport > stream over UDP. I found that there are ffmpeg commands for

Re: [FFmpeg-devel] [PATCHv4] ffplay: add support for interactive volume control

2015-09-27 Thread Stefano Sabatini
On date Sunday 2015-09-27 10:41:37 -0400, Ganesh Ajjanagadde encoded: > This is a feature heavily inspired by the mpv player. At the moment, methods > for adjusting volume in ffplay are rather clumsy: either one needs to set it > system-wide, or one needs to set it via the volume filter. > > This

Re: [FFmpeg-devel] [PATCH]h264: Fix ticket #3147 H264 - Wrong field order

2015-09-27 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 03:24:48PM +, Thomas Mundt wrote: > Fate always stops at vf_mergeplanes with error, with or without my > patch:[AVFilterGraph @ 00eb4500] No such filter: '' > Error initializing complex filters. > Invalid argument that should not happen, how can this be reproduced ?

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 11:40 AM, Clément Bœsch wrote: > On Sun, Sep 27, 2015 at 10:12:52AM -0400, Ganesh Ajjanagadde wrote: > [...] >> Yes, I consider the current usage terrible unless I am missing >> something. volume filter is static, so once set, I can't adjust the >> volume until I reinsert a

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread Stefano Sabatini
On date Sunday 2015-09-27 09:44:33 -0400, Ganesh Ajjanagadde encoded: > On Sun, Sep 27, 2015 at 8:37 AM, Stefano Sabatini wrote: > > --- > > src/index | 234 > > ++ > > 1 file changed, 234 insertions(+) [...] > > + > > +During the

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread Stefano Sabatini
On date Sunday 2015-09-27 15:28:50 +0200, James Darnley encoded: > On 2015-09-27 14:37, Stefano Sabatini wrote: > > --- > > src/index | 234 > > ++ > > 1 file changed, 234 insertions(+) > > > > diff --git a/src/index b/src/index > > ind

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-27 Thread Clément Bœsch
On Sun, Sep 27, 2015 at 10:12:52AM -0400, Ganesh Ajjanagadde wrote: [...] > Yes, I consider the current usage terrible unless I am missing > something. volume filter is static, so once set, I can't adjust the > volume until I reinsert a new volume filter, etc. I can't simply > adjust the volume lik

Re: [FFmpeg-devel] [PATCH]h264: Fix ticket #3147 H264 - Wrong field order

2015-09-27 Thread Thomas Mundt
Fate always stops at vf_mergeplanes with error, with or without my patch:[AVFilterGraph @ 00eb4500] No such filter: '' Error initializing complex filters. Invalid argument Is there a way to limit fate that it just checks h264 part? Kieran Kunhya schrieb am 13:23 Sonntag, 27.September 2015

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 10:40 AM, Ganesh Ajjanagadde wrote: > On Sun, Sep 27, 2015 at 10:12 AM, Ganesh Ajjanagadde wrote: >> On Sun, Sep 27, 2015 at 9:57 AM, Clément Bœsch wrote: >>> On Sat, Sep 26, 2015 at 11:36:42AM -0400, Ganesh Ajjanagadde wrote: >>> [...] -memcpy(stream, (uint8

Re: [FFmpeg-devel] [PATCH 3/3] configure: Disable automatic image base calculation

2015-09-27 Thread Hendrik Leppkes
On Mon, Sep 21, 2015 at 6:41 AM, Alex Smith wrote: > From: Alex Smith > > There's no reason for it. ASLR will rebase it regardless so "preventing > collisions" isn't really relevant. This also brings it in line with > what a msvc produced dll will have (an image base of 0x1000). > > Signed-

Re: [FFmpeg-devel] [PATCH 2/3 v2] configure: Support for HEASLR on mingw targets

2015-09-27 Thread Hendrik Leppkes
On Sat, Sep 26, 2015 at 8:43 AM, Alex Smith wrote: > From: Alex Smith > > The appropriate flag for HEASLR (--high-entropy-va) was added in > binutils 2.25. > > Also set the image base >4GB so higher entropy gets applied to image > base randomization when used with HEASLR (8 -> 17 bits of > random

[FFmpeg-devel] [PATCHv4] ffplay: add support for interactive volume control

2015-09-27 Thread Ganesh Ajjanagadde
This is a feature heavily inspired by the mpv player. At the moment, methods for adjusting volume in ffplay are rather clumsy: either one needs to set it system-wide, or one needs to set it via the volume filter. This patch adds key bindings identical to the mpv defaults for muting/unmuting and in

Re: [FFmpeg-devel] [PATCH] doc/ffplay: add information regarding volume control

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 10:38 AM, Ganesh Ajjanagadde wrote: > ffplay now supports dynamic volume control. This documents the supported > behavior. > > Signed-off-by: Ganesh Ajjanagadde > --- > doc/ffplay.texi | 9 + > 1 file changed, 9 insertions(+) > > diff --git a/doc/ffplay.texi b/doc

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 10:12 AM, Ganesh Ajjanagadde wrote: > On Sun, Sep 27, 2015 at 9:57 AM, Clément Bœsch wrote: >> On Sat, Sep 26, 2015 at 11:36:42AM -0400, Ganesh Ajjanagadde wrote: >> [...] >>> -memcpy(stream, (uint8_t *)is->audio_buf + is->audio_buf_index, >>> len1); >>> +

[FFmpeg-devel] [PATCH] doc/ffplay: add information regarding volume control

2015-09-27 Thread Ganesh Ajjanagadde
ffplay now supports dynamic volume control. This documents the supported behavior. Signed-off-by: Ganesh Ajjanagadde --- doc/ffplay.texi | 9 + 1 file changed, 9 insertions(+) diff --git a/doc/ffplay.texi b/doc/ffplay.texi index be696c8..2a35c21 100644 --- a/doc/ffplay.texi +++ b/doc/ff

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 9:57 AM, Clément Bœsch wrote: > On Sat, Sep 26, 2015 at 11:36:42AM -0400, Ganesh Ajjanagadde wrote: > [...] >> -memcpy(stream, (uint8_t *)is->audio_buf + is->audio_buf_index, >> len1); >> +if (is->muted) >> +memset(stream, 0, len1); > > This doe

Re: [FFmpeg-devel] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-27 Thread Clément Bœsch
On Sat, Sep 26, 2015 at 11:36:42AM -0400, Ganesh Ajjanagadde wrote: [...] > -memcpy(stream, (uint8_t *)is->audio_buf + is->audio_buf_index, len1); > +if (is->muted) > +memset(stream, 0, len1); This doesn't work with u8 sampling. Would it make sense to insert + reconfig

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

2015-09-27 Thread Clément Bœsch
On Fri, Sep 25, 2015 at 04:26:00PM -0500, Kyle Swanson wrote: [...] > This is a feature supported by many other loudness scanners. In the > United States, this filter is used to measure loudness for much of our > distributed public radio content. Our current workaround is to first > determine chann

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 8:37 AM, Stefano Sabatini wrote: > --- > src/index | 234 > ++ > 1 file changed, 234 insertions(+) > > diff --git a/src/index b/src/index > index 9d12a7b..8937172 100644 > --- a/src/index > +++ b/src/index > @@ -

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread James Darnley
On 2015-09-27 14:37, Stefano Sabatini wrote: > --- > src/index | 234 > ++ > 1 file changed, 234 insertions(+) > > diff --git a/src/index b/src/index > index 9d12a7b..8937172 100644 > --- a/src/index > +++ b/src/index > @@ -37,6 +37,240

[FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread Stefano Sabatini
--- src/index | 234 ++ 1 file changed, 234 insertions(+) diff --git a/src/index b/src/index index 9d12a7b..8937172 100644 --- a/src/index +++ b/src/index @@ -37,6 +37,240 @@ News + September 27th, 2015, GSoC 2015 results + +

[FFmpeg-devel] [PATCH] fate: Add basic license header check

2015-09-27 Thread Michael Niedermayer
From: Michael Niedermayer --- tests/Makefile |1 + tests/fate-run.sh |5 + tests/fate/source-check.sh | 20 tests/fate/source.mak |3 +++ tests/ref/fate/source | 18 ++ 5 files changed, 47 insertions(+) crea

Re: [FFmpeg-devel] [PATCH]h264: Fix ticket #3147 H264 - Wrong field order

2015-09-27 Thread Kieran Kunhya
On 27 September 2015 at 09:09, Thomas Mundt wrote: > Hi Kieran, > no, you´re right. I misinterpreted SEI_PIC_STRUCT_FRAME because it´s also the > default when pic_struct_present_flag is 0. Just checked avci header - no > pic_struct_present_flag. Sorry!But fixing avci field order detection should

Re: [FFmpeg-devel] [PATCH] avformat/hls: remove unused function

2015-09-27 Thread Hendrik Leppkes
On Sun, Sep 27, 2015 at 12:32 PM, compn wrote: > On Sat, 26 Sep 2015 19:20:57 +0200 > Hendrik Leppkes wrote: > >> On Sat, Sep 26, 2015 at 7:00 PM, wm4 wrote: >> > On Sat, 26 Sep 2015 12:53:58 -0400 >> > Ganesh Ajjanagadde wrote: >> > >> >> On Sat, Sep 26, 2015 at 12:37 PM, wm4 >> >> wrote: >>

Re: [FFmpeg-devel] [PATCH] avformat/hls: remove unused function

2015-09-27 Thread compn
On Sat, 26 Sep 2015 19:20:57 +0200 Hendrik Leppkes wrote: > On Sat, Sep 26, 2015 at 7:00 PM, wm4 wrote: > > On Sat, 26 Sep 2015 12:53:58 -0400 > > Ganesh Ajjanagadde wrote: > > > >> On Sat, Sep 26, 2015 at 12:37 PM, wm4 > >> wrote: > >> > On Sat, 26 Sep 2015 12:12:44 -0400 > >> > Ganesh Ajjana

Re: [FFmpeg-devel] [PATCH 2/7] dnxhddec: simplify block parsing calls

2015-09-27 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 10:09:03AM +0200, Christophe Gisquet wrote: > Fewer arguments, less duplicated code. > --- > libavcodec/dnxhddec.c | 43 --- > 1 file changed, 16 insertions(+), 27 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint

Re: [FFmpeg-devel] [PATCH 1/7] dnxhddec: remove unused qscale parameter

2015-09-27 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 10:09:02AM +0200, Christophe Gisquet wrote: > --- > libavcodec/dnxhddec.c | 25 - > 1 file changed, 12 insertions(+), 13 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB If you think th

Re: [FFmpeg-devel] [PATCH]h264: Fix ticket #3147 H264 - Wrong field order

2015-09-27 Thread Thomas Mundt
Hi Kieran, no, you´re right. I misinterpreted SEI_PIC_STRUCT_FRAME because it´s also the default when pic_struct_present_flag is 0. Just checked avci header - no pic_struct_present_flag. Sorry!But fixing avci field order detection should be done.Does attached patch make more sense to you? Regard

[FFmpeg-devel] [PATCH 7/7] dnxhddec: add my contributions

2015-09-27 Thread Christophe Gisquet
--- libavcodec/dnxhddec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index 7b96004..423bce2 100644 --- a/libavcodec/dnxhddec.c +++ b/libavcodec/dnxhddec.c @@ -2,8 +2,10 @@ * VC3/DNxHD decoder. * Copyright (c) 2007 SmartJog S.A., Baptiste

[FFmpeg-devel] [PATCH 6/7] dnxhddata: deduplicate table

2015-09-27 Thread Christophe Gisquet
CID 1256 chroma table is the same as luma table for CID 1235 and 1256, which is consistent with the content being (normally) RGB. CID 1244 weight tables are identica to CID 1244, but as this was not tested against a real sample, it was not added. --- libavcodec/dnxhddata.c | 16 +++-

[FFmpeg-devel] [PATCH 5/7] dnxhddec: reindent/cosmetics

2015-09-27 Thread Christophe Gisquet
--- libavcodec/dnxhddec.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index 06d4780..7b96004 100644 --- a/libavcodec/dnxhddec.c +++ b/libavcodec/dnxhddec.c @@ -64,18 +64,18 @@ typedef struct DN

[FFmpeg-devel] [PATCH 4/7] dnxhddec: use unsafe bitstream reader

2015-09-27 Thread Christophe Gisquet
Each line is padded by the format, and errors are now reported and stop the decoding. Around 5% speedup. --- libavcodec/dnxhddec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index 8d30446..06d4780 100644 --- a/libavcodec/dnxhddec.c +++ b/libav

[FFmpeg-devel] [PATCH 2/7] dnxhddec: simplify block parsing calls

2015-09-27 Thread Christophe Gisquet
Fewer arguments, less duplicated code. --- libavcodec/dnxhddec.c | 43 --- 1 file changed, 16 insertions(+), 27 deletions(-) diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index 34cf379..007eabe 100644 --- a/libavcodec/dnxhddec.c +++ b/libavcode

[FFmpeg-devel] [PATCH 3/7] dnxhddec: check and report bitstream errors

2015-09-27 Thread Christophe Gisquet
This only occur when an overrun in coefficient decoding is detected. --- libavcodec/dnxhddec.c | 48 ++-- 1 file changed, 34 insertions(+), 14 deletions(-) diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index 007eabe..8d30446 100644 --- a/li

[FFmpeg-devel] [PATCH 1/7] dnxhddec: remove unused qscale parameter

2015-09-27 Thread Christophe Gisquet
--- libavcodec/dnxhddec.c | 25 - 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index fa93407..34cf379 100644 --- a/libavcodec/dnxhddec.c +++ b/libavcodec/dnxhddec.c @@ -63,7 +63,7 @@ typedef struct DNXHDContex

[FFmpeg-devel] [PATCH 0/7] dnxhddec cleanup

2015-09-27 Thread Christophe Gisquet
Nothing major, mostly related to error reporting. Christophe Gisquet (7): dnxhddec: remove unused qscale parameter dnxhddec: simplify block parsing calls dnxhddec: check and report bitstream errors dnxhddec: use unsafe bitstream reader dnxhddec: reindent/cosmetics dnxhddata: deduplicat