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

2015-09-26 Thread Andrey Utkin
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 toget

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

2015-09-26 Thread Christophe Gisquet
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 mailing list ffmpeg-devel@ffmpeg.org

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

2015-09-26 Thread Claudio Freire
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 patch tweaks search

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

2015-09-26 Thread Paul B Mahol
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 100644 libavfilter/vf_

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

2015-09-26 Thread Carl Eugen Hoyos
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 ff_hevc_muxer

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

2015-09-26 Thread Anshul
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 b/doc/encoders.texi inde

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

2015-09-26 Thread DeHackEd
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 changed, 42 insertio

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

2015-09-26 Thread Matt Oliver
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 > > > instead of just pthreads. > > > > > >

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

2015-09-26 Thread Anshul
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 + 2 file

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

2015-09-26 Thread Ronald S. Bultje
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 | 4 ++-- >> > 1 file changed, 2 insertions(+), 2 deletions(-)

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

2015-09-26 Thread Ronald S. Bultje
Hi, On Fri, Sep 25, 2015 at 3:28 PM, Henrik Gramner wrote: > Tested functions are internally kept in a binary search tree for efficient > lookups. The downside of the current implementation is that the tree > quickly > becomes unbalanced which causes an unneccessary amount of comparisons > betwe

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

2015-09-26 Thread Anshul
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 + libavcodec/libx264.c | 37 ++

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

2015-09-26 Thread Michael Niedermayer
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 no threading. What >

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

2015-09-26 Thread DeHackEd
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 --- a/doc/encoders.te

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

2015-09-26 Thread Christophe Gisquet
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

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

2015-09-26 Thread Christophe Gisquet
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 --- a/liba

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

2015-09-26 Thread Christophe Gisquet
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 d9601c7b74b5dea0f0be9989cc657fad90c1be7d Mon Sep 17 00:00:00 2001 Fr

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

2015-09-26 Thread Christophe Gisquet
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 mishandling of code around ex

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

2015-09-26 Thread Henrik Gramner
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@ffmpeg.org http://ffmpeg.or

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

2015-09-26 Thread Henrik Gramner
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 therefore never run any tests.

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

2015-09-26 Thread Henrik Gramner
--- 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 10064

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

2015-09-26 Thread Thomas Mundt
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 1/5] dnxhddec: proper rule for interlaced mb flag

2015-09-26 Thread Michael Niedermayer
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 [...

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

2015-09-26 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] [PATCHv2] avcodec/videotoolbox: fix -Wunused-but-set-variable

2015-09-26 Thread Ganesh Ajjanagadde
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 -Wunused-but-set-variable reported at e

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

2015-09-26 Thread Ganesh Ajjanagadde
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&log=compile&slot=x86_64-archlinux-gcc-disableswscale, > http://fate.ffmpeg.org/log.cgi?

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

2015-09-26 Thread Ganesh Ajjanagadde
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: Fixes -Wunused-function from http://fate.ffmpeg.

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

2015-09-26 Thread Ganesh Ajjanagadde
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 wrote: >>> > On Tue, Sep 15, 2015 at 6:

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

2015-09-26 Thread Ganesh Ajjanagadde
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 with this expression rewr

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

2015-09-26 Thread Hendrik Leppkes
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 Niedermayer wrote: On Tue, Sep 15, 2015 at 08:48:3

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

2015-09-26 Thread Timothy Gu
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 toggle_mute(VideoState *is) >

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

2015-09-26 Thread Ganesh Ajjanagadde
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 >>> wrote: On Tue, Sep 15, 2015 at 04:54:19PM +0200,

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

2015-09-26 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

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

2015-09-26 Thread Ganesh Ajjanagadde
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: Gane

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

2015-09-26 Thread Ganesh Ajjanagadde
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 @@ static void toggle_pause(VideoState *is) >> is

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

2015-09-26 Thread Michael Niedermayer
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 not using git > send-email for

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

2015-09-26 Thread Henrik Gramner
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 hand. > > Apologies, but since the thread w

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

2015-09-26 Thread Thomas Mundt
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 field

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

2015-09-26 Thread Henrik Gramner
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

2015-09-26 Thread wm4
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: > >>> > >>> On Wed, Sep 16, 2015 at 8:26 PM, Ganesh Ajjanagadde > >>>

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

2015-09-26 Thread wm4
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 being used for such a purpose. > > This patch simp

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

2015-09-26 Thread Ganesh Ajjanagadde
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] avformat/hls: remove unused function

2015-09-26 Thread Ganesh Ajjanagadde
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 wrote: >> >> On 18.09.2015 15:09, Ganesh Ajjanagadde wrote: >> >>> >> >>>

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

2015-09-26 Thread Henrik Gramner
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] checkasm: remove use of deprecated av_set_cpu_flags_mask()

2015-09-26 Thread Ronald S. Bultje
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(-) > > diff --git a/tes

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

2015-09-26 Thread wm4
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 Ajjanagadde > >> wrote: > >> > On Fri, Sep 18, 2015 at 11:08 AM, w

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

2015-09-26 Thread Ganesh Ajjanagadde
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. But I guess we could change those as we

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

