Re: [FFmpeg-devel] Adding support from some GoPro udta tags

2016-10-14 Thread Clément Bœsch
On Mon, Oct 10, 2016 at 11:59:59AM +0200, Jean Caillé wrote: > > Hello, > > GoPro sets some specific udta tags in their videos. Two of > those are the FIRM and HMMT udta tags. > The FIRM tag is a string containing the firmware version. > Tht MHHT tag contains a number of "Hero Moments" tags.

Re: [FFmpeg-devel] [PATCH] Improved selftest coverage for libavutil/fifo.c

2016-10-14 Thread Michael Niedermayer
On Thu, Oct 13, 2016 at 03:13:56PM -0700, Thomas Turner wrote: > Tested functions: av_fifo_generic_peek(), av_fifo_grow() > > Signed-off-by: Thomas Turner > --- > libavutil/tests/fifo.c | 39 +-- > tests/ref/fate/fifo| 43

Re: [FFmpeg-devel] [PATCH] d3d11va: use the proper slice index

2016-10-14 Thread wm4
On Tue, 11 Oct 2016 10:18:06 +0200 Hendrik Leppkes wrote: > On Mon, Oct 10, 2016 at 11:34 PM, compn wrote: > > On Mon, 10 Oct 2016 17:07:06 +0200 > > Hendrik Leppkes wrote: > > > >> On Mon, Oct 10, 2016 at 4:01 PM, Michael

Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-14 Thread Greg Rowe
On Fri, Oct 14, 2016 at 18:09:51 +, Greg Rowe wrote: You attached the old patch. ;-) Is there a good way to generate the tone while avoiding floating point operations? Well that's embarrassing. I'll fix that after we figure out how best to handle the tone generation. :) Good point:

Re: [FFmpeg-devel] Developer "cehoyos" closed my bug without any explanation, and without solving it

2016-10-14 Thread wm4
On Tue, 11 Oct 2016 00:23:20 +0200 Alexey Eromenko wrote: > Hello all, > > Bug #5882; Certain x264 videos do not play on Apple decoders > (Quicktime/iTunes/iPad) > > Today was closed by developer "cehoyos" without any explanation, and > without any resolution. > He treats all

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread wm4
On Fri, 14 Oct 2016 16:31:58 +0200 Nicolas George wrote: > Le tridi 23 vendémiaire, an CCXXV, Steven Liu a écrit : > > > Rather than NIHing it, I think using libexpat would be a good choice > > > (even if its API is terrible). > > ok, i'll learn how to use it, and use it next

Re: [FFmpeg-devel] [PATCH] avcodec/vda: define av_vda_default_init2 when CONFIG_H264_VDA_HWACCEL equ 0

2016-10-14 Thread wm4
On Thu, 13 Oct 2016 20:57:07 +0800 Steven Liu wrote: > ping > > 2016-10-12 17:36 GMT+08:00 Steven Liu : > > > on OSX: > > ../configure --disable-everything --enable-demuxer=hls make > > error message: Undefined symbols for architecture x86_64: > >

Re: [FFmpeg-devel] [PATCH] avformat/udp: deprecate local_port option

2016-10-14 Thread wm4
On Sat, 8 Oct 2016 15:35:02 +0800 Steven Liu wrote: > If you send a patch like this, make sure you do the following things: - explain in the commit message WHY - document the deprecation in the, you know, documentation - document the replacement It should be natural

Re: [FFmpeg-devel] [OPW] OPW Project Proposal

2016-10-14 Thread Pallavi Kumari
Hi Michael, Please find the task at https://github.com/atana1/audio_stream Regards, Pallavi nick - atana On Wed, Oct 12, 2016 at 6:26 AM, Michael Niedermayer wrote: > On Tue, Oct 11, 2016 at 11:54:04PM +0200, Michael Niedermayer wrote: > > On Tue, Oct 11, 2016 at

Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-14 Thread Michael Niedermayer
On Fri, Oct 14, 2016 at 06:09:51PM +, Greg Rowe wrote: > Michael, > > In the attached patch I've tried to make all of the changes you've pointed > out. I also renamed tone_hz to tone_frequency on Moritz Barsnick's > suggestion. > > Is there a good way to generate the tone while avoiding

[FFmpeg-devel] [PATCH] ffmpeg_cleanup: fix crash with unrecognized codec

