Re: [FFmpeg-devel] [PATCH]lavf/matroskaenc: Print an error if an unreadable rawvideo pix_fmt is written

On Sat, Jan 14, 2017 at 4:18 PM, Carl Eugen Hoyos wrote: > 2017-01-13 6:28 GMT+01:00 wm4 : >> On Thu, 12 Jan 2017 23:31:11 +0100 >> Carl Eugen Hoyos wrote: >> >>> 2017-01-12 23:20 GMT+01:00 Hendrik Leppkes : >>>

Re: [FFmpeg-devel] [PATCH]lavf/matroskaenc: Print an error if an unreadable rawvideo pix_fmt is written

2017-01-13 6:28 GMT+01:00 wm4 : > On Thu, 12 Jan 2017 23:31:11 +0100 > Carl Eugen Hoyos wrote: > >> 2017-01-12 23:20 GMT+01:00 Hendrik Leppkes : >> > On Fri, Jan 13, 2017 at 1:10 AM, Carl Eugen Hoyos >> > wrote:

Re: [FFmpeg-devel] [PATCH]lavf/matroskaenc: Do not write two CodecID elements for rawvideo

2017-01-13 18:44 GMT+01:00 Michael Niedermayer : > On Thu, Jan 12, 2017 at 03:01:03PM +0100, Carl Eugen Hoyos wrote: >> 2017-01-11 19:55 GMT+01:00 Michael Niedermayer : >> > On Tue, Jan 10, 2017 at 05:23:04PM +0100, Carl Eugen Hoyos wrote: >> >> Hi! >> >>

Re: [FFmpeg-devel] [PATCH] avformat/mxf: add support for Prores from Final Cut Pro

2017-01-12 18:27 GMT+01:00 Paul B Mahol : > Signed-off-by: Paul B Mahol > --- > libavformat/mxf.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/mxf.c b/libavformat/mxf.c > index e9c48e8..8a40b67 100644 > --- a/libavformat/mxf.c > +++

Re: [FFmpeg-devel] libavcodec/psd : add test for channel depth/count in bitmap mode

On Wed, Jan 11, 2017 at 09:55:33PM +0100, Martin Vignali wrote: > Hello, > > Patch in attach add test for channel depth and channel count in bitmap mode > and return invalid data, if channel count or depth != 1. > > Martin > psd.c |6 ++ > 1 file changed, 6 insertions(+) >

Re: [FFmpeg-devel] fate/psd : add test for bitmap and duotone

On Wed, Jan 11, 2017 at 09:59:24PM +0100, Martin Vignali wrote: > Hello, > > In attach patch to add fate test for bitmap file, and duotone. > > The duotone test, is based on the actual state of the decoder, where the > sample is interpreted as gray file. > > Sample can be found here : >

[FFmpeg-devel] [PATCH] Fix huge reported DTS on Ogg Theora files with bogus granulepos

Patch fixes an issue with broken Ogg Theora videos containing bogus negative granulepos on some packets: libavformat was reporting huge positive dts timestamps for these packets, causing downstream problems for ffmpeg2theora at Wikimedia Commons (near-infinite loop during transcoding). Bug

Re: [FFmpeg-devel] [PATCH 1/1] avformat: Add alignment mode to Pro-MPEG

On Fri, Jan 13, 2017 at 09:26:16 -0500, Andreas Håkon wrote: > { "d", "FEC D", OFFSET(d), AV_OPT_TYPE_INT, { .i64 = 5 }, 4, 20, .flags > = E }, > +{ "align", "Alignment mode: 0=none; 1=COL (default); 2=ROW", > OFFSET(align), AV_OPT_TYPE_INT, { .i64 = 1 }, 0, 2, .flags = E }, This

Re: [FFmpeg-devel] [PATCH] avformat: Add rtp_mpegtsraw (MPEG-TS RAW Stream output)

