Re: [FFmpeg-devel] [PATCH 0/4] ffplay and lavd SDL2 set

2016-09-14 Thread Josh de Kock
On 14/09/2016 23:44, Thomas Volkert wrote: On 15.09.2016 00:27, Josh de Kock wrote: Hi, Resending this set with ffplay now having two versions, a SDL2 and a SDL1 version. I've integrated all comments up until now (hopefully). Josh Josh de Kock (3): lavd: Add SDL2 output device ffplay: m

Re: [FFmpeg-devel] [PATCH] doc/muxers: add flv muxer document into doc/muxers

2016-09-14 Thread Steven Liu
2016-09-15 7:30 GMT+08:00 Lou Logan : > On Wed, 14 Sep 2016 10:25:01 +0800, Steven Liu wrote: > > > From eb8856c6296eedb97a903028a93b2b1b23c50c97 Mon Sep 17 00:00:00 2001 > > From: Steven Liu > > Date: Wed, 14 Sep 2016 10:20:51 +0800 > > Subject: [PATCH] doc/muxers: add flv muxer document into do

[FFmpeg-devel] [PATCH 2/2] cuvid: Check for non 420 chroma formats - they aren't supported

2016-09-14 Thread Philip Langdale
Despite the video parser seeming to correctly handle 422 and 444 chroma formats, the video decoder fails miserably to actually decode frames - even though no errors are ever returned; you just get frames showing unintialized garbage. Signed-off-by: Philip Langdale --- libavcodec/cuvid.c | 6

[FFmpeg-devel] [PATCH 0/2] cuvid: Misc improvements

2016-09-14 Thread Philip Langdale
A couple of cuvid improvements Philip Langdale (2): cuvid: Fully re-initialize the parser after a flush. cuvid: Check for non 420 chroma formats - they aren't supported libavcodec/cuvid.c | 16 1 file changed, 16 insertions(+) -- 2.7.4 _

[FFmpeg-devel] [PATCH 1/2] cuvid: Fully re-initialize the parser after a flush.

2016-09-14 Thread Philip Langdale
I'm not really sure how this worked at all before, but we do need to reinitalize the parser with the stream extradata. Signed-off-by: Philip Langdale --- libavcodec/cuvid.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavcodec/cuvid.c b/libavcodec/cuvid.c index 19a7772..880a

Re: [FFmpeg-devel] [PATCH] doc/muxers: add flv muxer document into doc/muxers

2016-09-14 Thread Lou Logan
On Wed, 14 Sep 2016 10:25:01 +0800, Steven Liu wrote: > From eb8856c6296eedb97a903028a93b2b1b23c50c97 Mon Sep 17 00:00:00 2001 > From: Steven Liu > Date: Wed, 14 Sep 2016 10:20:51 +0800 > Subject: [PATCH] doc/muxers: add flv muxer document into doc/muxers > > add flvflags aac_seq_header_detect a

Re: [FFmpeg-devel] [PATCH] doc/muxers: add hls_segment_size option document

2016-09-14 Thread Lou Logan
On Wed, 14 Sep 2016 07:40:08 +0800, Steven Liu wrote: > From 97ff17a17db5ecac295327281738f044239cf322 Mon Sep 17 00:00:00 2001 > From: Steven Liu > Date: Wed, 14 Sep 2016 07:14:18 +0800 > Subject: [PATCH] doc/muxers: add hls_segment_size option document > > and make an sample introduce how to us

Re: [FFmpeg-devel] [PATCH 1/4] lavd: Add SDL2 output device

2016-09-14 Thread James Almer
On 9/14/2016 7:27 PM, Josh de Kock wrote: > Acked-by: Michael Niedermayer > Signed-off-by: Josh de Kock > --- > configure| 28 +++- > libavdevice/Makefile | 1 + > libavdevice/alldevices.c | 1 + > libavdevice/sdl2.c | 377 >

Re: [FFmpeg-devel] [PATCH 3/4] ffplay: add SDL2 support

2016-09-14 Thread James Almer
On 9/14/2016 7:27 PM, Josh de Kock wrote: > diff --git a/ffplay_sdl1.c b/ffplay_sdl1.c > index adbe9cb..d698420 100644 > --- a/ffplay_sdl1.c > +++ b/ffplay_sdl1.c > @@ -23,7 +23,6 @@ > * simple media player based on the FFmpeg libraries > */ > > -#include "config.h" Why? It's needed below, a

Re: [FFmpeg-devel] [PATCH 0/4] ffplay and lavd SDL2 set

2016-09-14 Thread James Almer
On 9/14/2016 7:44 PM, Thomas Volkert wrote: > On 15.09.2016 00:27, Josh de Kock wrote: >> Hi, >> >> Resending this set with ffplay now having two versions, a SDL2 and a >> SDL1 version. I've integrated all comments up until now (hopefully). >> >> Josh >> >> Josh de Kock (3): >>lavd: Add SDL2 ou

