Re: [FFmpeg-devel] [PATCH 1/4] dnxhddec: implement slice multithreading

Hi, 2015-09-26 1:52 GMT+02:00 Michael Niedermayer : > This segfaults with > http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket2125/DNxHDtest2.mov Can't reproduce with frame or slice threading or no threading. What was the command-line? > #3 0x00af53fd in

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

On 09/26/2015 05:46 AM, DeHackEd wrote: Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 + libavcodec/libx264.c | 37 + 2 files changed, 42 insertions(+) diff --git a/doc/encoders.texi

Re: [FFmpeg-devel] [PATCH 1/4] dnxhddec: implement slice multithreading

On Sat, Sep 26, 2015 at 08:33:26AM +0200, Christophe Gisquet wrote: > Hi, > > 2015-09-26 1:52 GMT+02:00 Michael Niedermayer : > > This segfaults with > > http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket2125/DNxHDtest2.mov > > Can't reproduce with frame or slice threading or

[FFmpeg-devel] [PATCH] avfilter: add maskedmerge filter

Signed-off-by: Paul B Mahol --- doc/filters.texi | 15 ++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_maskedmerge.c | 324 +++ 4 files changed, 341 insertions(+) create mode

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

On 09/26/2015 05:26 AM, Anshul wrote: > > > On 09/26/2015 05:46 AM, DeHackEd wrote: >> Assumes 'GA94' format (ATSC standard) >> >> Signed-off-by: DHE >> --- >> doc/encoders.texi| 5 + >> libavcodec/libx264.c | 37 + >> 2 files

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

On 09/26/2015 04:16 PM, Anshul wrote: On 09/26/2015 03:57 PM, DeHackEd wrote: On 09/26/2015 05:26 AM, Anshul wrote: On 09/26/2015 05:46 AM, DeHackEd wrote: Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 +

[FFmpeg-devel] Help to debug h264 headers (or video) generation in kernel driver

I'm working on Linux kernel driver for multimedia grabber and H264 encoder based on Techwell/Intersil 5864 chip. Of course it will be open and pushed into upstream kernel. The device produces H264 encoded data, but it lacks any headers. The reference driver generates headers and glues frames

[FFmpeg-devel] [PATCH 1/5] dnxhddec: proper rule for interlaced mb flag

It currently only applies to CID 1260, but this flag is dependent on a higher-level flag located in the header. --- libavcodec/dnxhddec.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index 044e479..e5c5f07 100644 ---

[FFmpeg-devel] [PATCH 3/4] dnxhddec: proper rule for interlaced mb flag

I have reordered the patches so that they don't rely on the slice threading one. Therefore this one should apply cleanly. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [FFmpeg-cvslog] AAC encoder: tweak PNS usage to be more aggressive

On Fri, Sep 25, 2015 at 10:39 PM, James Almer wrote: >> ffmpeg | branch: master | Claudio Freire | Fri >> Sep 25 03:56:32 2015 -0300| [9458a62decfcaa1313b1ba69276466de536d0768] | >> committer: Claudio Freire >> >> AAC encoder: tweak PNS usage to be more aggressive >> >> This

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

On 09/26/2015 03:57 PM, DeHackEd wrote: On 09/26/2015 05:26 AM, Anshul wrote: On 09/26/2015 05:46 AM, DeHackEd wrote: Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 + libavcodec/libx264.c | 37

Re: [FFmpeg-devel] [PATCH 4/4] dnxhddec: parse and print adaptive color transform

2015-09-25 23:25 GMT+02:00 Christophe Gisquet : [SNIP] I have reordered the patches so that they don't rely on the slice threading one. Therefore this one should apply cleanly. (And also not using git send-email for replies) From

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

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 randomization). Windows does this for

Re: [FFmpeg-devel] [PATCH 1/4] dnxhddec: implement slice multithreading

Hi, 2015-09-26 8:33 GMT+02:00 Christophe Gisquet : > Is it something that should be avoidable through my code? I now think so, so please ignore me till I've understood all of the API. -- Christophe ___ ffmpeg-devel

