[FFmpeg-devel] [PATCH] libavfilter/scale: Populate ow/oh when using 0 as w/h

The input width and height is known at parse time so there's no reason ow/oh should not be usable when using 0 as the width or height expression. Previously in "scale=0:ow" ow would be set to "0" which works, conveniently, as "scale=0:0" is perfectly valid input but this breaks down when you do

Re: [FFmpeg-devel] [PATCH] libavfilter/scale: Populate ow/oh when using 0 as w/h

On Tue, Jun 13, 2017 at 10:04 PM, Michael Niedermayer wrote: > ok, makes sense > i agree the 2nd cast seems a good idea Great, thanks Michael. I'll submit an updated patch with the additional cast. ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 13/24] vaapi_encode: Use gop_size consistently in RC parameters

On 2017/6/13 6:40, Mark Thompson wrote: > The non-H.26[45] codecs already use this form. Since we don't > currently generate I frames for codecs which support them separately > to IDR, the p_per_i variable is set to infinity by default so that it > doesn't interfere with any other calculation.

[FFmpeg-devel] [PATCH V5 4/4] lavc/tests/golomb: Add unit test for set_ue_golomb_long.

From cde2c978d2f0d7ee799619d7cd3a54c771bab7e2 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Wed, 14 Jun 2017 10:42:36 +0800 Subject: [PATCH V5 4/4] lavc/tests/golomb: Add unit test for set_ue_golomb_long. Add unit test for set_ue_golomb_long. Signed-off-by: Jun Zhao

[FFmpeg-devel] [PATCH V5 2/4] lavc/put_bits: Add put_bits64() to support up to 64 bits.

From 703be162119f78e1815d1f3b23afb24978ff36d7 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Wed, 14 Jun 2017 10:22:10 +0800 Subject: [PATCH V5 2/4] lavc/put_bits: Add put_bits64() to support up to 64 bits. put_bits64() can write up to 64 bits into a bitstream. Reviewed-by:

[FFmpeg-devel] [PATCH V5 3/4] lavc/golobm: Add set_ue_golomb_long to support up to 2^32 -2.

From e4ed2cf5a0282ca8bc24bf3d6535b8b445e23341 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Wed, 14 Jun 2017 10:35:20 +0800 Subject: [PATCH V5 3/4] lavc/golobm: Add set_ue_golomb_long to support up to 2^32 -2. add set_ue_golomb_long to support up to 2^32-2. Reviewed-by:

[FFmpeg-devel] [PATCH V5 1/4] lavc/golomb: add value range comment for set_ue_golomb().

V5: Split the patch. V4: Fix rang check error in assert base on Mark's review V3: Clean the code logic base on Michael's review. V2: Add set_ue_golomb_long() to support 32bits UE golomb and update the unit test. From 4600950115a215d64cf049233195cfd035ac11cb Mon Sep 17 00:00:00 2001 From: Jun Zhao

Re: [FFmpeg-devel] [PATCH] libavfilter/scale: Populate ow/oh when using 0 as w/h

On Tue, Jun 13, 2017 at 12:10:07AM -0400, Kevin Mark wrote: > On Mon, Jun 12, 2017 at 9:42 PM, Michael Niedermayer > wrote: > > why is there a cast at all ? > > The cast is there because if you run this: > > ffmpeg -frames:v 5 -filter_complex >

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_encode_h26[45]: respect "slices" option in h26[45] vaapi encoder.

On 2017/6/14 7:41, Mark Thompson wrote: > On 07/06/17 01:53, Jun Zhao wrote: >> From 5c88956e36e7318cf1d1b7c41a9d4108fcf9d0a5 Mon Sep 17 00:00:00 2001 >> From: Jun Zhao >> Date: Fri, 12 May 2017 08:30:43 +0800 >> Subject: [PATCH] lavc/vaapi_encode_h26[45]: respect "slices"

Re: [FFmpeg-devel] [PATCH V4] lavc/golomb: Fix UE golomb overwrite issue.

On 2017/6/13 9:33, Michael Niedermayer wrote: > On Fri, Jun 09, 2017 at 10:34:19AM +0800, Jun Zhao wrote: >> V4: Fix rang check error in assert base on Mark's review >> V3: Clean the code logic base on Michael's review. >> V2: Add set_ue_golomb_long() to support 32bits UE golomb and update the

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_encode_h26[45]: respect "slices" option in h26[45] vaapi encoder.

