[FFmpeg-devel] [PATCH] lavc/qsvdec: hw device should be set

2017-12-28 Thread Zhong Li
Add the flag "AV_CODEC_HW_CONFIG_METHOD_HW_DEVICE_CTX" to indicate AVCodecContext.hw_device_ctx should be set before calling avcodec_open2() for qsv decoding. It is consistent with examples/qsvdec.c. It also can make function "hw_device_match_by_codec()" can find qsv device successfully.

Re: [FFmpeg-devel] [PATCH v2] avformat/hlsenc: Add CODECS attribute to master playlist

2017-12-28 Thread Jeyapal, Karthick
On 12/28/17 5:17 PM, Karthick J wrote: > From: Karthick Jeyapal > > --- > libavformat/dashenc.c | 2 +- > libavformat/hlsenc.c | 67 > ++- > libavformat/hlsplaylist.c | 5 +++- > libavformat/hlsplaylist.h | 3 ++- >

[FFmpeg-devel] [PATCH v3] avformat/hlsenc: Add CODECS attribute to master playlist

2017-12-28 Thread Karthick J
From: Karthick Jeyapal --- libavformat/dashenc.c | 2 +- libavformat/hlsenc.c | 67 ++- libavformat/hlsplaylist.c | 5 +++- libavformat/hlsplaylist.h | 3 ++- 4 files changed, 73 insertions(+), 4 deletions(-) diff

Re: [FFmpeg-devel] [PATCH 3/3] avformat/dashenc: Addition of #EXT-X-MEDIA tag and AUDIO attribute

2017-12-28 Thread Jeyapal, Karthick
On 12/29/17 12:13 PM, 刘歧 wrote: > >> On 27 Dec 2017, at 15:26, Steven Liu wrote: >> >> 2017-12-26 19:11 GMT+08:00 Karthick J : >>> From: Karthick Jeyapal >>> >>> This is required for AV playout from master.m3u8. >>> Otherwise

Re: [FFmpeg-devel] [PATCH 3/3] avformat/dashenc: Addition of #EXT-X-MEDIA tag and AUDIO attribute

2017-12-28 Thread 刘歧
> On 27 Dec 2017, at 15:26, Steven Liu wrote: > > 2017-12-26 19:11 GMT+08:00 Karthick J : >> From: Karthick Jeyapal >> >> This is required for AV playout from master.m3u8. >> Otherwise master.m3u8 lists only video-only and/or

Re: [FFmpeg-devel] [PATCH] configure: check SDL2 function with a header

2017-12-28 Thread KO Myung-Hun
Hi/2. Derek Buitenhuis wrote: > On 12/28/2017 2:44 PM, KO Myung-Hun wrote: >> On OS/2, '_' is not prepended to a function name. >> --- >> configure | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > It's not immediately clear to be how checking a header instead relates > to

Re: [FFmpeg-devel] [PATCH] compat/os2threads: support static mutexes

2017-12-28 Thread KO Myung-Hun
Hi/2. wm4 wrote: > On Thu, 28 Dec 2017 22:03:56 +0900 > KO Myung-Hun wrote: > >> --- >> compat/os2threads.h | 14 -- >> 1 file changed, 8 insertions(+), 6 deletions(-) >> >> diff --git a/compat/os2threads.h b/compat/os2threads.h >> index 40a119ffe1..2177a033ec

Re: [FFmpeg-devel] [PATCH] h264: add AVOption to set x264_build default

2017-12-28 Thread Derek Buitenhuis
On 12/28/2017 10:19 PM, Carl Eugen Hoyos wrote: > I care much less about what he says (did he really talk to you?) or > writes, more about what he does. For the record: Nobody talked to me. There is no conspiracy. - Derek ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [MSVC toolchain] Patch to allow building FFmpeg with Linux bash on Windows (WSL)

2017-12-28 Thread Aaron Levinson
On 12/28/2017 6:43 PM, Cyber Sinh wrote: The attached patch changes the configure script for FFmpeg (and associated shell scripts) to call MSVC tools including their extensions (cl.exe instead of cl for example). This is necessary, because WSL can automatically launch Windows processes from the

Re: [FFmpeg-devel] [PATCH] avfilter/vidstab: set bytesPerPixel only for packed formats.