[FFmpeg-devel] [PATCH] configure: Combine dynamicbase and nxcompat checks

They were added to binutils in the same version so it's safe to combine. Signed-off-by: Alex Smith --- configure | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/configure b/configure index 3339126..b572d59 100755 --- a/configure +++ b/configure @@

Re: [FFmpeg-devel] [PATCH 1/2] checkasm: clip vp9 loopfilter test pixels inside allowed bitdepth range.

Hi, On Fri, Sep 25, 2015 at 6:50 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Sep 25, 2015 at 6:42 PM, Henrik Gramner > wrote: > >> On Fri, Sep 25, 2015 at 5:25 PM, Ronald S. Bultje >> wrote: >> > --- >> > tests/checkasm/vp9dsp.c |

Re: [FFmpeg-devel] [PATCH] checkasm: Use a self-balancing tree

On Sat, Sep 26, 2015 at 12:04 AM, Michael Niedermayer wrote: > ahh, ok, no objection from me then On Sat, Sep 26, 2015 at 12:53 PM, Ronald S. Bultje wrote: > lgtm. Thanks, applied. ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH] checkasm: remove use of deprecated av_set_cpu_flags_mask()

--- A working implementation of the original patch if we actually do want to get rid of av_set_cpu_flags_mask(). --- tests/checkasm/checkasm.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c index 2f967e3..8c75cc9

Re: [FFmpeg-devel] [PATCH] avcodec/xface: suppress -Wstrict-overflow warnings

On Fri, Sep 18, 2015 at 5:31 PM, Ganesh Ajjanagadde wrote: > This patch results in identical behavior of xface, and suppresses > -Wstrict-overflow > warnings observed in GCC 5.2. > I have manually checked that this usage is safe, and overflow possibility does > not exist

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

On Sat, Sep 26, 2015 at 12:06 PM, wm4 wrote: > On Sat, 26 Sep 2015 10:59:44 -0400 > Ganesh Ajjanagadde wrote: > >> On Fri, Sep 18, 2015 at 11:16 AM, Ganesh Ajjanagadde >> wrote: >> > On Fri, Sep 18, 2015 at 11:08 AM, wm4

Re: [FFmpeg-devel] [PATCH] checkasm: remove use of deprecated av_set_cpu_flags_mask()

Hi, On Sat, Sep 26, 2015 at 9:54 AM, Henrik Gramner wrote: > --- > A working implementation of the original patch if we actually do want to > get > rid of av_set_cpu_flags_mask(). > --- > tests/checkasm/checkasm.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-)

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

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 Ajjanagadde

Re: [FFmpeg-devel] [PATCHv2] avcodec/videotoolbox: fix -Wunused-but-set-variable

On Sat, Sep 19, 2015 at 5:41 PM, Ganesh Ajjanagadde wrote: > pix_fmt was declared presumably to shorten the argument passed to the > function. > However, it is currently not being used for such a purpose. > This patch simply removes it instead. > This fixes

Re: [FFmpeg-devel] policy on "necro-bumping" patches

On Sat, Sep 26, 2015 at 5:03 PM, Ganesh Ajjanagadde wrote: > On Sat, Sep 19, 2015 at 9:03 AM, Ganesh Ajjanagadde wrote: >> On Wed, Sep 16, 2015 at 8:29 PM, Michael Niedermayer >> wrote: >>> On Tue, Sep 15, 2015 at 04:54:19PM +0200, Michael

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

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

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

Hi, some h264 encoders, like broadcast avc-intra in this case, always set sei_pic_struct to "frame" even at interlaced encodings with separate fields. FFmpeg h264 decoder always assumes field order as "bottom field first" unless "top field first" is definitely detected. This leads to wrong

Re: [FFmpeg-devel] [PATCH] avutil/avstring: Inline some tiny functions