On 07/06/17 01:53, Jun Zhao wrote: > From 5c88956e36e7318cf1d1b7c41a9d4108fcf9d0a5 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Fri, 12 May 2017 08:30:43 +0800 > Subject: [PATCH] lavc/vaapi_encode_h26[45]: respect "slices" in h26[45] vaapi > encoder. > > Enable

Re: [FFmpeg-devel] Refund request for Customs fees of donations

On Thu, Feb 23, 2017 at 08:35:21PM +0100, Thilo Borgmann wrote: > Hi, > > in the aftermath of ELCE 2016 we recieved a donation of 5 displays from a > company. These went to Michael Niedermayer, Carl Eugen Hoyos, Ronald Bultje, > Clement Boesch and myself. > > Unfortunately, due to lack of

Re: [FFmpeg-devel] configure/libopenjpegdec.c/libopenjpegenc.c: Add support for LibOpenJPEG v2.2/git

On 13-6-2017 05:58, Michael Bradshaw wrote: > Are you sure you built ffmpeg using OpenJPEG v2.2? Because your patch is > missing the openjpeg_2_2_openjpeg_h entry in HEADERS_LIST in configure, so > you shouldn't be able to successfully build with OpenJPEG v2.2. Whoops! In my

Re: [FFmpeg-devel] [PATCH 06/24] ffmpeg: Enable generic hwaccel support for VAAPI

On 13/06/17 21:19, Michael Niedermayer wrote: > On Mon, Jun 12, 2017 at 11:40:23PM +0100, Mark Thompson wrote: >> (cherry picked from commit 62a1ef9f26c654a3e988aa465c4ac1d776c4c356) >> --- >> Makefile | 1 - >> ffmpeg.h | 2 - >> ffmpeg_opt.c | 20 - >> ffmpeg_vaapi.c |

[FFmpeg-devel] [PATCH] Add missing parenthesis to show_help_default (-h) output

--- ffmpeg_opt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index c997ea8faf..2740ef9e75 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -3127,7 +3127,7 @@ void show_help_default(const char *opt, const char *arg)

Re: [FFmpeg-devel] [PATCH 04/24] hwcontext: Make it easier to work with device types

On 13/06/17 21:23, Michael Niedermayer wrote: > On Mon, Jun 12, 2017 at 11:40:21PM +0100, Mark Thompson wrote: >> Adds functions to convert to/from strings and a function to iterate >> over all supported device types. Also adds a new invalid type >> AV_HWDEVICE_TYPE_NONE, which acts as a sentinel

Re: [FFmpeg-devel] [PATCH 00/24] Generic hardware device setup and miscellaneous related merges

On 6/12/2017 7:40 PM, Mark Thompson wrote: > This merges a set of stuff from libav to do with hardware codecs/processing. > > The two most interesting features of this are: > > * Generic hardware device setup. This finishes the uniform structure for > hardware device setup which has been in

Re: [FFmpeg-devel] [PATCH 04/24] hwcontext: Make it easier to work with device types

On Mon, Jun 12, 2017 at 11:40:21PM +0100, Mark Thompson wrote: > Adds functions to convert to/from strings and a function to iterate > over all supported device types. Also adds a new invalid type > AV_HWDEVICE_TYPE_NONE, which acts as a sentinel value. > > (cherry picked from commit

Re: [FFmpeg-devel] [PATCH 06/24] ffmpeg: Enable generic hwaccel support for VAAPI

On Mon, Jun 12, 2017 at 11:40:23PM +0100, Mark Thompson wrote: > (cherry picked from commit 62a1ef9f26c654a3e988aa465c4ac1d776c4c356) > --- > Makefile | 1 - > ffmpeg.h | 2 - > ffmpeg_opt.c | 20 - > ffmpeg_vaapi.c | 233 >

[FFmpeg-devel] [PATCH] hevc: Add support for alternative transfer characterics SEI

The use of this SEI is for backward compatibility in HLG HDR systems: older devices that cannot interpret the "arib-std-b67" transfer will get the compatible transfer (usually bt709 or bt2020) from the VUI, while newer devices that can interpret HDR will read the SEI and use its value instead.

Re: [FFmpeg-devel] [PATCH 07/11] avfilter/vf_signature: fix memory leaks in error cases

Am 13.06.2017 um 00:24 schrieb Michael Niedermayer: On Sun, Jun 11, 2017 at 04:05:49PM +0200, Timo Rothenpieler wrote: Fixes CIDs 1403234 and 1403235 --- libavfilter/vf_signature.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) LGTM thx applied