On Fri, Jan 13, 2017 at 07:57:28 -0500, Andreas Håkon wrote: > +static int rtp_mpegtsraw_write_close(AVFormatContext *s) > +{ ffmpeg bracket style is different. https://ffmpeg.org/developer.html#toc-Code-formatting-conventions (indent's "-kr" implies "-br", which means that the opening curly

Re: [FFmpeg-devel] swresample/aarch64: add s16 and fltp neon resampling functions

On Fri, Jan 13, 2017 at 03:43:24PM +0100, Paul B Mahol wrote: > On 1/13/17, Matthieu Bouron wrote: > > On Mon, Jan 09, 2017 at 11:00:44PM +0100, Michael Niedermayer wrote: > >> On Mon, Jan 09, 2017 at 05:45:29PM +0100, Matthieu Bouron wrote: > >> > Hello, > >> > > >> >

Re: [FFmpeg-devel] [PATCH] speedhq: Align blocks variable properly.

On 1/13/2017 3:04 PM, Steinar H. Gunderson wrote: > Seemingly ff_clear_block_sse assumed that the block array is aligned, > so make sure it is. And so does IDCTDSPContext.idct_put() > --- > libavcodec/speedhq.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

[FFmpeg-devel] [PATCH] speedhq: Align blocks variable properly.

Seemingly ff_clear_block_sse assumed that the block array is aligned, so make sure it is. --- libavcodec/speedhq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/speedhq.c b/libavcodec/speedhq.c index 9c21c0b1be..30160dd3f2 100644 --- a/libavcodec/speedhq.c +++

Re: [FFmpeg-devel] [PATCH 1/3] lavc/pthread_frame: protect read state access in setup finish function

On Fri, 13 Jan 2017 12:19:14 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Fri, Jan 13, 2017 at 9:39 AM, wm4 wrote: > > > On Fri, 13 Jan 2017 09:07:48 -0500 > > "Ronald S. Bultje" wrote: > > > > > Hi, > > > > > > On Jan 13, 2017

Re: [FFmpeg-devel] [PATCH]lavf/matroskaenc: Do not write two CodecID elements for rawvideo

On Thu, Jan 12, 2017 at 03:01:03PM +0100, Carl Eugen Hoyos wrote: > 2017-01-11 19:55 GMT+01:00 Michael Niedermayer : > > On Tue, Jan 10, 2017 at 05:23:04PM +0100, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> Attached patch fixes ticket #6068. > >> > >> Please comment, Carl

Re: [FFmpeg-devel] [PATCH] HTTP: optimize forward seek performance

> On Jan 12, 2017, at 10:59 AM, Joel Cunningham wrote: > > Nicolas, > > I’ve found existing “read-ahead logic” in avio_seek to do what I’ve > implemented in http_stream_forward(). This is controlled by > SHORT_SEEK_THRESHOLD, currently set to 4KB. I proto-typed

Re: [FFmpeg-devel] [PATCH 1/3] lavc/pthread_frame: protect read state access in setup finish function

Hi, On Fri, Jan 13, 2017 at 9:39 AM, wm4 wrote: > On Fri, 13 Jan 2017 09:07:48 -0500 > "Ronald S. Bultje" wrote: > > > Hi, > > > > On Jan 13, 2017 6:40 AM, "Clément Bœsch" wrote: > > > > From: Clément Bœsch > > > >

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec.c: testing MPEG-4 ALS decoder with floating point audio data

On Fri, Jan 13, 2017 at 03:13:15AM -0800, Thomas Turner wrote: > If als_07_2ch192k32bF.mp4 isn't already located in > fate-suite/lossless-audio/, you can download at: > >

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Accept time base hint

On Wed, Dec 28, 2016 at 5:03 AM, Michael Niedermayer wrote: > On Tue, Dec 27, 2016 at 09:47:47PM -0800, Michael Bradshaw wrote: > > From: Michael Bradshaw > > > > Signed-off-by: Michael Bradshaw > > --- > >

Re: [FFmpeg-devel] [PATCH 2/3] avformat/avienc: add reserve_index_space option

On Tue, Jan 10, 2017 at 10:41:04AM +0100, Tobias Rapp wrote: > On 10.01.2017 00:15, Michael Niedermayer wrote: > >On Mon, Jan 09, 2017 at 09:56:51AM +0100, Tobias Rapp wrote: > >>Allows the user to reserve space for the ODML master index. A sufficient > >>sized master index in the AVI header

Re: [FFmpeg-devel] [PATCH 1/3] avformat/avienc: write chained master index only if std_compliance is normal or below

On Tue, Jan 10, 2017 at 09:26:53AM +0100, Tobias Rapp wrote: > On 10.01.2017 00:19, Michael Niedermayer wrote: > >On Mon, Jan 09, 2017 at 09:56:50AM +0100, Tobias Rapp wrote: > >>OpenDML specification v1.02 states that entries of a master index chunk > >>shall point to standard or field index

Re: [FFmpeg-devel] [PATCH 2/4] dxva2: use a single macro to test if the DXVA context is valid

On Fri, Jan 13, 2017 at 02:02:29PM +0100, Steve Lhomme wrote: > On Fri, Jan 13, 2017 at 1:04 PM, Michael Niedermayer wrote: > > On Fri, Jan 13, 2017 at 01:01:14PM +0100, Michael Niedermayer wrote: > >> On Fri, Jan 13, 2017 at 10:40:23AM +, Stève Lhomme wrote: > >> > On Fri,

Re: [FFmpeg-devel] [PATCH] avcodec/h264dec: Call ff_print_debug_info2() in flush codepath

On Fri, Jan 13, 2017 at 12:55:33PM +0100, Clément Bœsch wrote: > On Thu, Jan 12, 2017 at 12:11:37PM +0100, Michael Niedermayer wrote: > > Found-by: ubitux > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/h264dec.c | 9 + > > 1 file changed, 9

Re: [FFmpeg-devel] [PATCH 3/3] lavc/h264dec: reconstruct and debug flush frames as well

On Fri, Jan 13, 2017 at 12:40:07PM +0100, Clément Bœsch wrote: > From: Clément Bœsch > > --- > libavcodec/h264dec.c | 99 > +--- > 1 file changed, 56 insertions(+), 43 deletions(-) LGTM thx [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 2/3] lavc/h264_slice: drop redundant current_slice reset

On Fri, Jan 13, 2017 at 12:40:06PM +0100, Clément Bœsch wrote: > From: Clément Bœsch > > It is done unconditionally in ff_h264_field_end() > --- > libavcodec/h264_slice.c | 2 -- > 1 file changed, 2 deletions(-) ok thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] swresample/aarch64: add s16 and fltp neon resampling functions

