Re: [FFmpeg-devel] [PATCH] swscale: Make sws_alloc_set_opts() public

2015-08-09 Thread Ronald S. Bultje
Hi, On Sat, Aug 8, 2015 at 12:58 PM, Paul B Mahol one...@gmail.com wrote: On 8/8/15, wm4 nfx...@googlemail.com wrote: On Sat, 8 Aug 2015 18:17:37 +0200 Michael Niedermayer mich...@niedermayer.cc wrote: On Sat, Aug 08, 2015 at 05:58:29PM +0200, wm4 wrote: On Sat, 8 Aug 2015 17:14:58

Re: [FFmpeg-devel] [PATCH 4/6] avcodec: delay removal of avcodec_*_frame

2015-08-09 Thread Ronald S. Bultje
Hi, On Sun, Aug 9, 2015 at 6:41 AM, wm4 nfx...@googlemail.com wrote: On Sun, 9 Aug 2015 04:23:01 +0200 Michael Niedermayer mich...@niedermayer.cc wrote: On Sat, Aug 08, 2015 at 01:37:08PM +0200, Andreas Cadhalpun wrote: They are still widely used and have been deprecated for less than

Re: [FFmpeg-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread wm4
On Sun, 9 Aug 2015 04:14:39 +0200 Michael Niedermayer mich...@niedermayer.cc wrote: On Sat, Aug 08, 2015 at 01:36:58PM +0200, Andreas Cadhalpun wrote: Only the PixelFormat removal is not delayed, because it is a very generic name. Signed-off-by: Andreas Cadhalpun

[FFmpeg-devel] [PATCH] videotoolbox: allow to enable the async decoding.

2015-08-09 Thread Sebastien Zwickert
This patch allows to use the Videotoolbox API in asynchonous mode. Note that when using async decoding the user is responsible for releasing the async frame. Moreover, an option called videotoolbox_async was added to enable async decoding with ffmpeg CLI. --- ffmpeg.h | 1 +

Re: [FFmpeg-devel] [PATCH] avfilter/vsrc_testsrc: correct colors for smptebars

2015-08-09 Thread Kieran Kunhya
On 9 August 2015 at 12:20, Paul B Mahol one...@gmail.com wrote: --- libavfilter/vsrc_testsrc.c | 36 +++- 1 file changed, 23 insertions(+), 13 deletions(-) I'll test the SD version on the scope on Monday (cc'ing dericed who can probably test HD) Kieran.

Re: [FFmpeg-devel] [PATCH] avcodec/vc1dec: Re-order init to avoid initting hwaccel too early

2015-08-09 Thread Michael Niedermayer
On Sat, Aug 08, 2015 at 07:57:29PM -0700, Philip Langdale wrote: At least for vdpau, the hwaccel init code tries to check the video profile and ensure that there is a matching vdpau profile available. If it can't find a match, it will fail to initialise. In the case of wmv3/vc1, I observed

Re: [FFmpeg-devel] [PATCH] videotoolbox: allow to enable the async decoding.

2015-08-09 Thread wm4
On Sun, 9 Aug 2015 07:46:53 -0400 Ronald S. Bultje rsbul...@gmail.com wrote: Hi, On Sun, Aug 9, 2015 at 7:11 AM, Sebastien Zwickert dilar...@gmail.com wrote: This patch allows to use the Videotoolbox API in asynchonous mode. If this is for N:M (in:out), I think this should be

Re: [FFmpeg-devel] [PATCH] avcodec/vc1dec: Re-order init to avoid initting hwaccel too early

2015-08-09 Thread Andy Furniss
Philip Langdale wrote: At least for vdpau, the hwaccel init code tries to check the video profile and ensure that there is a matching vdpau profile available. If it can't find a match, it will fail to initialise. I recently noticed this - that it silently falls back to s/w in my test case.