2016-10-14 Thread James Zern
since: 3e5e5bd Merge commit '398f015f077c6a2406deffd9e37ff34b9c7bb3bc' Signed-off-by: James Zern --- ffmpeg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffmpeg.c b/ffmpeg.c index 693981f..af8ed76 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -533,7 +533,7 @@

Re: [FFmpeg-devel] [PATCH 2/2] lavf/mov: support gopro hero moments udta tag

2016-10-14 Thread Michael Niedermayer
On Mon, Oct 10, 2016 at 12:00:01PM +0200, Jean Caillé wrote: > From: Jean Caillé > > --- > libavformat/mov.c | 19 +++ > 1 file changed, 19 insertions(+) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index add1812..3e73320 100644 > ---

Re: [FFmpeg-devel] [PATCH 1/2] lavf/mov: support gopro firmware udta tag

2016-10-14 Thread Michael Niedermayer
On Mon, Oct 10, 2016 at 12:00:00PM +0200, Jean Caillé wrote: > From: Jean Caillé > > --- > libavformat/mov.c | 1 + > 1 file changed, 1 insertion(+) applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Old school: Use the lowest

Re: [FFmpeg-devel] [PATCH] openssl: Support version 1.1.0.

2016-10-14 Thread Matt Oliver
--- configure | 3 +- libavformat/tls_openssl.c | 163 -- 2 files changed, 102 insertions(+), 64 deletions(-) diff --git a/configure b/configure index e014615..3a92eb3 100755 --- a/configure +++ b/configure @@ -5816,7 +5816,8 @@

Re: [FFmpeg-devel] [PATCH] openssl: Support version 1.1.0.

2016-10-14 Thread Matt Oliver
On 14 October 2016 at 23:04, wm4 wrote: > On Mon, 10 Oct 2016 02:39:51 +1100 > Matt Oliver wrote: > > > --- > > configure | 3 +- > > libavformat/tls_openssl.c | 159 > > -- > > 2 files

Re: [FFmpeg-devel] [PATCH] avformat/udp: deprecate local_port option

2016-10-14 Thread Steven Liu
2016-10-13 20:57 GMT+08:00 Steven Liu : > ping > > 2016-10-12 14:21 GMT+08:00 Steven Liu : > >> >> Signed-off-by: Steven Liu >> --- >> doc/protocols.texi|3 +++ >> libavformat/udp.c | 19 ++- >>

Re: [FFmpeg-devel] [PATCH] avformat/udp: deprecate local_port option