On Sat, Sep 26, 2015 at 6:10 PM, Ganesh Ajjanagadde wrote: > Have you compared with av_always_inline? No, other functions in that header used "inline", so I went with the same for consistency. But I guess we could change those as well if av_always_inline is preferable.

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

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 Ajjanagadde wrote: > > > >> On Sat, Sep 26, 2015 at 12:06 PM, wm4

Re: [FFmpeg-devel] [PATCH] avutil/avstring: Inline some tiny functions

On Sat, Sep 26, 2015 at 1:18 PM, Hendrik Leppkes wrote: > On Sat, Sep 26, 2015 at 6:48 PM, Ganesh Ajjanagadde wrote: >> On Sat, Sep 26, 2015 at 12:26 PM, Henrik Gramner wrote: >>> On Sat, Sep 26, 2015 at 6:10 PM, Ganesh Ajjanagadde

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

The previous implementation was behaving incorrectly in some corner cases. --- tests/checkasm/checkasm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c index 8c75cc9..715b08c 100644 --- a/tests/checkasm/checkasm.c

Re: [FFmpeg-devel] policy on "necro-bumping" patches

On Sat, Sep 19, 2015 at 9:03 AM, Ganesh Ajjanagadde wrote: > On Wed, Sep 16, 2015 at 8:29 PM, Michael Niedermayer wrote: >> On Tue, Sep 15, 2015 at 04:54:19PM +0200, Michael Niedermayer wrote: >>> On Tue, Sep 15, 2015 at 08:48:33AM -0400, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Sat, Sep 19, 2015 at 1:00 AM, Ganesh Ajjanagadde wrote: > Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. > The concept of av_uninit is inherently useful though. This patch silences a > bunch of warnings on clang e.g >

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

This will fix AVC-Intra encoding in .mov container. Regards,Thomas libx264.diff Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 4/4] dnxhddec: parse and print adaptive color transform

On Sat, Sep 26, 2015 at 02:36:51PM +0200, Christophe Gisquet wrote: > 2015-09-25 23:25 GMT+02:00 Christophe Gisquet : > [SNIP] > > I have reordered the patches so that they don't rely on the slice threading > one. > Therefore this one should apply cleanly. (And also

[FFmpeg-devel] [PATCH] avutil/avstring: Inline some tiny functions

They're short enough that inlining them actually reduces code size due to all the overhead associated with making a function call. --- libavutil/avstring.c | 22 -- libavutil/avstring.h | 22 ++ 2 files changed, 18 insertions(+), 26 deletions(-) diff --git

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

On Sat, Sep 26, 2015 at 12:37 PM, wm4 wrote: > On Sat, 26 Sep 2015 12:12:44 -0400 > Ganesh Ajjanagadde wrote: > >> On Sat, Sep 26, 2015 at 12:06 PM, wm4 wrote: >> > On Sat, 26 Sep 2015 10:59:44 -0400 >> > Ganesh Ajjanagadde

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

On Fri, Sep 18, 2015 at 9:05 AM, Ganesh Ajjanagadde wrote: > On Fri, Sep 18, 2015 at 4:40 AM, wm4 wrote: >> On 17.09.2015 15:07, Ganesh Ajjanagadde wrote: >>> >>> When compiled with --disable-pthreads, e.g >>> >>>

[FFmpeg-devel] [PATCH]lavf/rawenc: Force one stream for hevc and m4v

Hi! Attached patch forces one stream for the output formats hevc and (raw) m4v. Please comment, Carl Eugen diff --git a/libavformat/rawenc.c b/libavformat/rawenc.c index d65c7c7..7a1a3d0 100644 --- a/libavformat/rawenc.c +++ b/libavformat/rawenc.c @@ -241,6 +241,7 @@ AVOutputFormat

Re: [FFmpeg-devel] [PATCH 3/3] avdevice/decklink: Allow compilation with native threads.