Re: [FFmpeg-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread Ronald S. Bultje
Hi, On Sun, Aug 9, 2015 at 6:41 AM, wm4 nfx...@googlemail.com wrote: On Sun, 9 Aug 2015 04:14:39 +0200 Michael Niedermayer mich...@niedermayer.cc wrote: On Sat, Aug 08, 2015 at 01:36:58PM +0200, Andreas Cadhalpun wrote: Only the PixelFormat removal is not delayed, because it is a very

Re: [FFmpeg-devel] [PATCH] videotoolbox: allow to enable the async decoding.

2015-08-09 Thread Hendrik Leppkes
On Sun, Aug 9, 2015 at 1:11 PM, Sebastien Zwickert dilar...@gmail.com wrote: This patch allows to use the Videotoolbox API in asynchonous mode. Note that when using async decoding the user is responsible for releasing the async frame. Moreover, an option called videotoolbox_async was added to

[FFmpeg-devel] [PATCH] movtextdec: Use default style information from movtext header

2015-08-09 Thread Niklesh Lalwani
From: Niklesh niklesh.lalw...@iitb.ac.in Sets the default style in ASS from the default style information present in movtext header Signed-off-by: Niklesh niklesh.lalw...@iitb.ac.in --- libavcodec/movtextdec.c | 82 ++--- 1 file changed, 78

[FFmpeg-devel] PATCH 2/3] libavcodec/qsvdec.c: Extended error messages for MFXVideoDECODE_Init() result

2015-08-09 Thread Ivan Uskov
Hello all, the attached patch does extend error codes processing to give exact message when input format is not supported by QSV (for example mpeg 422 or avc 10 bit). -- Best regards, Ivanmailto:ivan.us...@nablet.com

[FFmpeg-devel] [PATCH 1/3] libavcodec/qsvdec.c: the ff_get_format() missed at refactoring has been restored

2015-08-09 Thread Ivan Uskov
Hello All, This patch, next two patches and also the patch posted by me a August 4 are fixing all issues about QSV-accelerated decoding. I will absent two next weeks since August 11 and will not accessible by e-mail. But even if these patches will not applied to master repository are can be used