On 1/13/17, Matthieu Bouron wrote: > On Mon, Jan 09, 2017 at 11:00:44PM +0100, Michael Niedermayer wrote: >> On Mon, Jan 09, 2017 at 05:45:29PM +0100, Matthieu Bouron wrote: >> > Hello, >> > >> > The following patch is a port of >> >

Re: [FFmpeg-devel] [PATCH 1/3] lavc/pthread_frame: protect read state access in setup finish function

On Fri, 13 Jan 2017 09:07:48 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Jan 13, 2017 6:40 AM, "Clément Bœsch" wrote: > > From: Clément Bœsch > > --- > libavcodec/pthread_frame.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

Re: [FFmpeg-devel] [PATCH 1/3] lavc/pthread_frame: protect read state access in setup finish function

Hi, On Jan 13, 2017 6:40 AM, "Clément Bœsch" wrote: From: Clément Bœsch --- libavcodec/pthread_frame.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c index 7ef5e9f6be..cb6d76284e

[FFmpeg-devel] [PATCH 1/1] avformat: Add alignment mode to Pro-MPEG

Hi, The Pro-MPEG FEC module has troubles with some receivers. This patch fixes this problem incorporating the option for selecting different alignment modes. The problem is related to the alignment of the COLUMN FEC packets (aka. when send the packet). Current block-alignment implies a delay

Re: [FFmpeg-devel] swresample/aarch64: add s16 and fltp neon resampling functions

On Mon, Jan 09, 2017 at 11:00:44PM +0100, Michael Niedermayer wrote: > On Mon, Jan 09, 2017 at 05:45:29PM +0100, Matthieu Bouron wrote: > > Hello, > > > > The following patch is a port of 6d82b52712af87c7831872c27fd2abfd18b04f88 to > > the aarch64 platform. > > > > Here are some results of an

Re: [FFmpeg-devel] No header for avformat/hlsenc

On Fri, Jan 13, 2017 at 4:51 PM, wm4 wrote: > On Fri, 13 Jan 2017 16:04:57 +0530 > Kamaldeep Tumkur wrote: > >> Thanks a lot for explaining this. I do see now that the needed >> metadata is being set on the HTTPContext struct in the http

Re: [FFmpeg-devel] [PATCH 2/4] dxva2: use a single macro to test if the DXVA context is valid