Re: [FFmpeg-devel] [PATCH 05/11] avformat/pcmdec: fix memory leak

Am 11.06.2017 um 19:07 schrieb Paul B Mahol: On 6/11/17, Timo Rothenpieler wrote: Fixes CID 1396267 --- libavformat/pcmdec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/pcmdec.c b/libavformat/pcmdec.c index 3c7e8ac84b..d0ceea6fa9 100644 ---

Re: [FFmpeg-devel] [PATCH 04/11] avformat/librtmp: check return value of setsockopt

Am 12.06.2017 um 05:07 schrieb Steven Liu: 2017-06-11 22:05 GMT+08:00 Timo Rothenpieler : Fixes CID 1396837 --- libavformat/librtmp.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/librtmp.c b/libavformat/librtmp.c index

Re: [FFmpeg-devel] [PATCH 01/11] avfilter/unsharp: fix uninitialized pointer read

Am 13.06.2017 um 00:26 schrieb Michael Niedermayer: On Sun, Jun 11, 2017 at 04:05:43PM +0200, Timo Rothenpieler wrote: Fixes CID 1396855 --- libavfilter/unsharp_opencl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) LGTM thx applied

Re: [FFmpeg-devel] [PATCH 02/11] avfilter/vf_scale_npp: fix out-of-bounds reads

applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: drop an state