Re: [FFmpeg-devel] [PATCH 0/4] ffplay and lavd SDL2 set

2016-09-14 Thread Thomas Volkert
On 15.09.2016 00:27, Josh de Kock wrote: Hi, Resending this set with ffplay now having two versions, a SDL2 and a SDL1 version. I've integrated all comments up until now (hopefully). Josh Josh de Kock (3): lavd: Add SDL2 output device ffplay: make copy for SDL1 MAINTAINERS: update my

[FFmpeg-devel] [PATCH 4/4] MAINTAINERS: update my entries

2016-09-14 Thread Josh de Kock
Signed-off-by: Josh de Kock --- MAINTAINERS | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index f7e8298..52d8eed 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -24,7 +24,8 @@ ffmpeg: ffmpeg.c Michael Niedermayer

[FFmpeg-devel] [PATCH 2/4] ffplay: make copy for SDL1

2016-09-14 Thread Josh de Kock
Signed-off-by: Josh de Kock --- Makefile | 1 + ffplay.c | 5 + ffplay.c => ffplay_sdl1.c | 0 3 files changed, 6 insertions(+) copy ffplay.c => ffplay_sdl1.c (100%) diff --git a/Makefile b/Makefile index 8aa72fd..15ba3df 100644 --- a/Makefile +++ b/Makefil

[FFmpeg-devel] [PATCH 1/4] lavd: Add SDL2 output device

2016-09-14 Thread Josh de Kock
Acked-by: Michael Niedermayer Signed-off-by: Josh de Kock --- configure| 28 +++- libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/sdl2.c | 377 +++ 4 files changed, 406 insertions(+), 1 deletion(-)

[FFmpeg-devel] [PATCH 3/4] ffplay: add SDL2 support

2016-09-14 Thread Josh de Kock
From: Marton Balint Tested-by: James Almer (Windows, mingw-w64) Signed-off-by: Josh de Kock --- configure | 7 +- ffplay.c | 594 -- ffplay_sdl1.c | 1 - 3 files changed, 250 insertions(+), 352 deletions(-) diff --git a/conf

[FFmpeg-devel] [PATCH 0/4] ffplay and lavd SDL2 set

2016-09-14 Thread Josh de Kock
Hi, Resending this set with ffplay now having two versions, a SDL2 and a SDL1 version. I've integrated all comments up until now (hopefully). Josh Josh de Kock (3): lavd: Add SDL2 output device ffplay: make copy for SDL1 MAINTAINERS: update my entries Marton Balint (1): ffplay: add SDL2

Re: [FFmpeg-devel] [PATCH] Fix target_level for EAC3.

2016-09-14 Thread Niki Bowe
ping. On Fri, Sep 9, 2016 at 12:48 PM, Nikolas Bowe wrote: > Currently when using target_level with EAC3 it produces silence. This > small patch fixes target_level for decoding EAC3. > > Example: > ffmpeg -y -i /tmp/test.wav -acodec eac3 -dialnorm -14 -ac 6 -b:a 384000 > /tmp/test.m2ts > ffmpeg

[FFmpeg-devel] [PATCH 1/2] vf_colorspace: Interpret unspecified color range as limited range

2016-09-14 Thread Vittorio Giovara
This is the assumption that is made in pixel format conversion do throughout the code (in particular swscale and vf_colormatrix). Signed-off-by: Vittorio Giovara --- libavfilter/vf_colorspace.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_color

Re: [FFmpeg-devel] [PATCH 2/2] vf_colorspace: Add modern names for color range option

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 5:09 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Allows to use values returned from API and from ffprobe directly. > > Signed-off-by: Vittorio Giovara > --- > libavfilter/vf_colorspace.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libav

[FFmpeg-devel] [PATCH 2/2] vf_colorspace: Add modern names for color range option

2016-09-14 Thread Vittorio Giovara
Allows to use values returned from API and from ffprobe directly. Signed-off-by: Vittorio Giovara --- libavfilter/vf_colorspace.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c index 7e87cd8..b993928 100644 --- a/libavfilter/vf_col

Re: [FFmpeg-devel] [PATCH 4/4] lavf/mov: Add support for edit list parsing.

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 12:20:52AM -0700, Sasi Inguva wrote: > On Tue, Sep 13, 2016 at 4:39 PM, Sasi Inguva wrote: > > > Sorry for the very late reply. I was busy with other things. > > > > On Sat, Sep 3, 2016 at 4:48 PM, Michael Niedermayer < > > mich...@niedermayer.cc> wrote: > > > >> On Sat, S