2015-09-26 Thread Ganesh Ajjanagadde
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 wrote: >> > >> >> On Fri, Sep 18, 2015 at 11:16 AM, Ganesh Ajjanagadde

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

2015-09-26 Thread wm4
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 wrote: > >> > On Sat, 26 Sep 2015 10:59:44 -0400 > >> > Ganesh Ajjanag

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

2015-09-26 Thread Hendrik Leppkes
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 functions in that header used "inline", so I went with t

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

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

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

2015-09-26 Thread Ganesh Ajjanagadde
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 >>> wrote: Have you compared with av_always_inline? >>> >

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

2015-09-26 Thread Ganesh Ajjanagadde
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 >>> >>> http://fate.ffmpeg.org/report.cgi?time=20150917015044&slot=alpha-debian-qemu-gcc-4.

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

2015-09-26 Thread Henrik Gramner
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] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

2015-09-26 Thread Ganesh Ajjanagadde
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 > http://fate.ffmpeg.org/log.cgi?time=2015091818

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

2015-09-26 Thread Kieran Kunhya
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 http://ffmpeg.org/mailman/listinfo

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

2015-09-26 Thread Kieran Kunhya
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 http://ffmpeg.org/mailman/listinfo/ffmpeg-de

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

2015-09-26 Thread Timo Rothenpieler
> 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 work

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

2015-09-26 Thread James Almer
Signed-off-by: James Almer --- tests/checkasm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/checkasm/Makefile b/tests/checkasm/Makefile index 03e0e3f..f30e9c4 100644 --- a/tests/checkasm/Makefile +++ b/tests/checkasm/Makefile @@ -30,7 +30,7 @@ $(CHECKASM): $(E

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

2015-09-26 Thread James Almer
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 +107,8 @@ static voi

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

2015-09-26 Thread Henrik Gramner
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 ffmpeg-devel@ffmpeg.org http://ffmpeg.org/ma

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

2015-09-26 Thread Henrik Gramner
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 ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mail

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

2015-09-26 Thread James Almer
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.

[FFmpeg-devel] Getting timestamp from transport stream

2015-09-26 Thread Sabur Hassan Baidya
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 client server communication which I tested with the .ts file. But I want to contro

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

2015-09-26 Thread Ganesh Ajjanagadde
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 --- ffplay.c | 8 ++-- 1 file

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

2015-09-26 Thread Moritz Barsnick
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] avutil/avstring: Inline some tiny functions

2015-09-26 Thread Henrik Gramner
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 fairly detailed >>> explanation of av_always_inline. Essen

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

2015-09-26 Thread Timo Rothenpieler
> @@ -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 = av_malloc(strlen(filena

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

2015-09-26 Thread Paul B Mahol
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 unchanged while 255 for

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

2015-09-26 Thread Ganesh Ajjanagadde
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 --- ffplay.c | 7 +-- 1 file

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

2015-09-26 Thread Ganesh Ajjanagadde
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; >> -av_strlcpy(is->filename, filename, siz

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

2015-09-26 Thread Michael Niedermayer
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 patchones

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

2015-09-26 Thread Michael Niedermayer
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: > > > > > > > Allows the decklink device to be used

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

2015-09-26 Thread Michael Niedermayer
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 [...] -- Michael GnuPG fingerpri

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

2015-09-26 Thread Marton Balint
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 ident

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

2015-09-26 Thread Michael Niedermayer
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 wrote: > >> > On Sat, 26 Sep 2015 12:12:44 -0400 > >> > Ganesh Ajjana

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

2015-09-26 Thread Marton Balint
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 thro

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

2015-09-26 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] [PATCHv2 1/2] ffplay: add support for interactive volume control

2015-09-26 Thread Ganesh Ajjanagadde
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 >> system-wide, or one ne

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

2015-09-26 Thread Michael Niedermayer
On Sat, Sep 26, 2015 at 02:42:27PM +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 > > Patch reworked: > - Preallocate RowContext to avoid reloading it > -

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

2015-09-26 Thread Michael Niedermayer
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 ex

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

2015-09-26 Thread Ganesh Ajjanagadde
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&log=compile&slot=x86_64-archlinux-gcc-threads-misc, "warning: assuming signed overflow does not occur when assuming that (X - c) >

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

2015-09-26 Thread Ganesh Ajjanagadde
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 checked that all usages

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

2015-09-26 Thread Marton Balint
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 ident

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

2015-09-26 Thread Marton Balint
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: Gane

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

2015-09-26 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] [PATCH] libx264: copy A53 closed captions from source

2015-09-26 Thread Michael Niedermayer
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 --git a/doc/encoders.

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

2015-09-26 Thread James Almer
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 available

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

2015-09-26 Thread Ronald S. Bultje
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, and suppresses > -Wstrict-overflow > >>

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

2015-09-26 Thread Ganesh Ajjanagadde
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 -0400, Ganesh Ajjanagadde wrote: >> >> This patch results i

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

2015-09-26 Thread Ganesh Ajjanagadde
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, Michael Niedermayer >>> wrote: >>> > On Fri, Sep 18, 2015 at

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

2015-09-26 Thread Dave Yeo
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 _