On 26 September 2015 at 04:08, Michael Niedermayer wrote: > On Fri, Sep 25, 2015 at 09:41:59PM +1000, Matt Oliver wrote: > > On 25 September 2015 at 17:58, Matt Oliver wrote: > > > > > Allows the decklink device to be used with native win32/os2 threads >

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

Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 + libavcodec/libx264.c | 37 + 2 files changed, 42 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 3550bcc..aabbda0 100644

Re: [FFmpeg-devel] [PATCH 1/4] dnxhddec: implement slice multithreading

Hi, 2015-09-26 1:52 GMT+02:00 Michael Niedermayer : > This segfaults with > http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket2125/DNxHDtest2.mov Patch reworked: - Preallocate RowContext to avoid reloading it - Use of init_thread_copy due to that - Fix crashes due to

Re: [FFmpeg-devel] [PATCH] cmdutils: silence unused warnings under --disable-swscale, --disable-swresample

On Sat, Sep 19, 2015 at 10:20 AM, Ganesh Ajjanagadde wrote: > This patch silences such warnings by placing initializations under a header > guard, > see e.g > http://fate.ffmpeg.org/log.cgi?time=20150919095430=compile=x86_64-archlinux-gcc-disableswscale, >

Re: [FFmpeg-devel] [PATCHv2] avcodec/videotoolbox: fix -Wunused-but-set-variable

On Sat, 26 Sep 2015 10:55:29 -0400 Ganesh Ajjanagadde wrote: > On Sat, Sep 19, 2015 at 5:41 PM, Ganesh Ajjanagadde > wrote: > > pix_fmt was declared presumably to shorten the argument passed to the > > function. > > However, it is currently not

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

On Sat, 26 Sep 2015 12:12:44 -0400 Ganesh Ajjanagadde wrote: > On Sat, Sep 26, 2015 at 12:06 PM, wm4 wrote: > > On Sat, 26 Sep 2015 10:59:44 -0400 > > Ganesh Ajjanagadde wrote: > > > >> On Fri, Sep 18, 2015 at 11:16 AM, Ganesh

Re: [FFmpeg-devel] [PATCH] avutil/avstring: Inline some tiny functions

On Sat, Sep 26, 2015 at 12:26 PM, Henrik Gramner wrote: > On Sat, Sep 26, 2015 at 6:10 PM, Ganesh Ajjanagadde wrote: >> Have you compared with av_always_inline? > > No, other functions in that header used "inline", so I went with the > same for consistency.

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

On Sat, Sep 26, 2015 at 7:11 PM, Michael Niedermayer wrote: > On Fri, Sep 18, 2015 at 05:15:50PM -0400, Ganesh Ajjanagadde wrote: >> This patch results in identical behavior of movenc, and suppresses >> -Wstrict-overflow >> warnings observed in GCC 5.2. >> I have manually

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

This patch results in identical behavior of movenc, and suppresses -Wstrict-overflow warnings observed in GCC 5.2: http://fate.ffmpeg.org/log.cgi?time=20150926231053=compile=x86_64-archlinux-gcc-threads-misc, "warning: assuming signed overflow does not occur when assuming that (X - c) > X is

Re: [FFmpeg-devel] [PATCHv2] ffplay: dynamically allocate filename buffer

On Sat, 26 Sep 2015, Ganesh Ajjanagadde wrote: filename was set to an arbitrary 1024 characters. ffplay would thus be unable to play files whose name exceeds that arbitrary threshold. This patch dynamically allocates and frees the filename buffer to remove such limitations. Signed-off-by:

Re: [FFmpeg-devel] [PATCH] ffplay: dynamically allocate filename buffer

On Sat, Sep 26, 2015 at 4:50 PM, Timo Rothenpieler wrote: >> @@ -3120,7 +3121,10 @@ static VideoState *stream_open(const char *filename, >> AVInputFormat *iformat) >> is = av_mallocz(sizeof(VideoState)); >> if (!is) >> return NULL; >> -

[FFmpeg-devel] [PATCHv2] ffplay: dynamically allocate filename buffer