2017-12-28 Thread Michael Niedermayer
On Thu, Dec 28, 2017 at 12:50:00PM -0900, Lou Logan wrote: > On Thu, Dec 28, 2017, at 11:51 AM, Michael Niedermayer wrote: > > > > who maintains vf_vidstabtransform.c ? > > Realistically, nobody. Ideally, should have been Georg Martius who is the > original author of the filter and also of

[FFmpeg-devel] [MSVC toolchain] Patch to allow building FFmpeg with Linux bash on Windows (WSL)

2017-12-28 Thread Cyber Sinh
The attached patch changes the configure script for FFmpeg (and associated shell scripts) to call MSVC tools including their extensions (cl.exe instead of cl for example). This is necessary, because WSL can automatically launch Windows processes from the Linux side but only if the process is in

Re: [FFmpeg-devel] PR new filter gltransition

2017-12-28 Thread Rostislav Pehlivanov
On 27 December 2017 at 23:13, Travis Fischer wrote: > Hello, > > This is my first time contributing to ffmpeg, so please go easy on me :) > > I have a git diff here > which add a > new filter to apply a GLSL

[FFmpeg-devel] [PATCH] avcodec/exr: Check buf_size more completely

2017-12-28 Thread Michael Niedermayer
Fixes: Out of heap array read Fixes: 4683/clusterfuzz-testcase-minimized-6152313673613312 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/exr.c | 8 1 file

[FFmpeg-devel] [PATCH v2] libavcodec/mediacodecdec: switch to new decoder api

2017-12-28 Thread Aman Gupta
From: Aman Gupta Using the new API gives the decoder the ability to produce N frames per input packet. This is particularly useful with mpeg2 decoders on some android devices, which automatically deinterlace video and produce one frame per field. Signed-off-by: Aman Gupta

Re: [FFmpeg-devel] [PATCH]configure: libvmaf depends on pthreads

2017-12-28 Thread Carl Eugen Hoyos
2017-11-26 15:10 GMT+01:00 Nicolas George : > -> our code depends on pthreads for this filters, it must be expressed > in configure: Carl Eugen's patch is right Pushed with pthreads, as this is currently used. Thank you, Carl Eugen

Re: [FFmpeg-devel] [PATCH]lavf/mov: Do not blindly allocate stts entries

2017-12-28 Thread Carl Eugen Hoyos
2017-11-28 21:32 GMT+01:00 Michael Niedermayer : > On Mon, Nov 27, 2017 at 05:24:14AM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch avoids allocations >1GB for (short and) invalid mov >> files with only reasonable speed impact. >> >> Please review, Carl Eugen >

[FFmpeg-devel] [PATCH] libavcodec/mediacodecdec: update to use new decoding api

2017-12-28 Thread Aman Gupta
From: Aman Gupta --- libavcodec/mediacodecdec.c | 75 ++ 1 file changed, 43 insertions(+), 32 deletions(-) diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c index b698ceaef9..29f1076bb0 100644 ---

Re: [FFmpeg-devel] [PATCH] h264: add AVOption to set x264_build default

2017-12-28 Thread Hendrik Leppkes
On Thu, Dec 28, 2017 at 11:19 PM, Carl Eugen Hoyos wrote: > 2017-12-28 16:43 GMT+01:00 Derek Buitenhuis : >> On 12/28/2017 12:44 AM, Carl Eugen Hoyos wrote: out of nowhere without being provoked is not a welcome behavior at all. >>> >>> That

Re: [FFmpeg-devel] [PATCH] h264: add AVOption to set x264_build default

2017-12-28 Thread Carl Eugen Hoyos
2017-12-28 16:43 GMT+01:00 Derek Buitenhuis : > On 12/28/2017 12:44 AM, Carl Eugen Hoyos wrote: >>> out of nowhere without being provoked is not a welcome behavior at all. >> >> That doesn't sound correct to me. > > He literally said nothing to you on this thread before

Re: [FFmpeg-devel] [PATCH] reinterlace filter