On Fri, Jan 13, 2017 at 1:04 PM, Michael Niedermayer wrote: > On Fri, Jan 13, 2017 at 01:01:14PM +0100, Michael Niedermayer wrote: >> On Fri, Jan 13, 2017 at 10:40:23AM +, Stève Lhomme wrote: >> > On Fri, Jan 13, 2017 at 10:47 AM, wm4 wrote: >> > > On

[FFmpeg-devel] [PATCH 3/3] dxva2: allow an empty array of ID3D11VideoDecoderOutputView

We can pick the correct slice index directly from the ID3D11VideoDecoderOutputView casted from data[3]. Also added myself as maintainer for DXVA2 and D3D11VA. --- MAINTAINERS | 3 ++- libavcodec/dxva2_internal.h | 4 ++-- libavcodec/version.h| 2 +- 3 files changed, 5

[FFmpeg-devel] [PATCH 2/3] dxva2: get the slice number directly from the surface in D3D11VA

No need to loop through the known surfaces, we'll use the requested surface anyway. The loop is only done for DXVA2. --- libavcodec/dxva2.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/libavcodec/dxva2.c b/libavcodec/dxva2.c index 2ac3f3e..c782f1c 100644

[FFmpeg-devel] [PATCH 1/3] dxva2: use a single macro to test if the DXVA context is valid

--- libavcodec/dxva2_h264.c | 4 +--- libavcodec/dxva2_hevc.c | 4 +--- libavcodec/dxva2_internal.h | 5 + libavcodec/dxva2_mpeg2.c| 4 +--- libavcodec/dxva2_vc1.c | 4 +--- libavcodec/dxva2_vp9.c | 4 +--- 6 files changed, 10 insertions(+), 15 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] avformat: Add rtp_mpegtsraw (MPEG-TS RAW Stream output)

Hi, I resend the attachment with the correct mime type: text/plain Sorry! Original Message Subject: [PATCH] avformat: Add rtp_mpegtsraw (MPEG-TS RAW Stream output) From: andreas.ha...@protonmail.com To: ffmpeg-devel@ffmpeg.org Hi, This patch adds

Re: [FFmpeg-devel] [PATCH 1/1] avformat: Fix Pro-MPEG non-square matrix

Hi, Impossible to send in plain-text... Attached as text/plain in base64From 5b4401af5bdfe4736f5192f5dc118948f4906bda Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andreas=20H=C3=A5kon?= Date: Fri, 13 Jan 2017 13:07:57 +0100 Subject: [PATCH 1/1] avformat: Fix Pro-MPEG

Re: [FFmpeg-devel] [PATCH 1/1] avformat: Fix Pro-MPEG non-square matrix

Hi, > Original Message > Subject: [FFmpeg-devel] [PATCH 1/1] avformat: Fix Pro-MPEG non-square matrix > From: andreas.ha...@protonmail.com > To: FFmpeg development discussions and patches > > Hi, > > I share this for comment... > With square matrices I

[FFmpeg-devel] [PATCH 1/1] avformat: Fix Pro-MPEG non-square matrix

Hi, I share this for comment... With square matrices I don't see any problem, as row=col when sending the COL FEC packets. But, with a non-square matrix the "col_out_idx" has inconsistent values (repeat values). Please, review this code. I feel now is correct. Best, A.H.

Re: [FFmpeg-devel] [PATCH 2/4] dxva2: use a single macro to test if the DXVA context is valid

On Fri, Jan 13, 2017 at 01:01:14PM +0100, Michael Niedermayer wrote: > On Fri, Jan 13, 2017 at 10:40:23AM +, Stève Lhomme wrote: > > On Fri, Jan 13, 2017 at 10:47 AM, wm4 wrote: > > > On Fri, 13 Jan 2017 09:52:18 +0100 > > > Steve Lhomme wrote: > > >

Re: [FFmpeg-devel] [PATCH 2/4] dxva2: use a single macro to test if the DXVA context is valid

On Fri, Jan 13, 2017 at 10:40:23AM +, Stève Lhomme wrote: > On Fri, Jan 13, 2017 at 10:47 AM, wm4 wrote: > > On Fri, 13 Jan 2017 09:52:18 +0100 > > Steve Lhomme wrote: > > > >> For the record all 3 remaining patches (2/4, 3/4 and 4/4) are now > >>

Re: [FFmpeg-devel] [PATCH] avcodec/h264dec: Call ff_print_debug_info2() in flush codepath