filename was set to an arbitrary 1024 characters. ffplay would thus be unable to play files whose name exceeds that arbitrary threshold. This patch dynamically allocates and frees the filename buffer to remove such limitations. Signed-off-by: Ganesh Ajjanagadde ---

Re: [FFmpeg-devel] [PATCH] configure: Combine dynamicbase and nxcompat checks

On Sat, Sep 26, 2015 at 02:15:24AM -0400, Alex Smith wrote: > They were added to binutils in the same version so it's safe to combine. > > Signed-off-by: Alex Smith > --- > configure | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) applied thanks [...] --

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

On Sat, 26 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

Re: [FFmpeg-devel] [PATCHv2 2/2] ffplay: introduce key repeats

On Sat, 26 Sep 2015, Ganesh Ajjanagadde wrote: Key repeats have been introduced simply because they improve usability in my experience for volume, brightness, and other such controls by speeding up the time taken to go from 0 to max intensity. As a side benefit, this enables rapid seeking

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

On Fri, Sep 18, 2015 at 05:15:50PM -0400, Ganesh Ajjanagadde wrote: > This patch results in identical behavior of movenc, and suppresses > -Wstrict-overflow > warnings observed in GCC 5.2. > I have manually checked that all usages are safe, and overflow possibility > does > not exist with this

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

On Sat, 26 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

Re: [FFmpeg-devel] [PATCH 3/3] avdevice/decklink: Allow compilation with native threads.

On Sat, Sep 26, 2015 at 08:45:13PM +1000, Matt Oliver wrote: > On 26 September 2015 at 04:08, Michael Niedermayer wrote: > > > On Fri, Sep 25, 2015 at 09:41:59PM +1000, Matt Oliver wrote: > > > On 25 September 2015 at 17:58, Matt Oliver wrote: > > > > > >

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

On Sat, Sep 26, 2015 at 07:20:57PM +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

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

On Sat, Sep 26, 2015 at 6:29 PM, Marton Balint wrote: > > On Sat, 26 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 >>

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

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

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

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 ---

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

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

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

On Fri, Sep 18, 2015 at 11:16 AM, Ganesh Ajjanagadde wrote: > On Fri, Sep 18, 2015 at 11:08 AM, wm4 wrote: >> On 18.09.2015 15:09, Ganesh Ajjanagadde wrote: >>> >>> On Wed, Sep 16, 2015 at 8:26 PM, Ganesh Ajjanagadde >>> wrote:

[FFmpeg-devel] [PATCHv2 2/2] ffplay: introduce key repeats

Key repeats have been introduced simply because they improve usability in my experience for volume, brightness, and other such controls by speeding up the time taken to go from 0 to max intensity. As a side benefit, this enables rapid seeking through a file via left/right keys. Signed-off-by:

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

On Sat, Sep 26, 2015 at 11:16 AM, Timothy Gu wrote: > On Sat, Sep 26, 2015 at 7:47 AM, Ganesh Ajjanagadde > wrote: >> diff --git a/ffplay.c b/ffplay.c >> index d302793..4f3322b 100644 >> --- a/ffplay.c >> +++ b/ffplay.c >> @@ -1348,6 +1353,25 @@

Re: [FFmpeg-devel] policy on "necro-bumping" patches

On Sat, Sep 26, 2015 at 5:18 PM, Ganesh Ajjanagadde wrote: > On Sat, Sep 26, 2015 at 11:09 AM, Hendrik Leppkes wrote: >> Please ping the actual thread of the patch, and not some seemingly >> unrelated thread, so people are actually aware of the topic at

Re: [FFmpeg-devel] [PATCH] avutil/avstring: Inline some tiny functions

On Sat, Sep 26, 2015 at 12:04 PM, Henrik Gramner wrote: > They're short enough that inlining them actually reduces code size due to > all the overhead associated with making a function call. > --- > libavutil/avstring.c | 22 -- > libavutil/avstring.h | 22

Re: [FFmpeg-devel] [PATCH] avutil/avstring: Inline some tiny functions

