Re: [FFmpeg-devel] [PATCH 13/17] lavfi: add ff_inlink_set_frame_wanted().

2016-12-30 Thread Michael Niedermayer
On Thu, Dec 29, 2016 at 03:33:59PM +0100, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > libavfilter/avfilter.c | 8 > libavfilter/filters.h | 8 > 2 files changed, 16 insertions(+) > > > Changes in this commit: rename ff_link -> ff_inlink and

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support indexed files

2016-12-30 Thread Michael Niedermayer
On Fri, Dec 30, 2016 at 02:48:12PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #6045. > > Please comment, Carl Eugen > psd.c | 25 + > 1 file changed, 25 insertions(+) > 0881ec94d476ac47ae4d91487d968d6ac82f7992 >

[FFmpeg-devel] patch for Sricam, Floureon, etc. IP cameras

2016-12-30 Thread John Comeau
these cameras return `Transport: RTP/AVP` for a SETUP request of RTP/AVP/TCP, yet they only use TCP transport. `make check` still passes after patch. -- John Comeau KE5TFZ j...@unternet.net http://jc.unternet.net/ "A place for everything, and everything all over the place" diff --git a/TODO.txt

Re: [FFmpeg-devel] [PATCH 06/17] lavfi: add ff_inlink_process_timeline().

2016-12-30 Thread Michael Niedermayer
On Fri, Dec 30, 2016 at 12:37:41PM +0100, Nicolas George wrote: > Le nonidi 9 nivôse, an CCXXV, Michael Niedermayer a écrit : > > maybe the enabledness value should be returned by the function (too) > > It does not seem very useful at first glance, but I may be missing > something. // do some

Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc

2016-12-30 Thread Miroslav Slugeň
Dne 30.12.2016 v 17:38 Mark Thompson napsal(a): On 30/12/16 16:12, Miroslav Slugeň wrote: Dne 30.12.2016 v 15:32 Mark Thompson napsal(a): On 29/12/16 21:02, Miroslav Slugeň wrote: Somebody changed AUD to active in NVENC by default, which is not very clever, libx264 also has this future

Re: [FFmpeg-devel] [PATCH 2/2] avformat/hls: Add subtitle support

2016-12-30 Thread Franklin Phillips
On Fri, Dec 16, 2016 at 05:13:50PM +0100, wm4 wrote: > On Thu, 15 Dec 2016 22:01:46 + > Franklin Phillips wrote: > > > Hi, > > > > I tested this code by converting HLS streams into mp4 files and it > > seemed to work fine. However I recently compiled mpv with these

[FFmpeg-devel] [PATCH 2/2] avformat/hls: Add subtitle support

2016-12-30 Thread Franklin Phillips
Each subtile segment is a WebVTT file and needs to be demuxed separately. These segments also contain a header to synchronize their timing with the MPEG TS stream so those timestamps are requested from the WebVTT demuxer through an AVOption. Signed-off-by: Franklin Phillips

Re: [FFmpeg-devel] [PATCH] configure: Check build with some header not just preprocessing for testing --std=c11

2016-12-30 Thread James Almer
On 12/28/2016 7:41 PM, Michael Niedermayer wrote: > Fixes build failure on FreeBSD with gcc 4.7 > > Signed-off-by: Michael Niedermayer > --- > configure | 9 + > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/configure b/configure > index

Re: [FFmpeg-devel] [PATCH] libavformat/hlsenc: default segment name and, use_localtime

2016-12-30 Thread Bodecs Bela
2016.12.30. 18:11 keltezéssel, Moritz Barsnick írta: On Fri, Dec 30, 2016 at 15:38:25 +0100, Bodecs Bela wrote: is not available on all system/environment. This patch checks %s availabilty at runtine and alter the default format string if necessary. You forgot to add the patch. please,

Re: [FFmpeg-devel] [PATCH] avcodec/hevc: Extract Content light level from HEVC SEI prefix message

2016-12-30 Thread Moritz Barsnick
On Fri, Dec 30, 2016 at 15:27:14 +0100, Nikola Kolarović wrote: > Follow-up patch is below, and I've run FATE regression tests on x86_64 > linux. You'll need to resend it properly, it is corrupted by newlines (probably by your mailer). Moritz ___

Re: [FFmpeg-devel] [PATCH] libavformat/hlsenc: default segment name and, use_localtime