Re: [FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 1:46 PM, Carl Eugen Hoyos wrote: > 2016-09-14 19:21 GMT+02:00 Ronald S. Bultje : > > [...] > > >> I wondered if somebody can easily fix muxing / demuxing which > >> could be considered a nicer solution. > > > > You mean remuxing (with -c:a copy) of swf files? > > Ther

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Josh de Kock
On 14/09/2016 20:07, James Almer wrote: [...] I'm fine with that if others are (Especially Marton and Josh, since they develop/maintain ffplay). It's a good compromise. But much like ffserver it should similarly be marked as deprecated and essentially unmaintained somewhere. I'm fine with this

Re: [FFmpeg-devel] [PATCH] avcodec/h264_parser: set missing pts for top/bottom field frames

2016-09-14 Thread Paul B Mahol
On 9/14/16, Michael Niedermayer wrote: > On Wed, Sep 14, 2016 at 04:42:59PM +0200, Paul B Mahol wrote: >> Adopted from 4eb49fdde8f84d54a763cfb5d355527b525ee2bf revert. >> >> Signed-off-by: Paul B Mahol >> --- >> libavcodec/h264_parser.c | 22 ++ >> 1 file changed, 22 insertio

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 04:07:50PM -0300, James Almer wrote: > On 9/14/2016 3:51 PM, Michael Niedermayer wrote: > > On Wed, Sep 14, 2016 at 06:00:00PM +0200, Carl Eugen Hoyos wrote: > >> 2016-09-14 17:54 GMT+02:00 Josh de Kock : > >>> We discussed this in IRC and thought that ffmpeg was a > >>> big

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread James Almer
On 9/14/2016 3:51 PM, Michael Niedermayer wrote: > On Wed, Sep 14, 2016 at 06:00:00PM +0200, Carl Eugen Hoyos wrote: >> 2016-09-14 17:54 GMT+02:00 Josh de Kock : >>> We discussed this in IRC and thought that ffmpeg was a >>> big enough library, and it switching to SDL2 may help >>> distros to consi

Re: [FFmpeg-devel] [PATCH v3 3/3] lavd: deprecate opengl outdev

2016-09-14 Thread Nicolas George
Le nonidi 29 fructidor, an CCXXIV, Josh de Kock a écrit : > What is your attachment to the SDL1 library? None at all. You did not read me carefully at all. My attachment is to the OpenGL device. > Firstly, the OpenGL device doesn't have the best implementation, and lavd I am rather surprised. II

Re: [FFmpeg-devel] [PATCH v3 3/3] lavd: deprecate opengl outdev

2016-09-14 Thread Josh de Kock
On 14/09/2016 18:53, Nicolas George wrote: Le sextidi 26 fructidor, an CCXXIV, Josh de Kock a écrit : This device depends on SDL which is deprecated. Signed-off-by: Josh de Kock Although your port of the code to SDL2 address my concern about deprecating without a replacement, I would be pain

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 06:00:00PM +0200, Carl Eugen Hoyos wrote: > 2016-09-14 17:54 GMT+02:00 Josh de Kock : > > We discussed this in IRC and thought that ffmpeg was a > > big enough library, and it switching to SDL2 may help > > distros to consider supporting SDL2 > > If this really were the onl

Re: [FFmpeg-devel] [PATCH] lavfi: VAAPI video processing filter

2016-09-14 Thread Mark Thompson
On 14/09/16 02:30, Jun Zhao wrote: > On 2016/9/14 6:06, Mark Thompson wrote: >> How about something like this, then? It adds a new filter to do the video >> processing, while leaving the scale filter as-is. > > Can we merge the vpp scale/the other vpp filters in one AVFilter, e,g > vf_postproce

Re: [FFmpeg-devel] mpeg12dec fix up DVD caption handling

2016-09-14 Thread Jonathan Campbell
> yes, please resbmit the remaining patch/changes with any comments > from everyone taken care of or with explanation why not > > thx Here you go, squashed into one patch. Sentences ending with a period have been capitalized and printf typecasts removed (Moritz). Jonathan Campbell From 8d640275

Re: [FFmpeg-devel] mpeg12dec fix up DVD caption handling

2016-09-14 Thread Jonathan Campbell
>> Would running git format-patch >patchfile work to submit it as one patch or >> do I need to squash it into one patch? >> >> Do I need to resubmit anything? > > yes, please resbmit the remaining patch/changes with any comments > from everyone taken care of or with explanation why not > > thx >

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:18 PM, Carl Eugen Hoyos wrote: > 2016-09-14 18:15 GMT+02:00 James Almer : > > > Configure failed to detect it > > configure requires 2.0.1 > > Carl Eugen (who wonders how Ronald tests...) ffmpeg (the tool), of course! :) Ronald __

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Marton Balint
On Wed, 14 Sep 2016, Carl Eugen Hoyos wrote: 2016-09-14 18:15 GMT+02:00 James Almer : Configure failed to detect it configure requires 2.0.1 YUV texture updates require 2.0.1 as far as I remember. Regards, Marton ___ ffmpeg-devel mailing list f

Re: [FFmpeg-devel] [PATCH v3 3/3] lavd: deprecate opengl outdev

2016-09-14 Thread Nicolas George
Le sextidi 26 fructidor, an CCXXIV, Josh de Kock a écrit : > This device depends on SDL which is deprecated. > > Signed-off-by: Josh de Kock Although your port of the code to SDL2 address my concern about deprecating without a replacement, I would be pained if this code were to be removed. I st