Re: [FFmpeg-devel] [libav-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread Ronald S. Bultje
Hi, On Sun, Aug 9, 2015 at 11:54 AM, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Hi, On 09.08.2015 12:57, Ronald S. Bultje wrote: Yeah, I'm with this. Andreas, the correct fix is to update applications, even if that means vendor-specific patches in Debian. These are

Re: [FFmpeg-devel] [libav-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread Michael Niedermayer
On Sun, Aug 09, 2015 at 06:39:05PM +0200, Andreas Cadhalpun wrote: On 09.08.2015 18:19, Hendrik Leppkes wrote: This mail seems missing on FFmpeg-devel i know its easy to forget to check CC/To but please make sure discussions dont loose the CC/To: ffmpeg-devel otherwise discussions would be

[FFmpeg-devel] [PATCH 3/3] libavcodec/qsvdec.c: correct flush() handler has been implemented

2015-08-09 Thread Ivan Uskov
Hello All, The attached patch does add correct processing for the flush(), including QSV decoder resetting an internal buffers discarding. -- Best regards, Ivanmailto:ivan.us...@nablet.com 0003-libavcodec-qsvdec.c-correct-flush-handler-has-been-i.patch

Re: [FFmpeg-devel] [libav-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread Andreas Cadhalpun
On 09.08.2015 18:19, Hendrik Leppkes wrote: On Sun, Aug 9, 2015 at 5:54 PM, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Hi, On 09.08.2015 12:57, Ronald S. Bultje wrote: Yeah, I'm with this. Andreas, the correct fix is to update applications, even if that means vendor-specific

Re: [FFmpeg-devel] [libav-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread wm4
On Sun, 9 Aug 2015 17:54:55 +0200 Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Hi, On 09.08.2015 12:57, Ronald S. Bultje wrote: Yeah, I'm with this. Andreas, the correct fix is to update applications, even if that means vendor-specific patches in Debian. These are

Re: [FFmpeg-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread Frank Lömker
Hello, On 08/09/2015 06:57 AM, Ronald S. Bultje wrote: Does not LGTM. Yeah, I'm with this. Andreas, the correct fix is to update applications, even if that means vendor-specific patches in Debian. These are exceptionally trivial patches that you can generate using fairly trivial sed

Re: [FFmpeg-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread Hendrik Leppkes
On Sun, Aug 9, 2015 at 8:28 PM, Frank Lömker floem...@gmx.de wrote: Hello, And that's even the easy part. A lot of people don't update there distribution every six months, often it's years old. If you try to support all these distributions you end up with lot's and lot's of #ifdefery. With

Re: [FFmpeg-devel] [PATCH] swscale: Make sws_alloc_set_opts() public

2015-08-09 Thread wm4
On Sun, 9 Aug 2015 13:12:34 +0200 Michael Niedermayer mich...@niedermayer.cc wrote: On Sat, Aug 08, 2015 at 07:45:23PM +0200, wm4 wrote: On Sat, 8 Aug 2015 19:05:39 +0200 Michael Niedermayer mich...@niedermayer.cc wrote: Where are the input and output colorspaces and ranges set?

Re: [FFmpeg-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread Clément Bœsch
On Sun, Aug 09, 2015 at 08:28:28PM +0200, Frank Lömker wrote: Hello, On 08/09/2015 06:57 AM, Ronald S. Bultje wrote: Does not LGTM. Yeah, I'm with this. Andreas, the correct fix is to update applications, even if that means vendor-specific patches in Debian. These are exceptionally

Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/qsvdec.c: the ff_get_format() missed at refactoring has been restored

2015-08-09 Thread Ivan Uskov
Hello Ivan, Sunday, August 9, 2015, 6:32:55 PM, you wrote: IU This patch, next two patches and also the patch posted by me a August 4 IU are fixing all issues about QSV-accelerated decoding. all *known* issues of course. :-) -- Best regards, Ivan

Re: [FFmpeg-devel] [PATCH] avcodec/vc1dec: Re-order init to avoid initting hwaccel too early

2015-08-09 Thread Philip Langdale
On Sun, 09 Aug 2015 10:36:25 +0100 Andy Furniss adf.li...@gmail.com wrote: Philip Langdale wrote: At least for vdpau, the hwaccel init code tries to check the video profile and ensure that there is a matching vdpau profile available. If it can't find a match, it will fail to initialise.

[FFmpeg-devel] [PATCH] mxfdec: set AVFMT_SEEK_TO_PTS demuxer flag

2015-08-09 Thread Marton Balint
Since 53f2ef2c4afb1d49a679dea9163cb0e4671f3117 seeking is done using PTS. Signed-off-by: Marton Balint c...@passwd.hu --- libavformat/mxfdec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 2d921db..5734976 100644 --- a/libavformat/mxfdec.c

Re: [FFmpeg-devel] [PATCH] avcodec: loongson optimize h264qpel with mmi v1

2015-08-09 Thread Michael Niedermayer
On Thu, Aug 06, 2015 at 09:27:52PM +0800, 周晓勇 wrote: From ef684fade1282029aed3dece261f79a9ec04369b Mon Sep 17 00:00:00 2001 From: ZhouXiaoyong zhouxiaoy...@loongson.cn Date: Thu, 6 Aug 2015 21:13:35 +0800 Subject: [PATCH] avcodec: loongson optimize h264qpel with mmi v1 Signed-off-by:

[FFmpeg-devel] [PATCH] avcodec/vc1dec: Re-order init to avoid initting hwaccel too early

2015-08-09 Thread Philip Langdale
At least for vdpau, the hwaccel init code tries to check the video profile and ensure that there is a matching vdpau profile available. If it can't find a match, it will fail to initialise. In the case of wmv3/vc1, I observed initialisation to fail all the time. It turns out that this is due to

Re: [FFmpeg-devel] [PATCH] avcodec/vc1dec: Re-order init to avoid initting hwaccel too early

2015-08-09 Thread Philip Langdale
On Sun, 9 Aug 2015 12:42:22 +0200 Michael Niedermayer mich...@niedermayer.cc wrote: On Sat, Aug 08, 2015 at 07:57:29PM -0700, Philip Langdale wrote: At least for vdpau, the hwaccel init code tries to check the video profile and ensure that there is a matching vdpau profile available. If

Re: [FFmpeg-devel] [PATCH] avcodec/vc1dec: Re-order init to avoid initting hwaccel too early

2015-08-09 Thread Michael Niedermayer
On Sun, Aug 09, 2015 at 10:22:54AM -0700, Philip Langdale wrote: At least for vdpau, the hwaccel init code tries to check the video profile and ensure that there is a matching vdpau profile available. If it can't find a match, it will fail to initialise. In the case of wmv3/vc1, I observed

Re: [FFmpeg-devel] [PATCH] mxfdec: set AVFMT_SEEK_TO_PTS demuxer flag

2015-08-09 Thread Michael Niedermayer
On Sun, Aug 09, 2015 at 08:32:02PM +0200, Marton Balint wrote: Since 53f2ef2c4afb1d49a679dea9163cb0e4671f3117 seeking is done using PTS. Signed-off-by: Marton Balint c...@passwd.hu --- libavformat/mxfdec.c | 1 + 1 file changed, 1 insertion(+) probably ok, iam not mxfdec maintainer though

[FFmpeg-devel] [PATCH] postproc: Deprecate QP_STORE_T, it lacks a PP/AV/FF prefix

2015-08-09 Thread Michael Niedermayer
From: Michael Niedermayer mich...@niedermayer.cc Signed-off-by: Michael Niedermayer mich...@niedermayer.cc --- libpostproc/postprocess.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libpostproc/postprocess.h b/libpostproc/postprocess.h index e00ed96..fd5eb5d 100644

Re: [FFmpeg-devel] [PATCH 5/6] aaccoder: Move the quantization function to a separate file

2015-08-09 Thread Michael Niedermayer
On Wed, Jul 29, 2015 at 05:44:28AM +0100, Rostislav Pehlivanov wrote: This commit moves the quantization function to a separate file. This is done because the current extensions under development have separate files, and due to the fact that there was no way to get the distortion measurements

Re: [FFmpeg-devel] [PATCH] movtextdec: Use default style information from movtext header

2015-08-09 Thread Philip Langdale
On Mon, 10 Aug 2015 01:18:58 +0530 Niklesh Lalwani niklesh.lalw...@iitb.ac.in wrote: + +typedef struct { +char *font; +int fontsize; +int color; +int back_color; +int bold; +int italic; +int underline; +int alignment; +}

[FFmpeg-devel] [PATCH] avformat: Move MAX_PROBE_PACKETS to internal.h

2015-08-09 Thread Michael Niedermayer
From: Michael Niedermayer mich...@niedermayer.cc Signed-off-by: Michael Niedermayer mich...@niedermayer.cc --- libavformat/avformat.h |1 - libavformat/internal.h |2 ++ 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index

Re: [FFmpeg-devel] [PATCH 2/2] ffplay: increase MIN_FRAMES to 25

2015-08-09 Thread Marton Balint
On Sun, 9 Aug 2015, Michael Niedermayer wrote: On Sat, Aug 08, 2015 at 10:07:00PM +0200, Marton Balint wrote: FFplay was using a 5 frame packet buffer, this is not much (e.g. 200 ms for 25fps video), when HLS is requesting a new segment via HTTP, it may take longer for the request to

Re: [FFmpeg-devel] [PATCH] movtextdec: Use default style information from movtext header

2015-08-09 Thread Niklesh Lalwani
+ +typedef struct { +char *font; +int fontsize; +int color; +int back_color; +int bold; +int italic; +int underline; +int alignment; +} Movtext_default; + This name is inconsistent. Use MovTextDefault. Yes, makes sense. Updated patch

Re: [FFmpeg-devel] [PATCH] movtextdec: Use default style information from movtext header

2015-08-09 Thread Michael Niedermayer
On Mon, Aug 10, 2015 at 01:18:58AM +0530, Niklesh Lalwani wrote: + +typedef struct { +char *font; +int fontsize; +int color; +int back_color; +int bold; +int italic; +int underline; +int alignment; +} Movtext_default; + This

Re: [FFmpeg-devel] [PATCH 5/6] aaccoder: Move the quantization function to a separate file

2015-08-09 Thread Rostislav Pehlivanov
Ah, you're right, thanks. I'll leave this out of any future revisions of the patch unless I get around to getting the MIPS encoder to compile to check it. On 9 August 2015 at 21:44, Michael Niedermayer mich...@niedermayer.cc wrote: On Wed, Jul 29, 2015 at 05:44:28AM +0100, Rostislav Pehlivanov

Re: [FFmpeg-devel] [PATCH 4/4] fate/subtitles: add a new test for SAMI demuxer

2015-08-09 Thread Yayoi Ukai
And sample with a new name. On Sun, Aug 9, 2015 at 9:39 PM, Yayoi yayoi.u...@gmail.com wrote: and decoder --- tests/fate/subtitles.mak | 3 ++ tests/ref/fate/sub-sami2 | 91 2 files changed, 94 insertions(+) create mode 100644

Re: [FFmpeg-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread Frank Lömker
Hello, On 08/09/2015 08:30 PM, Clément Bœsch wrote: On Sun, Aug 09, 2015 at 08:28:28PM +0200, Frank Lömker wrote: Hello, On 08/09/2015 06:57 AM, Ronald S. Bultje wrote: Does not LGTM. Yeah, I'm with this. Andreas, the correct fix is to update applications, even if that means

[FFmpeg-devel] [PATCH 1/4] avcodec/srtdec: factor out HTML parsing code

2015-08-09 Thread Yayoi
This code will be used in SAMI decoder in a later commit. --- libavcodec/Makefile| 2 +- libavcodec/htmlsubtitles.c | 201 + libavcodec/htmlsubtitles.h | 29 +++ libavcodec/srtdec.c| 149 + 4 files

[FFmpeg-devel] [PATCH 2/4] avcodec/samidec: use ff_htmlmarkup_to_ass()

2015-08-09 Thread Yayoi
--- libavcodec/Makefile | 2 +- libavcodec/samidec.c| 71 - tests/ref/fate/sub-sami | 18 ++--- 3 files changed, 56 insertions(+), 35 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 4ca824b..8201aa0 100644

Re: [FFmpeg-devel] [PATCH] postproc: Deprecate QP_STORE_T, it lacks a PP/AV/FF prefix

2015-08-09 Thread Michael Niedermayer
On Sun, Aug 09, 2015 at 08:27:15PM -0400, Ronald S. Bultje wrote: Hi, On Sun, Aug 9, 2015 at 5:59 PM, Michael Niedermayer michae...@gmx.at wrote: From: Michael Niedermayer mich...@niedermayer.cc Signed-off-by: Michael Niedermayer mich...@niedermayer.cc ---

[FFmpeg-devel] [PATCH 4/4] fate/subtitles: add a new test for SAMI demuxer

2015-08-09 Thread Yayoi
and decoder --- tests/fate/subtitles.mak | 3 ++ tests/ref/fate/sub-sami2 | 91 2 files changed, 94 insertions(+) create mode 100644 tests/ref/fate/sub-sami2 diff --git a/tests/fate/subtitles.mak b/tests/fate/subtitles.mak index d8b2034..f5270ea

[FFmpeg-devel] [PATCH 3/4] avformat/samidec:do not include trailing sami

2015-08-09 Thread Yayoi
footer in the last packet --- libavformat/samidec.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavformat/samidec.c b/libavformat/samidec.c index 948e1ed..bc9b745 100644 --- a/libavformat/samidec.c +++ b/libavformat/samidec.c @@ -68,11 +68,17 @@ static int

Re: [FFmpeg-devel] [PATCH] postproc: Deprecate QP_STORE_T, it lacks a PP/AV/FF prefix

2015-08-09 Thread Ronald S. Bultje
Hi, On Sun, Aug 9, 2015 at 5:59 PM, Michael Niedermayer michae...@gmx.at wrote: From: Michael Niedermayer mich...@niedermayer.cc Signed-off-by: Michael Niedermayer mich...@niedermayer.cc --- libpostproc/postprocess.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

Re: [FFmpeg-devel] [PATCH] avformat: Move MAX_PROBE_PACKETS to internal.h

2015-08-09 Thread Ronald S. Bultje
Hi, On Sun, Aug 9, 2015 at 5:21 PM, Michael Niedermayer michae...@gmx.at wrote: From: Michael Niedermayer mich...@niedermayer.cc Signed-off-by: Michael Niedermayer mich...@niedermayer.cc --- libavformat/avformat.h |1 - libavformat/internal.h |2 ++ 2 files changed, 2

Re: [FFmpeg-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

2015-08-09 Thread Andreas Cadhalpun
Hi, On 09.08.2015 12:57, Ronald S. Bultje wrote: Yeah, I'm with this. Andreas, the correct fix is to update applications, even if that means vendor-specific patches in Debian. These are exceptionally trivial patches that you can generate using fairly trivial sed scripting. The problem is not