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]

[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 ---

[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

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

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,

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

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

[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 ---

[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

[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,

[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

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 >

[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

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

[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

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:

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

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

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

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

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

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.

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

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

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 >

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

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! :)

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 ___

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

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 > >

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

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

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

[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

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

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) +{ +

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

[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

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

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: [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,

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

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

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

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.

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: [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 ___

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 }, >> >>

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

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

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

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?

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

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,

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

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

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 >

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 : >

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

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

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. >> >>

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

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

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

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

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 +- >>

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

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

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

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

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

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

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

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. >> >>

[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

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. > > --- >

[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

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

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

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,

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

[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

[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

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

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

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 > >>> ---

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

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: >> >