2016-12-30 Thread Moritz Barsnick
On Fri, Dec 30, 2016 at 15:38:25 +0100, Bodecs Bela wrote: > is not available on all system/environment. This patch checks %s > availabilty at runtine and alter the default format string if necessary. You forgot to add the patch. Moritz ___

[FFmpeg-devel] [PATCH 1/2] lavc/libmp3lame: add support for cutoff

2016-12-30 Thread Moritz Barsnick
Pass the cutoff option from lavc's avcodec_options[] to libmp3lame's lowpass option, without allowing to adjust its default behavior. Signed-off-by: Moritz Barsnick --- doc/encoders.texi | 4 libavcodec/libmp3lame.c | 4 2 files changed, 8 insertions(+) diff

[FFmpeg-devel] [PATCH 0/2] document cutoff option properly, and add support to libmp3lame

2016-12-30 Thread Moritz Barsnick
Hi, this pair of commits adds cutoff support to libmp3lame, and documents the option "global" to lavc such that it is clear that it is *not* generally available. Noted and requested in this thread: http://ffmpeg.org/pipermail/ffmpeg-user/2016-December/034703.html Testing: Encoded an MP3 without

[FFmpeg-devel] [PATCH 2/2] doc: document cutoff option to ac3 and adjust the option's global documentation

2016-12-30 Thread Moritz Barsnick
cutoff is implemented as an option global to lavc, but supported only by a few encoders. This fact is now reflected in its documentation. ac3's support of this option is added for completeness. Signed-off-by: Moritz Barsnick --- doc/codecs.texi | 3 ++- doc/encoders.texi | 4

Re: [FFmpeg-devel] [PATCH] avcodec/hevc: Extract Content light level from HEVC SEI prefix message

2016-12-30 Thread Nikola Kolarović
On 12/29/2016 07:46 PM, Nicolas George wrote: Thanks for the patch. L'octidi 8 nivôse, an CCXXV, Nikola Kolarović a écrit : From: Nikola Kolarović Extract max_content_light_level and max_frame_average_light_level which are used in HEVC Main 10 (HDR10) for

Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc

2016-12-30 Thread Mark Thompson
On 30/12/16 16:12, Miroslav Slugeň wrote: > Dne 30.12.2016 v 15:32 Mark Thompson napsal(a): >> On 29/12/16 21:02, Miroslav Slugeň wrote: >>> Somebody changed AUD to active in NVENC by default, which is not very >>> clever, libx264 also has this future disabled, so we should stay in sync >>> with

Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc

2016-12-30 Thread Miroslav Slugeň
Dne 30.12.2016 v 15:32 Mark Thompson napsal(a): On 29/12/16 21:02, Miroslav Slugeň wrote: Somebody changed AUD to active in NVENC by default, which is not very clever, libx264 also has this future disabled, so we should stay in sync with libx264 behavior. Enabled AUD will work only without

[FFmpeg-devel] [PATCH] libavformat/hlsenc: default segment name and, use_localtime

2016-12-30 Thread Bodecs Bela
Dear All, in hlcenc.c, in the hls_write_header() function the default format string for strftime() function contains %s specifier when use_localtime is true. This %s specifier will insert the seconds since EPOCH. But %s is not available on all system/environment. This patch checks %s

Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc

2016-12-30 Thread Mark Thompson
On 29/12/16 21:02, Miroslav Slugeň wrote: > Somebody changed AUD to active in NVENC by default, which is not very clever, > libx264 also has this future disabled, so we should stay in sync with libx264 > behavior. > > Enabled AUD will work only without B-frames. There is BUG in Nvidia NVENC >

Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc

2016-12-30 Thread Mark Thompson
On 30/12/16 11:00, Ali KIZIL wrote: > 2016-12-30 12:07 GMT+03:00 Miroslav Slugeň : > >> Dne 30.12.2016 v 05:48 Ali KIZIL napsal(a): >> >>> 2016-12-30 0:02 GMT+03:00 Miroslav Slugeň : >>> >>> >>> Somebody changed AUD to active in NVENC by default, which is

Re: [FFmpeg-devel] libavformat/hlsenc + configure: question about default strftime() format string

2016-12-30 Thread Bodecs Bela
2016.12.29. 3:33 keltezéssel, Steven Liu írta: 2016-12-29 8:32 GMT+08:00 Bodecs Bela : Dear All, in hlcenc.c, in the hls_write_header() function the default format string for strftime() function contains %s specifier. const char *pattern_localtime_fmt = "-%s.ts"; This