Re: [FFmpeg-devel] [PATCH]Fix NVENC HEVC Profiles

2016-09-14 Thread Yogender Kumar Gupta
Updated patch with requested changes. Thanks, Yogender On Wed, Sep 14, 2016 at 11:00 PM, Carl Eugen Hoyos wrote: > 2016-09-14 19:22 GMT+02:00 Yogender Kumar Gupta >: > > This patch fixes HEVC profiles for 444 and 444-10 that should be rext > > instead of main10. > > > +} > > +else if (

Re: [FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 19:21 GMT+02:00 Ronald S. Bultje : [...] >> I wondered if somebody can easily fix muxing / demuxing which >> could be considered a nicer solution. > > You mean remuxing (with -c:a copy) of swf files? There (also) is an encoder... Carl Eugen ___

Re: [FFmpeg-devel] [PATCH] avcodec/h264_parser: set missing pts for top/bottom field frames

2016-09-14 Thread Carl Eugen Hoyos
Hi! 2016-09-14 19:35 GMT+02:00 Paul B Mahol : > I see nothing wrong doing this in parser instead of in generic code. I am not saying there is anything wrong. I wanted to share my suspicion that this patch is a work-around for a bug that affects h.264 decoding in general. (And this suspicion may

Re: [FFmpeg-devel] [PATCH 3/6] lavc/qsv: Enable hwaccel qsv_vidmem.

2016-09-14 Thread Nablet Developer
ffmpeg_qsv.c | 636 +- libavcodec/qsv.h | 3 + libavcodec/qsv_internal.h | 2 + libavcodec/qsvdec.c | 5 +- libavcodec/qsvenc.c | 2 + 8 files changed, 649 insertions(+), 5 deletions(-) This is a giant patch

Re: [FFmpeg-devel] [PATCH] avcodec/h264_parser: set missing pts for top/bottom field frames

2016-09-14 Thread Paul B Mahol
On 9/14/16, Carl Eugen Hoyos wrote: > 2016-09-14 18:11 GMT+02:00 Paul B Mahol : >> On Wednesday, September 14, 2016, Carl Eugen Hoyos wrote: >> >>> 2016-09-14 16:42 GMT+02:00 Paul B Mahol : >>> > Adopted from 4eb49fdde8f84d54a763cfb5d355527b525ee2bf revert. >>> >>> I am of course not against this

Re: [FFmpeg-devel] [PATCH]Fix NVENC HEVC Profiles

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 19:22 GMT+02:00 Yogender Kumar Gupta : > This patch fixes HEVC profiles for 444 and 444-10 that should be rext > instead of main10. > +} > +else if (ctx->data_pix_fmt == AV_PIX_FMT_P010) { Please merge these lines. Carl Eugen ___ ffm

[FFmpeg-devel] [PATCH]Fix NVENC HEVC Profiles

2016-09-14 Thread Yogender Kumar Gupta
This patch fixes HEVC profiles for 444 and 444-10 that should be rext instead of main10. Thanks, Yogender 0001-Fix-NVENC-HEVC-Profiles.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listin

Re: [FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:29 PM, Carl Eugen Hoyos wrote: > Hi! > > 2016-09-14 18:05 GMT+02:00 Ronald S. Bultje : > > > > On Wed, Sep 14, 2016 at 12:01 PM, Carl Eugen Hoyos wrote: > > > >> 2016-09-12 12:52 GMT+02:00 Carl Eugen Hoyos : > >> > >> > Attached patch "fixes" ticket #5829, I am hap

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 18:56 GMT+02:00 Yogender Kumar Gupta : > Please let me know if anything else needs to be taken care of. if (condition) { do(); } else { do2(); } like everywhere else in FFmpeg (and on top of your patch). +if (ctx->aq && ctx->temporalAQ) +{ +av_log(avctx, AV_LOG_E

Re: [FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools

2016-09-14 Thread Yogender Kumar Gupta
I agree the change was not good, but since it was not compiling on my system using VS2013 I submitted this, but I hear that rather it may be an issue with my system as the code compiles well using VS2013 tools. I will retry at my end. On Wed, Sep 14, 2016 at 10:08 PM, Timo Rothenpieler wrote: >

[FFmpeg-devel] [PATCH] Patch for Improved Log for NVENC

2016-09-14 Thread Yogender Kumar Gupta
The existing logs have been improved to give a better idea of how what steps have been performed. Thanks, Yogender 0001-Improve-Log-for-NVENC.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Yogender Kumar Gupta
I have made only changes specific to SDK_7_0 in the attached patch. Also generated the patch using format-patch. Please let me know if anything else needs to be taken care of. Thanks, Yogender On Wed, Sep 14, 2016 at 8:48 PM, Carl Eugen Hoyos wrote: > 2016-09-14 15:43 GMT+02:00 Yogender Kumar

Re: [FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools

2016-09-14 Thread Timo Rothenpieler
On 9/14/2016 3:43 PM, Yogender Kumar Gupta wrote: > Attached is a patch for SDK 7_0 for NVENC. This adds other features > available in SDK 7_0 as well as fixes an issue with HEVC profile > I'd very much dislike applying this change. It makes the list very hard to read. While it could be re-arrange

Re: [FFmpeg-devel] [PATCH] added expr evaluation to drawtext - fontsize

2016-09-14 Thread Brett Harrison
Any feedback on the most recent patch? On Fri, Sep 9, 2016 at 5:26 PM, Brett Harrison wrote: > Here are the changes requested > > On Thu, Sep 8, 2016 at 8:50 AM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > >> On Tue, Sep 06, 2016 at 10:27:24AM -0700, Brett Harrison wrote: >> > This

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Timo Rothenpieler
On 9/14/2016 6:30 PM, Carl Eugen Hoyos wrote: > 2016-09-14 18:26 GMT+02:00 Timo Rothenpieler : >> On 9/14/2016 3:43 PM, Yogender Kumar Gupta wrote: >>> Attached is a patch for SDK 7_0 for NVENC. This adds other features >>> available in SDK 7_0 as well as fixes an issue with HEVC profile >>> >> >>

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread James Almer
On 9/14/2016 1:12 PM, Clément Bœsch wrote: > On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: >> 2016-09-12 20:58 GMT+02:00 Josh de Kock : >>> From: Marton Balint >>> >>> Depends on the other SDL patches. >> >> This works fine on my antique desktop system (where an >> extra reposi

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 18:26 GMT+02:00 Timo Rothenpieler : > On 9/14/2016 3:43 PM, Yogender Kumar Gupta wrote: >> Attached is a patch for SDK 7_0 for NVENC. This adds other features >> available in SDK 7_0 as well as fixes an issue with HEVC profile >> > > I'd very much dislike applying this change. I suspect

Re: [FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

2016-09-14 Thread Carl Eugen Hoyos
Hi! 2016-09-14 18:05 GMT+02:00 Ronald S. Bultje : > > On Wed, Sep 14, 2016 at 12:01 PM, Carl Eugen Hoyos wrote: > >> 2016-09-12 12:52 GMT+02:00 Carl Eugen Hoyos : >> >> > Attached patch "fixes" ticket #5829, I am happy if a better >> > solution can be found. > What makes you believe this is not t

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Timo Rothenpieler
On 9/14/2016 3:43 PM, Yogender Kumar Gupta wrote: > Attached is a patch for SDK 7_0 for NVENC. This adds other features > available in SDK 7_0 as well as fixes an issue with HEVC profile > I'd very much dislike applying this change. It makes the list very hard to read. While it could be re-arrang

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 18:23 GMT+02:00 Timo Rothenpieler : > I'm not sure what target_quality is supposed to do Could it be that it moves the time-to-encode / quality-of-encoding slider? (just guessing, libvpx does this iirc) Carl Eugen ___ ffmpeg-devel mailing li

Re: [FFmpeg-devel] [PATCH v6 1/3] lavd: Add SDL2 output device

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 18:20 GMT+02:00 Josh de Kock : > On 14/09/2016 17:17, Carl Eugen Hoyos wrote: >> >> 2016-09-11 23:14 GMT+02:00 Josh de Kock : >> >>> +{ AV_PIX_FMT_RGB24, SDL_PIXELFORMAT_RGB888 }, >> >>> +{ AV_PIX_FMT_BGR24, SDL_PIXELFORMAT_BGR888 }, >> >> Did you test them / why did you remove

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Hendrik Leppkes
Am 14.09.2016 18:00 schrieb "Carl Eugen Hoyos" : > > 2016-09-14 17:54 GMT+02:00 Josh de Kock : > > We discussed this in IRC and thought that ffmpeg was a > > big enough library, and it switching to SDL2 may help > > distros to consider supporting SDL2 > > If this really were the only reason, I sinc

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Timo Rothenpieler
On 9/14/2016 3:43 PM, Yogender Kumar Gupta wrote: > Attached is a patch for SDK 7_0 for NVENC. This adds other features > available in SDK 7_0 as well as fixes an issue with HEVC profile > What carl said. Also, a some of the added options are not used anywhere: zeroReorderDelay, enableNonRefP I

Re: [FFmpeg-devel] [PATCH] avcodec/h264_parser: set missing pts for top/bottom field frames

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 18:11 GMT+02:00 Paul B Mahol : > On Wednesday, September 14, 2016, Carl Eugen Hoyos wrote: > >> 2016-09-14 16:42 GMT+02:00 Paul B Mahol : >> > Adopted from 4eb49fdde8f84d54a763cfb5d355527b525ee2bf revert. >> >> I am of course not against this patch but wasn't the reason for the >> old co

Re: [FFmpeg-devel] [PATCH v6 1/3] lavd: Add SDL2 output device

2016-09-14 Thread Josh de Kock
On 14/09/2016 17:17, Carl Eugen Hoyos wrote: 2016-09-11 23:14 GMT+02:00 Josh de Kock : +{ AV_PIX_FMT_RGB24, SDL_PIXELFORMAT_RGB888 }, +{ AV_PIX_FMT_BGR24, SDL_PIXELFORMAT_BGR888 }, Did you test them / why did you remove the comments that said they do not work? I assumed since no

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 18:15 GMT+02:00 James Almer : > Configure failed to detect it configure requires 2.0.1 Carl Eugen (who wonders how Ronald tests...) ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v6 1/3] lavd: Add SDL2 output device

2016-09-14 Thread Carl Eugen Hoyos
2016-09-11 23:14 GMT+02:00 Josh de Kock : > +{ AV_PIX_FMT_RGB24, SDL_PIXELFORMAT_RGB888 }, > +{ AV_PIX_FMT_BGR24, SDL_PIXELFORMAT_BGR888 }, Did you test them / why did you remove the comments that said they do not work? > +{ AV_PIX_FMT_RGB0, SDL_PIXELFORMAT_RGBX }, > +{

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread James Almer
On 9/14/2016 1:00 PM, Carl Eugen Hoyos wrote: > 2016-09-14 17:54 GMT+02:00 Josh de Kock : >> We discussed this in IRC and thought that ffmpeg was a >> big enough library, and it switching to SDL2 may help >> distros to consider supporting SDL2 > > If this really were the only reason, I sincerely h

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Clément Bœsch
On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: > 2016-09-12 20:58 GMT+02:00 Josh de Kock : > > From: Marton Balint > > > > Depends on the other SDL patches. > > This works fine on my antique desktop system (where an > extra repository provides sdl2 2.0.3) but not on my laptop >

Re: [FFmpeg-devel] [PATCH] avcodec/h264_parser: set missing pts for top/bottom field frames

2016-09-14 Thread Paul B Mahol
On Wednesday, September 14, 2016, Carl Eugen Hoyos wrote: > 2016-09-14 16:42 GMT+02:00 Paul B Mahol >: > > Adopted from 4eb49fdde8f84d54a763cfb5d355527b525ee2bf revert. > > I am of course not against this patch but wasn't the reason for the > old code in utils.c that our h264 parser has insuffici

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:04 PM, Carl Eugen Hoyos wrote: > 2016-09-14 17:59 GMT+02:00 Michael Niedermayer : > > On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: > >> 2016-09-12 20:58 GMT+02:00 Josh de Kock : > >> > From: Marton Balint > >> > > >> > Depends on the other SDL

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread James Almer
On 9/14/2016 12:34 PM, Carl Eugen Hoyos wrote: > 2016-09-12 20:58 GMT+02:00 Josh de Kock : >> From: Marton Balint >> >> Depends on the other SDL patches. > > This works fine on my antique desktop system (where an > extra repository provides sdl2 2.0.3) but not on my laptop > that I did not consid

Re: [FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:01 PM, Carl Eugen Hoyos wrote: > 2016-09-12 12:52 GMT+02:00 Carl Eugen Hoyos : > > > Attached patch "fixes" ticket #5829, I am happy if a better > > solution can be found. > > Ping. What makes you believe this is not the right solution? As in: why was the rogue t

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 17:59 GMT+02:00 Michael Niedermayer : > On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: >> 2016-09-12 20:58 GMT+02:00 Josh de Kock : >> > From: Marton Balint >> > >> > Depends on the other SDL patches. >> >> This works fine on my antique desktop system (where an >> ext

Re: [FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

2016-09-14 Thread Carl Eugen Hoyos
2016-09-12 12:52 GMT+02:00 Carl Eugen Hoyos : > Attached patch "fixes" ticket #5829, I am happy if a better > solution can be found. Ping. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: > 2016-09-12 20:58 GMT+02:00 Josh de Kock : > > From: Marton Balint > > > > Depends on the other SDL patches. > > This works fine on my antique desktop system (where an > extra repository provides sdl2 2.0.3) but not on my laptop >

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 17:54 GMT+02:00 Josh de Kock : > We discussed this in IRC and thought that ffmpeg was a > big enough library, and it switching to SDL2 may help > distros to consider supporting SDL2 If this really were the only reason, I sincerely hope we wouldn't even consider this patch. My question

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Josh de Kock
On 14/09/2016 16:34, Carl Eugen Hoyos wrote: [...] If a supported Linux distribution does not ship a sufficiently recent sdl2, I am not sure if this is such a good idea. Does the patch offer new functionality? It doesn't add any extra functionality, but SDL1 has been unmaintained for more than 4

Re: [FFmpeg-devel] [PATCH]lavc: Enable a53cc by default

2016-09-14 Thread Carl Eugen Hoyos
2016-09-10 19:04 GMT+02:00 Michael Niedermayer : > On Fri, Sep 09, 2016 at 05:59:19PM +0200, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch enables saving closed caption by default if available. >> >> Please comment, Carl Eugen > >> libx264.c |2 +- >> qsvenc_h264.c |2 +- >> 2 fi

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-14 Thread Carl Eugen Hoyos
Hi! 2016-09-08 19:34 GMT+02:00 Nicolas George : > Still, it is probably possible to do something faster than xcbgrab while > still correct: only allocate a new segment if the first one is still in use. Will you try this? Thank you, Carl Eugen ___ ffmp

Re: [FFmpeg-devel] [PATCH] libavformat/webm_chunk: Option to specify HTTP header

2016-09-14 Thread Carl Eugen Hoyos
2016-09-13 22:12 GMT+02:00 Vignesh Venkatasubramanian : > Add an option to specify HTTP header in the WebM Chunk Muxer > (this works the same way as the 'method' parameter in the HLS muxer). If it is the same option, why does it have another name? (If the new name is better, maybe update the hls

Re: [FFmpeg-devel] calculation of probe score

2016-09-14 Thread Carl Eugen Hoyos
2016-09-13 21:14 GMT+02:00 Dave Rice : > 1 aac,1 This is either random data with aac extension or random data that gets misdetected: I locally use a patch that avoids returning 1 from the aac probe function. > 1 flac,1 This is random data with flac extension but please test specifically. >

Re: [FFmpeg-devel] [PATCH] avcodec/h264_parser: set missing pts for top/bottom field frames

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 04:42:59PM +0200, Paul B Mahol wrote: > Adopted from 4eb49fdde8f84d54a763cfb5d355527b525ee2bf revert. > > Signed-off-by: Paul B Mahol > --- > libavcodec/h264_parser.c | 22 ++ > 1 file changed, 22 insertions(+) > > diff --git a/libavcodec/h264_parser.

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Carl Eugen Hoyos
2016-09-12 20:58 GMT+02:00 Josh de Kock : > From: Marton Balint > > Depends on the other SDL patches. This works fine on my antique desktop system (where an extra repository provides sdl2 2.0.3) but not on my laptop that I did not consider eol so far, it only has 2.0.0. If a supported Linux dist

Re: [FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 15:43 GMT+02:00 Yogender Kumar Gupta : > Attached is a patch for SDK 7_0 for NVENC. This adds other features > available in SDK 7_0 as well as fixes an issue with HEVC profile The first two hunks do not match your description, please send a separate patch for the log changes. The positi

Re: [FFmpeg-devel] [PATCH] avcodec/h264_parser: set missing pts for top/bottom field frames

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 16:42 GMT+02:00 Paul B Mahol : > Adopted from 4eb49fdde8f84d54a763cfb5d355527b525ee2bf revert. I am of course not against this patch but wasn't the reason for the old code in utils.c that our h264 parser has insufficiencies? Carl Eugen ___ ff

Re: [FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools

2016-09-14 Thread Hendrik Leppkes
On Wed, Sep 14, 2016 at 12:45 PM, Matt Oliver wrote: > On 14 September 2016 at 20:22, Yogender Kumar Gupta < > yogender.gu...@gmail.com> wrote: > >> Currently libavcodec.c/nvenc.c is not compilable using MSVC tools. I am >> attaching a patch that fixes this issue. >> >> Thanks, >> Yogender >> > >

[FFmpeg-devel] [PATCH] avcodec/h264_parser: set missing pts for top/bottom field frames

2016-09-14 Thread Paul B Mahol
Adopted from 4eb49fdde8f84d54a763cfb5d355527b525ee2bf revert. Signed-off-by: Paul B Mahol --- libavcodec/h264_parser.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/libavcodec/h264_parser.c b/libavcodec/h264_parser.c index 615884f..cf6c3d1 100644 --- a/libavcodec/h2

Re: [FFmpeg-devel] [PATCH] avformat/utils: fix timebase error in avformat_seek_file()

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 04:13:45PM +0800, Xinzheng Zhang wrote: > When there is only one stream and stream_index has not specified, > The ts has been transferd by the timebase of stream0 without modifying the > stream_index > In this condation it cause seek failure. > > --- > libavformat/utils.c

[FFmpeg-devel] [PATCH] Patch for SDK 7.0 for NVENC

2016-09-14 Thread Yogender Kumar Gupta
Attached is a patch for SDK 7_0 for NVENC. This adds other features available in SDK 7_0 as well as fixes an issue with HEVC profile sdk_7_0.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/l

Re: [FFmpeg-devel] [PATCH 2/2] avformat/flvenc: add no_sequence_end flags for flvflags

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 10:04:27AM +0800, Steven Liu wrote: > 2016-09-14 10:00 GMT+08:00 Michael Niedermayer : > > > On Wed, Sep 14, 2016 at 09:42:48AM +0800, Steven Liu wrote: > > > 2016-09-14 8:31 GMT+08:00 Michael Niedermayer : > > > > > > > On Tue, Sep 13, 2016 at 10:10:40PM +0800, Steven Liu

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Josh de Kock
On Wed, Sep 14, 2016, at 10:12 AM, Michael Niedermayer wrote: > On Tue, Sep 13, 2016 at 01:45:12PM -0300, James Almer wrote: > > > > Tested on Windows (compiled with mingw-w64) and works fine. > > patchwork uses "^Tested-by:" to detect patches having been tested > should the regexp be changed/ext

Re: [FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools

2016-09-14 Thread Yogender Kumar Gupta
Microsoft (R) C/C++ Optimizing Compiler Version 18.00.30501 for x64 Copyright (C) Microsoft Corporation. All rights reserved. is what is reported when I run cl in my mingW environment. and the shell is launched from the VS2013 x64 Native Tools Command Prompt On Wed, Sep 14, 2016 at 4:15 PM, Matt

Re: [FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools

2016-09-14 Thread Matt Oliver
On 14 September 2016 at 20:22, Yogender Kumar Gupta < yogender.gu...@gmail.com> wrote: > Currently libavcodec.c/nvenc.c is not compilable using MSVC tools. I am > attaching a patch that fixes this issue. > > Thanks, > Yogender > What version of msvc are you using? 2015 and 2013 all work fine curr

[FFmpeg-devel] Patch for fixing of nvenc.c compilation using msvc tools

2016-09-14 Thread Yogender Kumar Gupta
Currently libavcodec.c/nvenc.c is not compilable using MSVC tools. I am attaching a patch that fixes this issue. Thanks, Yogender msvc_compile_fix.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/ma

[FFmpeg-devel] [PATCH] avfilter/drawutils: allow drawing opaque text on transparent background

2016-09-14 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/drawutils.c | 7 --- libavfilter/drawutils.h | 8 +++- libavfilter/vf_drawtext.c | 2 +- 3 files changed, 12 insertions(+), 5 deletions(-) diff --git a/libavfilter/drawutils.c b/libavfilter/drawutils.c index f6760be..f19fc5d 100644 --- a/li

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Michael Niedermayer
On Tue, Sep 13, 2016 at 01:45:12PM -0300, James Almer wrote: > On 9/12/2016 3:58 PM, Josh de Kock wrote: > > From: Marton Balint > > > > Depends on the other SDL patches. > > > > Signed-off-by: Josh de Kock > > --- > > configure | 6 +- > > ffplay.c | 594 > > ++

[FFmpeg-devel] [PATCH] avformat/utils: fix timebase error in avformat_seek_file()

2016-09-14 Thread Xinzheng Zhang
When there is only one stream and stream_index has not specified, The ts has been transferd by the timebase of stream0 without modifying the stream_index In this condation it cause seek failure. --- libavformat/utils.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/utils.c b/lib

Re: [FFmpeg-devel] [PATCH 1/2] lavf/utils: simplify matching MOV-like formats

2016-09-14 Thread Clément Bœsch
On Tue, Sep 13, 2016 at 01:06:17PM +0200, Michael Niedermayer wrote: > On Tue, Sep 13, 2016 at 11:00:27AM +0200, Clément Bœsch wrote: > > On Tue, Sep 13, 2016 at 10:51:25AM +0200, Clément Bœsch wrote: > > > From: Clément Bœsch > > > > > > --- > > > libavformat/utils.c | 4 +--- > > > 1 file chan

Re: [FFmpeg-devel] [PATCH] lavf: add avformat_transfer_internal_stream_timing_info() and use it in ffmpeg

2016-09-14 Thread Clément Bœsch
On Tue, Sep 13, 2016 at 02:56:16PM -0300, James Almer wrote: > On 9/13/2016 2:42 PM, Clément Bœsch wrote: > > On Tue, Sep 13, 2016 at 02:36:46PM -0300, James Almer wrote: > > [...] > >>> diff --git a/libavformat/avformat.h b/libavformat/avformat.h > >>> index 74915a1..389d985 100644 > >>> --- a/lib

Re: [FFmpeg-devel] [PATCH 2/2] lavf/utils: sync MOV-like formats with the muxers found in lavf/movenc

2016-09-14 Thread Clément Bœsch
On Tue, Sep 13, 2016 at 06:43:17PM +0200, Michael Niedermayer wrote: > On Tue, Sep 13, 2016 at 10:51:26AM +0200, Clément Bœsch wrote: > > From: Clément Bœsch > > > > --- > > libavformat/utils.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavformat/utils.c b/

Re: [FFmpeg-devel] [PATCH 4/4] lavf/mov: Add support for edit list parsing.

2016-09-14 Thread Sasi Inguva
On Tue, Sep 13, 2016 at 4:39 PM, Sasi Inguva wrote: > Sorry for the very late reply. I was busy with other things. > > On Sat, Sep 3, 2016 at 4:48 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > >> On Sat, Sep 03, 2016 at 12:06:39PM -0700, Sasi Inguva wrote: >> > Hi Michael, >> > >>