On Sat, Sep 26, 2015 at 6:48 PM, Ganesh Ajjanagadde wrote: > On Sat, Sep 26, 2015 at 12:26 PM, Henrik Gramner wrote: >> On Sat, Sep 26, 2015 at 6:10 PM, Ganesh Ajjanagadde wrote: >>> Have you compared with av_always_inline? >> >> No, other

[FFmpeg-devel] [PATCH 2/2] checkasm/vp9dsp: add const to suppress "discards const qualifier" warnings

Signed-off-by: James Almer --- tests/checkasm/vp9dsp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/checkasm/vp9dsp.c b/tests/checkasm/vp9dsp.c index 0f45a8b..d175e17 100644 --- a/tests/checkasm/vp9dsp.c +++ b/tests/checkasm/vp9dsp.c @@ -107,8

Re: [FFmpeg-devel] [PATCH] avfilter: add maskedmerge filter

On Sat, Sep 26, 2015 at 10:29:15 +0200, Paul B Mahol wrote: > +Merge first input stream with second input stream using per pixel weights in > +the third input stream. 0 means that pixel component from first stream is > +returned unchanged while 255 for 8-bit videos means that pixel component from

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Force one stream for hevc and m4v

On Sat, Sep 26, 2015 at 10:47:53AM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch forces one stream for the output formats hevc > and (raw) m4v. > > Please comment, Carl Eugen > rawenc.c |2 ++ > 1 file changed, 2 insertions(+) > 152253d1707fbee45355abd6364a9ecc592942c8

Re: [FFmpeg-devel] [PATCH] ffplay: dynamically allocate filename buffer

> @@ -3120,7 +3121,10 @@ static VideoState *stream_open(const char *filename, > AVInputFormat *iformat) > is = av_mallocz(sizeof(VideoState)); > if (!is) > return NULL; > -av_strlcpy(is->filename, filename, sizeof(is->filename)); > +is->filename =

Re: [FFmpeg-devel] [PATCH] avfilter: add maskedmerge filter

On 9/26/15, Moritz Barsnick wrote: > On Sat, Sep 26, 2015 at 10:29:15 +0200, Paul B Mahol wrote: > >> +Merge first input stream with second input stream using per pixel weights >> in >> +the third input stream. 0 means that pixel component from first stream is >> +returned

[FFmpeg-devel] [PATCH] ffplay: dynamically allocate filename buffer

filename was set to an arbitrary 1024 characters. ffplay would thus be unable to play files whose name exceeds that arbitrary threshold. This patch dynamically allocates and frees the filename buffer to remove such limitations. Signed-off-by: Ganesh Ajjanagadde ---

Re: [FFmpeg-devel] [PATCH] avutil/avstring: Inline some tiny functions

On Sat, Sep 26, 2015 at 7:23 PM, Ganesh Ajjanagadde wrote: > On Sat, Sep 26, 2015 at 1:18 PM, Hendrik Leppkes wrote: >> On Sat, Sep 26, 2015 at 6:48 PM, Ganesh Ajjanagadde wrote: >>> Commit 06eb4f0885746b7e8a652d7b6026abf834e04b73 has a

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

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, Sep 18, 2015 at 05:15:50PM -0400, Ganesh Ajjanagadde wrote: > >> This patch results in identical behavior of movenc,

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