2017-12-28 Thread Carl Eugen Hoyos
2017-12-28 13:52 GMT+01:00 Vasile Toncu : > diff --git a/libavfilter/Makefile b/libavfilter/Makefile > index 8916588..b2e9a50 100644 > --- a/libavfilter/Makefile > +++ b/libavfilter/Makefile > @@ -213,6 +213,7 @@ OBJS-$(CONFIG_PULLUP_FILTER) += >

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-28 Thread Carl Eugen Hoyos
2017-12-28 16:12 GMT+01:00 Vasile Toncu : > Is there anything that I can do to the current version of reinterlace > so that the filter can be accepted? Wasn't this already explained? Add the existing (GPL'd) asm optimizations to your new filter (and make sure the filter

Re: [FFmpeg-devel] Licensing question.

2017-12-28 Thread Carl Eugen Hoyos
2017-12-27 22:17 GMT+01:00 Lewis Hornsby : > I would like to include libffmpeg.dylib in the application. Please read http://ffmpeg.org/legal.html or post your license questions on the user mailing list, do not top-post there! Carl Eugen

Re: [FFmpeg-devel] PR new filter gltransition

2017-12-28 Thread Carl Eugen Hoyos
2017-12-28 0:13 GMT+01:00 Travis Fischer : > This is my first time contributing to ffmpeg, so please go easy on me :) > > I have a git diff here > The new file needs a license header (please copy it from another

Re: [FFmpeg-devel] [PATCH] avfilter/vidstab: set bytesPerPixel only for packed formats.

2017-12-28 Thread Lou Logan
On Thu, Dec 28, 2017, at 11:51 AM, Michael Niedermayer wrote: > > who maintains vf_vidstabtransform.c ? Realistically, nobody. Ideally, should have been Georg Martius who is the original author of the filter and also of vid.stab itself. Consider CCing him at "martius at mis.mpg.de" or

Re: [FFmpeg-devel] [PATCH] avfilter/vidstab: set bytesPerPixel only for packed formats.

2017-12-28 Thread Michael Niedermayer
On Thu, Dec 28, 2017 at 09:52:20AM +0530, Gyan Doshi wrote: > On 12/24/2017 10:36 AM, Gyan Doshi wrote: > > > >On 12/23/2017 4:39 PM, Gyan Doshi wrote: > >>Patch for ticket #6736. > > Ping. who maintains vf_vidstabtransform.c ? [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/flacdec: avoid undefined shift

2017-12-28 Thread Michael Niedermayer
On Tue, Dec 26, 2017 at 11:24:43PM +0100, Michael Niedermayer wrote: > Fixes: shift exponent 32 is too large for 32-bit type 'unsigned int' > Fixes: 4688/clusterfuzz-testcase-minimized-6572210748653568 > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH] Add android_capture indev

2017-12-28 Thread Michael Niedermayer
On Fri, Dec 22, 2017 at 09:36:59PM +0100, Felix Matouschek wrote: > Am 22.12.2017 20:50, schrieb Lou Logan: > > > >I think you forgot to attach the patch. > > Sorry, flaky mail client... attached it again. [...] > +static void image_available(void *context, AImageReader *reader) > +{ > +

Re: [FFmpeg-devel] Licensing question.

2017-12-28 Thread Lewis Hornsby
I would like to include libffmpeg.dylib in the application. FFMPEG is used only for previewing mp4 videos inside the application. Thanks for your time. Lewis > On 27 Dec 2017, at 21:14, Lou Logan wrote: > > On Wed, 27 Dec 2017 19:48:52 + > Lewis Hornsby

[FFmpeg-devel] PR new filter gltransition

2017-12-28 Thread Travis Fischer
Hello, This is my first time contributing to ffmpeg, so please go easy on me :) I have a git diff here which add a new filter to apply a GLSL transition between two video streams. The github repo for this patch includes all of the

Re: [FFmpeg-devel] [PATCH] lavr: deprecate the entire library

2017-12-28 Thread wm4
On Thu, 28 Dec 2017 13:25:45 -0300 James Almer wrote: > On 12/25/2017 2:53 PM, Rostislav Pehlivanov wrote: > > diff --git a/libavresample/avresample.h b/libavresample/avresample.h > > index 193443e2a6..57889c572c 100644 > > --- a/libavresample/avresample.h > > +++

Re: [FFmpeg-devel] [PATCH] lavr: deprecate the entire library

2017-12-28 Thread James Almer
On 12/25/2017 2:53 PM, Rostislav Pehlivanov wrote: > diff --git a/libavresample/avresample.h b/libavresample/avresample.h > index 193443e2a6..57889c572c 100644 > --- a/libavresample/avresample.h > +++ b/libavresample/avresample.h > @@ -103,10 +103,10 @@ > > #define AVRESAMPLE_MAX_CHANNELS 32 >

Re: [FFmpeg-devel] [PATCH] lavr: deprecate the entire library