On Tue, 13 Jun 2017 16:30:42 +0200 Clément Bœsch wrote: > On Tue, Jun 13, 2017 at 03:09:54PM +0200, Michael Niedermayer wrote: > [...] > > -Dialogue: 0,0:00:24.50,0:00:26.50,Default,,0,0,0,,{\an4}This text should > > be at the\Nmiddle and horizontally at the left\N(The second

Re: [FFmpeg-devel] [PATCH v3] avfilter/vf_fps: fix duration

2017-06-13 15:04 GMT+02:00 Thomas Mundt : > 2017-06-13 15:00 GMT+02:00 Nicolas George : > >> Le quintidi 25 prairial, an CCXXV, Thomas Mundt a écrit : >> > Live with the open ticket or with the modified patch? >> >> Either, actually, but I meant the modified

Re: [FFmpeg-devel] [PATCH] hevc: Add support for alternative transfer characterics SEI

On Sun, Jun 11, 2017 at 10:19 AM, James Almer wrote: > On 6/9/2017 6:35 PM, Vittorio Giovara wrote: >> Signed-off-by: Vittorio Giovara >> --- >> libavcodec/hevc_sei.c | 9 + >> libavcodec/hevc_sei.h | 7 +++ >> libavcodec/hevcdec.c | 5

Re: [FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: fix idct_col4_top coefficient

On Tue, Jun 13, 2017 at 05:26:43PM +0200, Clément Bœsch wrote: > On Tue, Jun 13, 2017 at 05:24:21PM +0200, Matthieu Bouron wrote: > > Fixes regression introduced by 5d0b8b1ae307951310c7d9a8fa282fbca9b997cd. > > --- > > libavcodec/aarch64/simple_idct_neon.S | 2 +- > > 1 file changed, 1

Re: [FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: fix idct_col4_top coefficient

On Tue, Jun 13, 2017 at 05:24:21PM +0200, Matthieu Bouron wrote: > Fixes regression introduced by 5d0b8b1ae307951310c7d9a8fa282fbca9b997cd. > --- > libavcodec/aarch64/simple_idct_neon.S | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/aarch64/simple_idct_neon.S

[FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: fix idct_col4_top coefficient

Fixes regression introduced by 5d0b8b1ae307951310c7d9a8fa282fbca9b997cd. --- libavcodec/aarch64/simple_idct_neon.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/aarch64/simple_idct_neon.S b/libavcodec/aarch64/simple_idct_neon.S index 92987985d2..5bd31e5be9 100644

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: drop an state

On Tue, Jun 13, 2017 at 03:09:54PM +0200, Michael Niedermayer wrote: [...] > -Dialogue: 0,0:00:24.50,0:00:26.50,Default,,0,0,0,,{\an4}This text should be > at the\Nmiddle and horizontally at the left\N(The second position must be > ignored) > +Dialogue:

Re: [FFmpeg-devel] [PATCH] Add FITS Decoder

On 6/13/17, Paras Chadha wrote: > Above mentioned changes have been done. > > Signed-off-by: Paras Chadha > --- > Changelog | 1 + > doc/general.texi| 2 + > libavcodec/Makefile | 1 + > libavcodec/allcodecs.c |

[FFmpeg-devel] [PATCH] Add FITS Decoder

Above mentioned changes have been done. Signed-off-by: Paras Chadha --- Changelog | 1 + doc/general.texi| 2 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 8 +

Re: [FFmpeg-devel] [PATCH] lavc/aarch64: add sbrdsp neon implementation

On Fri, Jun 02, 2017 at 01:59:00PM +0200, Matthieu Bouron wrote: > --- > > Hello, > > The following patch adds an aarch64 neon implementation of the sbrdsp (tested > on an Odroid-C2). It hasn't been benchmarked yet and it lacks the > hf_apply_noise{0,1,2,3} functions (which will be added later).

Re: [FFmpeg-devel] [PATCH 0/6] sse2/avx functions for 8-bit simple idct

Hi On Tue, Jun 13, 2017 at 03:07:32PM +0200, James Darnley wrote: > On 2017-06-13 00:18, James Darnley wrote: > > On 2017-06-12 18:57, Michael Niedermayer wrote: > >> ./ffplay ~/videos/matrixbench_mpeg2.mpg > >> looks pretty bad > > > > If that would happen to be the FATE sample > >

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

2017-01-05 13:29 GMT+03:00 Yogender Gupta : > >> There is BUG in Nvidia NVENC when you use AUD for H264 with B-frames, > it will return corrupted stream, because NVIDIA is inserting AUD type 0 > (I-frame) before B-frames instead of AUD type 7 (any-frame). > > Thanks for

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: drop an state

On Tue, 13 Jun 2017 15:09:54 +0200 Michael Niedermayer wrote: > Simplifies code but changes handling of multiple an tags > > Suggested-by: wm4 > Signed-off-by: Michael Niedermayer > --- > libavcodec/htmlsubtitles.c | 11 +-- >

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: Factor open brace handling into its own function

On Tue, Jun 13, 2017 at 01:19:56PM +0200, wm4 wrote: > On Tue, 13 Jun 2017 00:01:04 +0200 > Michael Niedermayer wrote: > > > Suggested-by: wm4 > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/htmlsubtitles.c | 44 > >

[FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: drop an state

Simplifies code but changes handling of multiple an tags Suggested-by: wm4 Signed-off-by: Michael Niedermayer --- libavcodec/htmlsubtitles.c | 11 +-- tests/ref/fate/sub-srt | 2 +- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH 0/6] sse2/avx functions for 8-bit simple idct

On 2017-06-13 00:18, James Darnley wrote: > On 2017-06-12 18:57, Michael Niedermayer wrote: >> ./ffplay ~/videos/matrixbench_mpeg2.mpg >> looks pretty bad > > If that would happen to be the FATE sample > mpeg2/matrixbench_mpeg2.lq1.mpg then I see that too. > > As I said on IRC I was able to

Re: [FFmpeg-devel] [PATCH v3] avfilter/vf_fps: fix duration

2017-06-13 15:00 GMT+02:00 Nicolas George : > Le quintidi 25 prairial, an CCXXV, Thomas Mundt a écrit : > > Live with the open ticket or with the modified patch? > > Either, actually, but I meant the modified patch. > Okay, I´ll write the patch then and send it soon. Thanks and

Re: [FFmpeg-devel] [PATCH v3] avfilter/vf_fps: fix duration

Le quintidi 25 prairial, an CCXXV, Thomas Mundt a écrit : > Live with the open ticket or with the modified patch? Either, actually, but I meant the modified patch. Regards, -- Nicolas George signature.asc Description: Digital signature ___

Re: [FFmpeg-devel] [PATCH v3] avfilter/vf_fps: fix duration

2017-06-13 14:46 GMT+02:00 Nicolas George : > Le quintidi 25 prairial, an CCXXV, Thoms Mundt a écrit : > > Since I dont see any solution or wip for this, I could modify my patch > and > > remove the use of pkt_duration. This would only work for constant frame > rate > > input,

Re: [FFmpeg-devel] [PATCH v3] avfilter/vf_fps: fix duration

Le quintidi 25 prairial, an CCXXV, Thoms Mundt a écrit : > Since I dont see any solution or wip for this, I could modify my patch and > remove the use of pkt_duration. This would only work for constant frame rate > input, but fixes ticket #2674 which is open for 4 years now. I can live with that.

Re: [FFmpeg-devel] [PATCH] lavc/mediacodecdec: switch to the new generic filtering mechanism

On Sat, Jun 10, 2017 at 01:38:20AM +0200, Matthieu Bouron wrote: > On Fri, Jun 09, 2017 at 08:18:25PM -0300, James Almer wrote: > > On 6/9/2017 7:53 PM, Matthieu Bouron wrote: > > > --- > > > libavcodec/mediacodecdec.c | 70 > > > -- > > > 1 file

Re: [FFmpeg-devel] [PATCH v3] avfilter/vf_fps: fix duration

/Nicolas George > schrieb am So, 2.4.2017:/ Le tridi 13 germinal, an CCXXV, Thomas Mundt a écrit : >/I found the use of this function in f_loop.c, so I thought it´s ok. / Well, it is a bug. I missed it when it was first committed.

Re: [FFmpeg-devel] [PATCH 00/24] Generic hardware device setup and miscellaneous related merges

On Mon, 12 Jun 2017 23:40:17 +0100 Mark Thompson wrote: > This merges a set of stuff from libav to do with hardware codecs/processing. All patches LGTM. I don't think it makes sense to delay pushing those either. ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 09/24] vf_deinterlace_vaapi: Add support for field rate output

On Mon, 12 Jun 2017 23:40:26 +0100 Mark Thompson wrote: > In order to work correctly with the i965 driver, this also fixes the > direction of forward/backward references - forward references are > intended to be those from the past to the current frame, not from the > current

Re: [FFmpeg-devel] [PATCH 01/24] hwcontext_vaapi: Try to support the VDPAU wrapper

On Mon, 12 Jun 2017 23:40:18 +0100 Mark Thompson wrote: > The driver is somewhat bitrotten (not updated for years) but is still > usable for decoding with this change. To support it, this adds a new > driver quirk to indicate no support at all for surface attributes. > > Based

Re: [FFmpeg-devel] [PATCH] doc/filters: Correct scale doc regarding w/h <= 0

On Mon, Jun 12, 2017 at 01:59:05 -0400, Kevin Mark wrote: > Signed-off-by: Kevin Mark > --- > doc/filters.texi | 39 --- > 1 file changed, 20 insertions(+), 19 deletions(-) Yes, this text looks much more sane and easy to understand.

Re: [FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: Factor open brace handling into its own function

On Tue, 13 Jun 2017 00:01:04 +0200 Michael Niedermayer wrote: > Suggested-by: wm4 > Signed-off-by: Michael Niedermayer > --- > libavcodec/htmlsubtitles.c | 44 ++-- > 1 file changed, 26 insertions(+), 18

Re: [FFmpeg-devel] [PATCH] Add support for RockChip Media Process Platform

On Tue, 13 Jun 2017 11:17:35 +0100 Mark Thompson wrote: > > +vp8_rkmpp_hwaccel_deps="rkmpp" > > vp9_d3d11va_hwaccel_deps="d3d11va DXVA_PicParams_VP9" > > vp9_d3d11va_hwaccel_select="vp9_decoder" > > vp9_dxva2_hwaccel_deps="dxva2 DXVA_PicParams_VP9" > > Why do these hwaccels

Re: [FFmpeg-devel] [PATCH] Add FITS Decoder

On 6/13/17, Paras Chadha wrote: > Hi, > > Any other changes to be made to get this patch applied? > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > Use this style

Re: [FFmpeg-devel] [PATCH] Add support for RockChip Media Process Platform

Hello, The use case for this is mostly on embedded devices that can use drm to display video with zerocopy. Most devices i have played with that can use such implementation will allow to create a drm overlay from that drmprime information with drmModeAddFB2: Here is a sample usage for it

Re: [FFmpeg-devel] [PATCH] Add support for RockChip Media Process Platform

On 13/06/17 07:21, LongChair . wrote: > From: LongChair > > This adds hardware decoding for h264 / HEVC / VP8 using MPP Rockchip API. > Will return frames holding a av_drmprime struct in buf[3} that allows drm / > dmabuf usage. > Was tested on RK3288 (TinkerBoard) and

Re: [FFmpeg-devel] [PATCH] Add FITS Decoder

Hi, Any other changes to be made to get this patch applied? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] Add support for RockChip Media Process Platform

From: LongChair This adds hardware decoding for h264 / HEVC / VP8 using MPP Rockchip API. Will return frames holding a av_drmprime struct in buf[3} that allows drm / dmabuf usage. Was tested on RK3288 (TinkerBoard) and RK3328. Additions from patch v1 - Change