On 9/26/2015 8:41 AM, DeHackEd wrote: > +/** > + * 'GA94' is standard in North America for ATSC, but hard > coding > + * this style may not be the right thing to do -- other > formats > + * do exist. This information is not

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

On Sat, Sep 26, 2015 at 10:55 PM, 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,

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

On Sat, Sep 26, 2015 at 07:41:32AM -0400, DeHackEd wrote: > Assumes 'GA94' format (ATSC standard) > > Signed-off-by: DHE > --- > doc/encoders.texi| 5 + > libavcodec/libx264.c | 37 + > 2 files changed, 42 insertions(+) > > diff

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

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, Sep 18, 2015 at 05:15:50PM

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

On 09/26/15 03:27 AM, DeHackEd wrote: I think my mail client wrapped some lines. Sorry, this is my first patch-by-email. For Thunderbird, there is an extension, "Toggle Word Wrap" which gives the option of disabling word wrap under the Options menu. Or just attach. Dave

Re: [FFmpeg-devel] [PATCH][RFC] tests/checkasm/checkasm: remove use of deprecated av_set_cpu_flags_mask()

On Sat, Sep 26, 2015 at 2:07 AM, Ganesh Ajjanagadde wrote: > This patch completes the removal of all uses of av_set_cpu_flags_mask, so the > deprecated function can be removed in a future version bump. This is incorrect, it will set the flag to zero every time and

Re: [FFmpeg-devel] [PATCH 1/5] dnxhddec: proper rule for interlaced mb flag

On Sat, Sep 26, 2015 at 02:31:55PM +0200, Christophe Gisquet wrote: > It currently only applies to CID 1260, but this flag is dependent on > a higher-level flag located in the header. > --- > libavcodec/dnxhddec.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) applied thanks

Re: [FFmpeg-devel] policy on "necro-bumping" patches

On Sat, Sep 26, 2015 at 11:09 AM, Hendrik Leppkes wrote: > On Sat, Sep 26, 2015 at 5:03 PM, Ganesh Ajjanagadde wrote: >> On Sat, Sep 19, 2015 at 9:03 AM, Ganesh Ajjanagadde wrote: >>> On Wed, Sep 16, 2015 at 8:29 PM, Michael Niedermayer

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

On Sat, Sep 26, 2015 at 7:47 AM, Ganesh Ajjanagadde wrote: > diff --git a/ffplay.c b/ffplay.c > index d302793..4f3322b 100644 > --- a/ffplay.c > +++ b/ffplay.c > @@ -1348,6 +1353,25 @@ static void toggle_pause(VideoState *is) > is->step = 0; > } > > +static void

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

On Sat, 26 Sep 2015 10:59:44 -0400 Ganesh Ajjanagadde wrote: > On Fri, Sep 18, 2015 at 11:16 AM, Ganesh Ajjanagadde wrote: > > On Fri, Sep 18, 2015 at 11:08 AM, wm4 wrote: > >> On 18.09.2015 15:09, Ganesh Ajjanagadde wrote: > >>> > >>>

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

On 26 September 2015 at 16:59, Thomas Mundt wrote: > Hi, > some h264 encoders, like broadcast avc-intra in this case avc-intra sets pic-struct. Is this new? Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 1/3] avutil/opencl: Fix volatile pointer

> On Thu, Sep 24, 2015 at 05:12:23PM +0200, Timo Rothenpieler wrote: >> --- >> libavutil/opencl.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > LGTM pushed this one. If anyone with OpenCL experience could have a look at the other two, that'd be highly apreciated. I never

Re: [FFmpeg-devel] [PATCH 1/2] checkasm/Makefile: add missing testclean target

On Sat, Sep 26, 2015 at 9:23 PM, James Almer wrote: > Signed-off-by: James Almer > --- > tests/checkasm/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Lgtm. ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 2/2] checkasm/vp9dsp: add const to suppress "discards const qualifier" warnings

On Sat, Sep 26, 2015 at 9:23 PM, James Almer wrote: > Signed-off-by: James Almer > --- > tests/checkasm/vp9dsp.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Ok. ___ ffmpeg-devel mailing list

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

On 26 September 2015 at 15:25, Thomas Mundt wrote: > This will fix AVC-Intra encoding in .mov container. > Regards,Thomas ok, assuming fate passes. Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 1/2] checkasm/Makefile: add missing testclean target

On 9/26/2015 4:29 PM, Henrik Gramner wrote: > On Sat, Sep 26, 2015 at 9:23 PM, James Almer wrote: >> Signed-off-by: James Almer >> --- >> tests/checkasm/Makefile | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > Lgtm. Patchset pushed. Thanks.