2017-12-28 Thread Derek Buitenhuis
On 12/25/2017 5:53 PM, Rostislav Pehlivanov wrote: > Deprecate the entire library. Merged years ago to provide compatibility > with Libav, it remained unmaintained by the FFmpeg project and duplicated > functionality provided by libswresample. > > In order to improve consistency and reduce attack

Re: [FFmpeg-devel] [PATCH 1/2] lavu/qsv: add log message for libmfx version

2017-12-28 Thread Derek Buitenhuis
On 12/28/2017 9:27 AM, Zhong Li wrote: > +av_log(ctx, AV_LOG_VERBOSE, > + "Initialize MFX session: API version is %d.%d, implementation > version is %d.%d\n", > + MFX_VERSION_MAJOR, MFX_VERSION_MINOR, ver.Major, ver.Minor); Maybe AV_LOG_DEBUG? I have no strong opinion,

Re: [FFmpeg-devel] [PATCH 2/2] examples/qsvdec: remove the deprecated filed refcounted_frames

2017-12-28 Thread Derek Buitenhuis
On 12/28/2017 9:27 AM, Zhong Li wrote: > It was just useful for deprecated API (avcodec_decode_video2), > but useless for new decode APIs (avcodec_send_packet/avcodec_receive_frame). > > Signed-off-by: Zhong Li > --- > doc/examples/qsvdec.c | 1 - > 1 file changed, 1

Re: [FFmpeg-devel] [PATCH] configure: check SDL2 function with a header

2017-12-28 Thread Derek Buitenhuis
On 12/28/2017 2:44 PM, KO Myung-Hun wrote: > On OS/2, '_' is not prepended to a function name. > --- > configure | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) It's not immediately clear to be how checking a header instead relates to function name mangling (or lack thereof)? -

Re: [FFmpeg-devel] [PATCH] h264: add AVOption to set x264_build default

2017-12-28 Thread Derek Buitenhuis
On 12/28/2017 12:44 AM, Carl Eugen Hoyos wrote: >> out of nowhere without being provoked is not a welcome behavior at all. > > That doesn't sound correct to me. He literally said nothing to you on this thread before you insulted him. Cut it out. - Derek

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-28 Thread Nicolas George
Vasile Toncu (2017-12-28): > Is submision to an older version > possible? No. Branches are only maintained for bugs. New development only happens on Git head. Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-28 Thread Vasile Toncu
On Thu, 28 Dec 2017 at 17:14, Nicolas George wrote: > Vasile Toncu (2017-12-28): > > Is there anything that I can do to the current version of reinterlace so > > that the filter can be accepted? > > For me, yes: make it fully compatible with tinterlace and call it that. > The

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-28 Thread Nicolas George
Vasile Toncu (2017-12-28): > Is there anything that I can do to the current version of reinterlace so > that the filter can be accepted? For me, yes: make it fully compatible with tinterlace and call it that. Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-28 Thread Vasile Toncu
On Thu, 28 Dec 2017 at 15:41, Nicolas George wrote: > Vasile Toncu (2017-12-28): > > There are some features that tinterlace has and reinterlace does not. > > > > Tinterlace has some more flags, reinterlace has only simple low pass > filter. > > Also, in a newer version,

[FFmpeg-devel] [PATCH] configure: check SDL2 function with a header

2017-12-28 Thread KO Myung-Hun
On OS/2, '_' is not prepended to a function name. --- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 70de780f82..f937ddf2be 100755 --- a/configure +++ b/configure @@ -6019,14 +6019,14 @@ fi if enabled sdl2; then

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-28 Thread Vasile Toncu
On 28.12.2017 15:17, Nicolas George wrote: Vasile Toncu (2017-12-28): Are there any features that tinterlace has and this new filter has not? If not, then I think it would be better to just replace tinterlace entirely. You did not answer the question. I asked if there are features in

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-28 Thread Nicolas George
Vasile Toncu (2017-12-28): > There are some features that tinterlace has and reinterlace does not. > > Tinterlace has some more flags, reinterlace has only simple low pass filter. > Also, in a newer version, tinterlace does support input formats on 16 bit. > > Expecting these two, there are no

Re: [FFmpeg-devel] [PATCH] compat/os2threads: support static mutexes