On Thu, Jan 12, 2017 at 12:11:37PM +0100, Michael Niedermayer wrote: > Found-by: ubitux > Signed-off-by: Michael Niedermayer > --- > libavcodec/h264dec.c | 9 + > 1 file changed, 9 insertions(+) > > diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c > index

[FFmpeg-devel] [PATCH] avcodec/alsdec.c: testing MPEG-4 ALS decoder with floating point audio data

If als_07_2ch192k32bF.mp4 isn't already located in fate-suite/lossless-audio/, you can download at: http://standards.iso.org/ittf/PubliclyAvailableStandards/ISO_IEC_14496-26_2010_Bitstreams/DVD1/mpeg4audio-conformance/compressedMp4/als_07_2ch192k32bF.mp4 Signed-off-by: Thomas Turner

[FFmpeg-devel] [PATCH 1/3] lavc/pthread_frame: protect read state access in setup finish function

From: Clément Bœsch --- libavcodec/pthread_frame.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c index 7ef5e9f6be..cb6d76284e 100644 --- a/libavcodec/pthread_frame.c +++ b/libavcodec/pthread_frame.c

[FFmpeg-devel] [PATCH 2/3] lavc/h264_slice: drop redundant current_slice reset

From: Clément Bœsch It is done unconditionally in ff_h264_field_end() --- libavcodec/h264_slice.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c index 9a334a97cb..2dc98c109c 100644 --- a/libavcodec/h264_slice.c +++

[FFmpeg-devel] [PATCH 3/3] lavc/h264dec: reconstruct and debug flush frames as well

From: Clément Bœsch --- libavcodec/h264dec.c | 99 +--- 1 file changed, 56 insertions(+), 43 deletions(-) diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c index ed0b7246b7..0cb0352485 100644 --- a/libavcodec/h264dec.c

Re: [FFmpeg-devel] No header for avformat/hlsenc

On Fri, 13 Jan 2017 16:04:57 +0530 Kamaldeep Tumkur wrote: > Thanks a lot for explaining this. I do see now that the needed > metadata is being set on the HTTPContext struct in the http protocol > file. For the ffmpeg.c app, AVFormatContext along with a few others >

Re: [FFmpeg-devel] No header for avformat/hlsenc

Thanks a lot for explaining this. I do see now that the needed metadata is being set on the HTTPContext struct in the http protocol file. For the ffmpeg.c app, AVFormatContext along with a few others are the only accessible structs through the public headers. HTTPContext, HLSContext and others

Re: [FFmpeg-devel] [PATCH 2/4] dxva2: use a single macro to test if the DXVA context is valid

On Fri, Jan 13, 2017 at 10:47 AM, wm4 wrote: > On Fri, 13 Jan 2017 09:52:18 +0100 > Steve Lhomme wrote: > >> For the record all 3 remaining patches (2/4, 3/4 and 4/4) are now >> merged in libav. > > So how do we handle this. Do we wait until the merges

Re: [FFmpeg-devel] No header for avformat/hlsenc

On Fri, 13 Jan 2017 14:35:09 +0530 Kamaldeep Tumkur wrote: > Hi All, > > I am trying to use the HLSContext struct inside ffmpeg.c, in order to > have access to the context's send_buffer_size value. However, the > context struct is only defined inside hlsenc.c. > >

Re: [FFmpeg-devel] [PATCH 2/4] dxva2: use a single macro to test if the DXVA context is valid

On Fri, 13 Jan 2017 09:52:18 +0100 Steve Lhomme wrote: > For the record all 3 remaining patches (2/4, 3/4 and 4/4) are now > merged in libav. So how do we handle this. Do we wait until the merges catch up, or do you want to push your patches to ffmpeg's git immediately?

[FFmpeg-devel] No header for avformat/hlsenc

Hi All, I am trying to use the HLSContext struct inside ffmpeg.c, in order to have access to the context's send_buffer_size value. However, the context struct is only defined inside hlsenc.c. Is there a specific reason for hlsenc not to have a header file? Are there any other options I may be

Re: [FFmpeg-devel] [PATCH 2/4] dxva2: use a single macro to test if the DXVA context is valid

For the record all 3 remaining patches (2/4, 3/4 and 4/4) are now merged in libav. On Tue, Jan 10, 2017 at 11:18 AM, Steve Lhomme wrote: > This was not rejected and has been merged in libav. > 3/4 and 4/4 seem to be valid too and pending on both sides. I just > need to send an