2016-10-14 Thread Michael Niedermayer
On Sat, Oct 15, 2016 at 09:57:40AM +0800, Steven Liu wrote: [...] > and ping Luca Abeni iam not sure he is still maintaining udp, he hasnt been active since many years IIRC [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB There will always be a question for

Re: [FFmpeg-devel] [PATCH] avformat/udp: deprecate local_port option

2016-10-14 Thread Steven Liu
2016-10-15 10:19 GMT+08:00 Michael Niedermayer : > On Sat, Oct 15, 2016 at 09:57:40AM +0800, Steven Liu wrote: > [...] > > and ping Luca Abeni > > iam not sure he is still maintaining udp, he hasnt been active since > many years IIRC > ping Michael :-D > > [...] > -- >

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread wm4
On Thu, 13 Oct 2016 11:27:12 +0200 Nicolas George wrote: > Le duodi 22 vendémiaire, an CCXXV, Steven Liu a écrit : > > init add hds demuxer > > > > Based-on: patch by CORY MCCARTHY > > Based-on: patch by Gorilla Maguila > >

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread Steven Liu
2016-10-14 20:32 GMT+08:00 Carl Eugen Hoyos : > 2016-10-13 16:15 GMT+02:00 Steven Liu : > > > Is this right? Or can you give me a doc link to learn this. > > I am not really a "doc" type, more "trial-and-error", sorry. > Sorry for my abbreviations ,

Re: [FFmpeg-devel] h264_nvenc encoding issue

2016-10-14 Thread Sven C. Dack
On 14/10/16 13:47, tyt xtreme wrote: Hi, I already post my mail to ffmpeg-users group regarding this issue but no one could help me out and advised to ask more experienced users. So i hope you can tell me something and sorry posting here. Here is my re-encoding command's output (Windows 8.1

Re: [FFmpeg-devel] [PATCH] openssl: Support version 1.1.0.

2016-10-14 Thread wm4
On Mon, 10 Oct 2016 02:39:51 +1100 Matt Oliver wrote: > --- > configure | 3 +- > libavformat/tls_openssl.c | 159 > -- > 2 files changed, 98 insertions(+), 64 deletions(-) > > diff --git a/configure

Re: [FFmpeg-devel] [PATCH] Select cubic and lanczos as alternative where super-sampling is not supported

2016-10-14 Thread Sven C. Dack
*ping* The patch didn't seem to have caught anyone's interest. Could somebody please apply it or say what's wrong? Thanks On 09/09/16 10:55, Sven C. Dack wrote: CUDA/NPP doesn't allow super-sampling in some cases and the module then prints an error message "NPP resize error: -23".

Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-14 Thread Moritz Barsnick
> enum SilenceMode { > -SILENCE_TRIM, > +SILENCE_TRIM = 0, > SILENCE_TRIM_FLUSH, > SILENCE_COPY, > SILENCE_COPY_FLUSH, > -SILENCE_STOP > +SILENCE_STOP, > +SILENCE_END_MARKER > +}; > + > +static const char* SILENCE_MODE_NAMES[] = { > +

Re: [FFmpeg-devel] [PATCH]configure: Enable pie for toolchain=hardened.

2016-10-14 Thread Andreas Cadhalpun
On 14.10.2016 15:02, Carl Eugen Hoyos wrote: > 2016-10-04 12:24 GMT+02:00 Carl Eugen Hoyos : > >> Sorry if I miss something but with this patch, the hardening_check >> script succeeds here both for x86_32 and x86_64 (static and shared). > > Tested successfully on x86_64 and

Re: [FFmpeg-devel] [PATCH] libopenjpegenc: recreate image data buffer after encoding frame

2016-10-14 Thread Andreas Cadhalpun
On 14.10.2016 06:07, Michael Bradshaw wrote: > On Thu, Oct 13, 2016 at 12:21 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: >> >> OK. Attached patch does that for openjpeg 2. >> I didn't change the behavior for openjpeg 1, as reusing the image works >> there. > > > Looks good

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread Ricardo Constantino
On 2016-10-14 14:53, Steven Liu wrote: > @@ -295,6 +295,7 @@ External library support: > on OSX if openssl and gnutls are not used > [autodetect] >--enable-x11grab enable X11 grabbing (legacy) [no] >--disable-xlib disable xlib [autodetect] >

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread Nicolas George
Le tridi 23 vendémiaire, an CCXXV, Steven Liu a écrit : > > Rather than NIHing it, I think using libexpat would be a good choice > > (even if its API is terrible). > ok, i'll learn how to use it, and use it next step. :-) While I agree that expat is way better than libxml2 for that task, it is

Re: [FFmpeg-devel] [PATCH] ffmpeg_cleanup: fix crash with unrecognized codec

2016-10-14 Thread Michael Niedermayer
On Fri, Oct 14, 2016 at 12:33:50AM -0700, James Zern wrote: > since: > 3e5e5bd Merge commit '398f015f077c6a2406deffd9e37ff34b9c7bb3bc' > > Signed-off-by: James Zern > --- > ffmpeg.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thx [...] -- Michael

Re: [FFmpeg-devel] [PATCH] libopenjpegenc: fix out-of-bounds reads when filling the edges

2016-10-14 Thread Andreas Cadhalpun
On 14.10.2016 06:08, Michael Bradshaw wrote: > On Thu, Oct 13, 2016 at 6:49 PM, Michael Niedermayer > wrote: >> >>> libopenjpegenc.c | 18 +- >>> 1 file changed, 9 insertions(+), 9 deletions(-) >>> 17061aee3e88729993c9581f688cbfda01fccaac

[FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread Steven Liu
patch update. test passed: Linux: ../configure MingW: ../configure --cc='ccache x86_64-w64-mingw32-gcc' --arch=x86_64 --target-os=mingw32 --cross-prefix=x86_64-w64-mingw32- --disable-yasm test passed with hds demux Linux: ../configure --enable-xml2 OSX: ../configure --disable-everything

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread Steven Liu
2016-10-14 22:31 GMT+08:00 Nicolas George : > Le tridi 23 vendémiaire, an CCXXV, Steven Liu a écrit : > > > Rather than NIHing it, I think using libexpat would be a good choice > > > (even if its API is terrible). > > ok, i'll learn how to use it, and use it next step. :-) > >

[FFmpeg-devel] [PATCH] libavcodec/nvenc.c: add test for Temporal AQ support

2016-10-14 Thread Sven C. Dack
Adds a check to see if the hardware supports temporal aq. >From a68d7b359d53a4d858a25a146489f24a39f4dad0 Mon Sep 17 00:00:00 2001 From: "Sven C. Dack" Date: Fri, 14 Oct 2016 15:12:27 +0100 Subject: [PATCH] libavcodec/nvenc.c: add test for Temporal AQ support ---

Re: [FFmpeg-devel] [OPW] OPW Project Proposal

2016-10-14 Thread Michael Niedermayer
On Sat, Oct 15, 2016 at 03:33:27AM +0530, Pallavi Kumari wrote: > Hi Michael, > > Please find the task at https://github.com/atana1/audio_stream The code should be a avfilter in libavfilter also it should be using the existing fft code [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] RTSP: pass TLS args for RTSPS

2016-10-14 Thread Jay
Thanks to the suggestion from Compn on IRC, I set up a test server for this patch. Once the patch is applied it can be tested as follows. The cert is self signed and ( as with TLS ) the hostname is not verified with openssl. wget

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread Steven Liu
2016-10-15 0:10 GMT+08:00 Steven Liu : > > > 2016-10-15 0:04 GMT+08:00 Ricardo Constantino : > >> >> On 2016-10-14 14:53, Steven Liu wrote: >> > @@ -295,6 +295,7 @@ External library support: >> > on OSX if openssl and gnutls

[FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread Steven Liu
patch update. test passed: Linux: ../configure MingW: ../configure --cc='ccache x86_64-w64-mingw32-gcc' --arch=x86_64 --target-os=mingw32 --cross-prefix=x86_64-w64-mingw32- --disable-yasm test passed with hds demux Linux: ../configure --enable-xml2 OSX: ../configure --disable-everything

Re: [FFmpeg-devel] [PATCH]configure: Enable pie for toolchain=hardened.

2016-10-14 Thread Michael Niedermayer
On Thu, Oct 13, 2016 at 12:56:56AM +0200, Andreas Cadhalpun wrote: > On 12.10.2016 23:44, Carl Eugen Hoyos wrote: > > 2016-10-12 19:04 GMT+02:00 Andreas Cadhalpun > > : > >> On 04.10.2016 12:24, Carl Eugen Hoyos wrote: > >>> Sorry if I miss something but with

Re: [FFmpeg-devel] [PATCH]configure: Enable pie for toolchain=hardened.

2016-10-14 Thread Michael Niedermayer
On Fri, Oct 14, 2016 at 06:28:32PM +0200, Michael Niedermayer wrote: > On Thu, Oct 13, 2016 at 12:56:56AM +0200, Andreas Cadhalpun wrote: > > On 12.10.2016 23:44, Carl Eugen Hoyos wrote: > > > 2016-10-12 19:04 GMT+02:00 Andreas Cadhalpun > > > : > > >> On

[FFmpeg-devel] [PATCH] Improved selftest coverage for libavutil/fifo.c

2016-10-14 Thread Thomas Turner
Tested functions: av_fifo_generic_peek(), av_fifo_grow() Signed-off-by: Thomas Turner --- libavutil/tests/fifo.c | 39 +-- tests/ref/fate/fifo| 43 +++ 2 files changed, 80 insertions(+), 2

[FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-14 Thread Ronald S. Bultje
This is intended to workaround bug "665 Integer Divide Instruction May Cause Unpredictable Behavior" on some early AMD CPUs, which causes a div-by-zero in this codepath, such as reported in Mozilla bug #1293996. Note that this isn't guaranteed to fix the bug, since a compiler is free to reorder

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-10-14 Thread Steven Liu
2016-10-14 20:00 GMT+08:00 wm4 : > On Thu, 13 Oct 2016 11:27:12 +0200 > Nicolas George wrote: > > > Le duodi 22 vendémiaire, an CCXXV, Steven Liu a écrit : > > > init add hds demuxer > > > > > > Based-on: patch by CORY MCCARTHY > >

[FFmpeg-devel] [PATCH 1/2] lavd/decklink_dec: add option to disable drawing bars on signal loss

2016-10-14 Thread Marton Balint
Signed-off-by: Marton Balint --- doc/indevs.texi | 4 libavdevice/decklink_common.h | 1 + libavdevice/decklink_common_c.h | 1 + libavdevice/decklink_dec.cpp| 3 ++- libavdevice/decklink_dec_c.c| 1 + libavdevice/version.h | 2 +- 6 files

[FFmpeg-devel] [PATCH 2/2] lavd/decklink_dec: fix indentation

2016-10-14 Thread Marton Balint
Signed-off-by: Marton Balint --- libavdevice/decklink_dec.cpp | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp index 5318bbe..c98c51f 100644 --- a/libavdevice/decklink_dec.cpp

Re: [FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-14 Thread James Zern
On Fri, Oct 14, 2016 at 11:31 AM, Michael Niedermayer wrote: > On Fri, Oct 14, 2016 at 08:29:37PM +0200, Michael Niedermayer wrote: >> On Fri, Oct 14, 2016 at 11:09:30AM -0700, James Zern wrote: >> > Ronald, >> > >> > On Fri, Oct 14, 2016 at 10:01 AM, Ronald S. Bultje

[FFmpeg-devel] HLS Input -> tee muxer -> HLS+UDP output

2016-10-14 Thread Ibrahim Tachijian
Hey, When using an HLS input transcoding processes can really be speeded up compared to having a udp input simply because you have a few segments cached and can transcode immediately as fast as CPU allows. This means that ffmpeg will also produce the playlist and first segments really quickly.

Re: [FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-14 Thread Ronald S. Bultje
Hi Michael, On Fri, Oct 14, 2016 at 2:31 PM, Michael Niedermayer wrote: > On Fri, Oct 14, 2016 at 08:29:37PM +0200, Michael Niedermayer wrote: > > On Fri, Oct 14, 2016 at 11:09:30AM -0700, James Zern wrote: > > > Ronald, > > > > > > On Fri, Oct 14, 2016 at 10:01 AM,

Re: [FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-14 Thread James Zern
Ronald, On Fri, Oct 14, 2016 at 10:01 AM, Ronald S. Bultje wrote: > This is intended to workaround bug "665 Integer Divide Instruction May > Cause Unpredictable Behavior" on some early AMD CPUs, which causes a > div-by-zero in this codepath, such as reported in Mozilla bug

Re: [FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-14 Thread Michael Niedermayer
On Fri, Oct 14, 2016 at 11:09:30AM -0700, James Zern wrote: > Ronald, > > On Fri, Oct 14, 2016 at 10:01 AM, Ronald S. Bultje wrote: > > This is intended to workaround bug "665 Integer Divide Instruction May > > Cause Unpredictable Behavior" on some early AMD CPUs, which

Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-14 Thread Greg Rowe
Michael, In the attached patch I've tried to make all of the changes you've pointed out. I also renamed tone_hz to tone_frequency on Moritz Barsnick's suggestion. Is there a good way to generate the tone while avoiding floating point operations? If there is then don't bother reviewing this

Re: [FFmpeg-devel] [PATCH]configure: Enable pie for toolchain=hardened.

2016-10-14 Thread Andreas Cadhalpun
On 14.10.2016 18:28, Michael Niedermayer wrote: > On Thu, Oct 13, 2016 at 12:56:56AM +0200, Andreas Cadhalpun wrote: >> If you want both NX and ASLR security features for an executable it has >> to be built with '-pie' and must not contain text relocations. > > this should not be true > the

Re: [FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-14 Thread Michael Niedermayer
On Fri, Oct 14, 2016 at 08:29:37PM +0200, Michael Niedermayer wrote: > On Fri, Oct 14, 2016 at 11:09:30AM -0700, James Zern wrote: > > Ronald, > > > > On Fri, Oct 14, 2016 at 10:01 AM, Ronald S. Bultje > > wrote: > > > This is intended to workaround bug "665 Integer Divide

Re: [FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-14 Thread James Zern
On Fri, Oct 14, 2016 at 11:54 AM, Ronald S. Bultje wrote: > Hi Michael, > > On Fri, Oct 14, 2016 at 2:31 PM, Michael Niedermayer > wrote: > >> On Fri, Oct 14, 2016 at 08:29:37PM +0200, Michael Niedermayer wrote: >> > On Fri, Oct 14, 2016 at 11:09:30AM

Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-14 Thread Moritz Barsnick
On Fri, Oct 14, 2016 at 18:09:51 +, Greg Rowe wrote: > In the attached patch I've tried to make all of the changes you've > pointed out. I also renamed tone_hz to tone_frequency on Moritz > Barsnick's suggestion. You attached the old patch. ;-) > Is there a good way to generate the tone