2017-12-28 Thread wm4
On Thu, 28 Dec 2017 22:03:56 +0900 KO Myung-Hun wrote: > --- > compat/os2threads.h | 14 -- > 1 file changed, 8 insertions(+), 6 deletions(-) > > diff --git a/compat/os2threads.h b/compat/os2threads.h > index 40a119ffe1..2177a033ec 100644 > ---

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-28 Thread Nicolas George
Vasile Toncu (2017-12-28): > >Are there any features that tinterlace has and this new filter has not? > >If not, then I think it would be better to just replace tinterlace > >entirely. > The main difference between those those two filters is that reinterlace > processes > planes of output frames

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-28 Thread Vasile Toncu
On 27.12.2017 15:31, Nicolas George wrote: Vasile Toncu (2017-12-27): It is because of licencing issues. I wanted the new filter to be LGPL. Thanks for the explanation. That is a valid reason. But as is, it would result would be duplicated code for people who do not worry about licensing.

[FFmpeg-devel] [PATCH] compat/os2threads: support static mutexes

2017-12-28 Thread KO Myung-Hun
--- compat/os2threads.h | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/compat/os2threads.h b/compat/os2threads.h index 40a119ffe1..2177a033ec 100644 --- a/compat/os2threads.h +++ b/compat/os2threads.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2011 KO Myung-Hun

Re: [FFmpeg-devel] [PATCH] reinterlace filter

2017-12-28 Thread Nicolas George
Vasile Toncu (2017-12-28): > Reinterlace filter is a new video filter that does various > interlace/interleave/merge ops between lines of consecutive frames. > > It adds two new modes to the already existing tinterlace filter, > merge_bff and merge_tff. > > It is under LGPL license and starts

[FFmpeg-devel] [PATCH] reinterlace filter

2017-12-28 Thread Vasile Toncu
Hello, Reinterlace filter is a new video filter that does various interlace/interleave/merge ops between lines of consecutive frames. It adds two new modes to the already existing tinterlace filter, merge_bff and merge_tff. It is under LGPL license and starts from ffmpeg version n3.0.2. This

[FFmpeg-devel] [PATCH v2] avformat/hlsenc: Add CODECS attribute to master playlist

2017-12-28 Thread Karthick J
From: Karthick Jeyapal --- libavformat/dashenc.c | 2 +- libavformat/hlsenc.c | 67 ++- libavformat/hlsplaylist.c | 5 +++- libavformat/hlsplaylist.h | 3 ++- 4 files changed, 73 insertions(+), 4 deletions(-) diff

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Add CODECS attribute to master playlist

2017-12-28 Thread 刘歧
> On 28 Dec 2017, at 19:19, Karthick J wrote: > > From: Karthick Jeyapal > > --- > libavformat/dashenc.c | 2 +- > libavformat/hlsenc.c | 67 ++- > libavformat/hlsplaylist.c | 5 +++- >

[FFmpeg-devel] [PATCH] avformat/hlsenc: Add CODECS attribute to master playlist

2017-12-28 Thread Karthick J
From: Karthick Jeyapal --- libavformat/dashenc.c | 2 +- libavformat/hlsenc.c | 67 ++- libavformat/hlsplaylist.c | 5 +++- libavformat/hlsplaylist.h | 3 ++- 4 files changed, 73 insertions(+), 4 deletions(-) diff

[FFmpeg-devel] [PATCH 2/2] examples/qsvdec: remove the deprecated filed refcounted_frames

2017-12-28 Thread Zhong Li
It was just useful for deprecated API (avcodec_decode_video2), but useless for new decode APIs (avcodec_send_packet/avcodec_receive_frame). Signed-off-by: Zhong Li --- doc/examples/qsvdec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/doc/examples/qsvdec.c

[FFmpeg-devel] [PATCH 1/2] lavu/qsv: add log message for libmfx version

2017-12-28 Thread Zhong Li
It is benefit to diagnose issues related to different libmfx version. Signed-off-by: Zhong Li --- libavutil/hwcontext_qsv.c | 4 1 file changed, 4 insertions(+) diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c index 9b6040b..6228c04 100644 ---

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

2017-12-28 Thread Paul B Mahol
On 12/27/17, Tomas Haerdin wrote: > ons 2017-12-27 klockan 19:21 +0100 skrev Paul B Mahol: >> > >> +static int filter_frame(AVFilterLink *inlink, AVFrame *in) >> +{ >> +AVFilterContext *ctx = inlink->dst; >> +AVFilterLink *outlink = ctx->outputs[0]; >> +