[FFmpeg-devel] [PATCH]lavc/psd: Support indexed files

2016-12-30 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #6045. Please comment, Carl Eugen From 0bd8d59e1ad0009f88aa6871d4ed915a12f5900d Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Fri, 30 Dec 2016 14:45:36 +0100 Subject: [PATCH] lavc/psd: Support indexed files. Fixes ticket #6045. ---

Re: [FFmpeg-devel] [PATCH] avutil/tests: Added selftest for aes_ctr.c

2016-12-30 Thread Clément Bœsch
On Fri, Dec 30, 2016 at 01:36:35AM -0800, Thomas Turner wrote: [...] > +const DECLARE_ALIGNED(8, uint8_t, plain)[] = { > +0x6d, 0x6f, 0x73, 0x74, ' ', 0x72, 0x61, 0x6e, 0x64, 0x6f, 0x6d > +}; "most random"? why only the space in ascii? [...] -- Clément B.

Re: [FFmpeg-devel] [PATCH 06/17] lavfi: add ff_inlink_process_timeline().

2016-12-30 Thread Nicolas George
Le decadi 10 nivôse, an CCXXV, Clement Boesch a écrit : > You want a watermark to be effective only in a timerange, the second input > (the watermark) would get ignored outside that range, and the first one > pass through. This would be filter specific (FLAG_SUPPORT_TIMELINE_INTERNAL). > > That's

Re: [FFmpeg-devel] [PATCH 06/17] lavfi: add ff_inlink_process_timeline().

2016-12-30 Thread Clément Bœsch
On Fri, Dec 30, 2016 at 12:37:41PM +0100, Nicolas George wrote: [...] > In fact, I do not see how timeline could make sense for a filter with > several inputs. "Disabling" a filter means replacing it by the > pass-through filter "null", but it has a single input and output. > You want a

Re: [FFmpeg-devel] [PATCH 06/17] lavfi: add ff_inlink_process_timeline().

2016-12-30 Thread Nicolas George
Le nonidi 9 nivôse, an CCXXV, Michael Niedermayer a écrit : > maybe the enabledness value should be returned by the function (too) It does not seem very useful at first glance, but I may be missing something. > how does this work with multiple inlinks ? > (dstctx / dstctx->enable /

Re: [FFmpeg-devel] [PATCH 05/17] lavfi: add ff_inlink_process_commands().

2016-12-30 Thread Nicolas George
Le nonidi 9 nivôse, an CCXXV, Michael Niedermayer a écrit : > avfilter_process_command() has a return code, should this be checked > or passed on somehow ? Currently, the return code of avfilter_process_command() is ignored here. Since this is a refactoring commit, I do not want to introduce

Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc

2016-12-30 Thread Ali KIZIL
2016-12-30 12:07 GMT+03:00 Miroslav Slugeň : > Dne 30.12.2016 v 05:48 Ali KIZIL napsal(a): > >> 2016-12-30 0:02 GMT+03:00 Miroslav Slugeň : >> >> >> Somebody changed AUD to active in NVENC by default, which is not very >>> clever, libx264 also has this

[FFmpeg-devel] [PATCH] avutil/tests: Added selftest for aes_ctr.c

2016-12-30 Thread Thomas Turner
Signed-off-by: Thomas Turner --- libavutil/Makefile| 1 + libavutil/tests/aes_ctr.c | 70 +++ tests/fate/libavutil.mak | 5 3 files changed, 76 insertions(+) create mode 100644 libavutil/tests/aes_ctr.c diff

Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc

2016-12-30 Thread Miroslav Slugeň
Dne 30.12.2016 v 05:48 Ali KIZIL napsal(a): 2016-12-30 0:02 GMT+03:00 Miroslav Slugeň : Somebody changed AUD to active in NVENC by default, which is not very clever, libx264 also has this future disabled, so we should stay in sync with libx264 behavior. Enabled AUD will

Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc

2016-12-30 Thread Miroslav Slugeň
Dne 30.12.2016 v 05:48 Ali KIZIL napsal(a): 2016-12-30 0:02 GMT+03:00 Miroslav Slugeň : Somebody changed AUD to active in NVENC by default, which is not very clever, libx264 also has this future disabled, so we should stay in sync with libx264 behavior. Enabled AUD will