Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-02 Thread compn
On Mon, 02 May 2016 18:50:18 +0200 Piotr Bandurski wrote: > > > > > VIDC.m702=digivcap.dll Matrox Offline HD > > > > It looks like I managed to create samples for this: > > > > > > but the problem is that there are some errors while decoding > > "m702_2.avi". > > > >

Re: [FFmpeg-devel] [PATCH] rtpenc: packetizer for VC-2 HQ RTP payload format (draft v1)

2016-05-02 Thread Michael Niedermayer
On Sun, May 01, 2016 at 01:42:58PM +0200, Thomas Volkert wrote: > From: Thomas Volkert > > --- > Changelog | 2 +- > MAINTAINERS| 1 + > libavformat/Makefile | 1 + > libavformat/rtpenc.c | 15 + > libavformat/rtpenc.h

Re: [FFmpeg-devel] [PATCH 1/4] fate: wma: add lossless 24bits test

2016-05-02 Thread Michael Niedermayer
On Sun, May 01, 2016 at 06:47:32PM +0200, Christophe Gisquet wrote: > 2016-05-01 15:54 GMT+02:00 Paul B Mahol : > > There where 2 distinct issues: 32bit instead of 16bit integers and > > wrong handling of raw pcm. > > The 96k is about the first one, last decoded frame md5 differs

Re: [FFmpeg-devel] [PATCH 1/4] fate: wma: add lossless 24bits test

2016-05-02 Thread Christophe Gisquet
Hi, 2016-05-02 16:02 GMT+02:00 Michael Niedermayer : >> +fate-lossless-wma24-rawtile: CMD = md5 -i >> $(TARGET_SAMPLES)/lossless-audio/g2_24bit.wma -f s24le > > where can i find that file ? > i assume i should upload it ? Sorry, I thought we had discussed it in this

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-02 Thread Piotr Bandurski
> > > > VIDC.m702=digivcap.dll Matrox Offline HD > > It looks like I managed to create samples for this: > > "This codec renders video to a proxy HD video format for video editing > purposes. > > Resolutions supported: 320x180 for 720p projects, and 480x270 for 1080i/p > projects" > > but

[FFmpeg-devel] [PATCH] Add "sar" alias to "aspect" option of video encoders

2016-05-02 Thread Andrey Utkin
It is impossible to pass "aspect" parameter to encoder from ffmpeg CLI because option from lavc/options_table.h is eclipsed by option with same name in ffmpeg_opt.c, which has different meaning (DAR, not SAR). --- libavcodec/options_table.h | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [FFmpeg-devel] [PATCH 1/4] fate: wma: add lossless 24bits test

2016-05-02 Thread Michael Niedermayer
On Mon, May 02, 2016 at 04:16:29PM +0200, Christophe Gisquet wrote: > Hi, > > 2016-05-02 16:02 GMT+02:00 Michael Niedermayer : > >> +fate-lossless-wma24-rawtile: CMD = md5 -i > >> $(TARGET_SAMPLES)/lossless-audio/g2_24bit.wma -f s24le > > > > where can i find that file ?

Re: [FFmpeg-devel] [PATCH] Add "sar" alias to "aspect" option of video encoders

2016-05-02 Thread Michael Niedermayer
On Mon, May 02, 2016 at 08:36:54PM +0300, Andrey Utkin wrote: > It is impossible to pass "aspect" parameter to encoder from ffmpeg CLI > because option from lavc/options_table.h is eclipsed by option with same > name in ffmpeg_opt.c, which has different meaning (DAR, not SAR). > --- >

Re: [FFmpeg-devel] [PATCH]lavf/webpenc: Do not set alpha flag for yuv420p

2016-05-02 Thread wm4
On Mon, 2 May 2016 08:36:16 -0400 "Ronald S. Bultje" wrote: > Hi, > > On Mon, May 2, 2016 at 5:49 AM, Carl Eugen Hoyos wrote: > > > On Wednesday 13 April 2016 10:27:03 am you wrote: > > > Maybe it should just check whether the pixel format is alpha,

Re: [FFmpeg-devel] [PATCH 1/4] fate: wma: add lossless 24bits test

2016-05-02 Thread Michael Niedermayer
On Sun, May 01, 2016 at 06:47:32PM +0200, Christophe Gisquet wrote: > 2016-05-01 15:54 GMT+02:00 Paul B Mahol : > > There where 2 distinct issues: 32bit instead of 16bit integers and > > wrong handling of raw pcm. > > The 96k is about the first one, last decoded frame md5 differs

Re: [FFmpeg-devel] [PATCH 1/2] avformat/tee: Fix TeeSlave.bsfs pointer array size

2016-05-02 Thread Michael Niedermayer
On Mon, May 02, 2016 at 01:34:45AM +0300, sebechlebsky...@gmail.com wrote: > From: Jan Sebechlebsky > > TeeSlave.bsfs is array of pointers to AVBitStreamFilterContext, > so element size should be really size of a pointer, not size > of TeeSlave structure. > >

Re: [FFmpeg-devel] [PATCH v2 0/3] DTS Express (LBR) decoder

2016-05-02 Thread Hendrik Leppkes
On Sun, May 1, 2016 at 11:43 PM, Hendrik Leppkes wrote: > On Sun, May 1, 2016 at 8:30 PM, James Almer wrote: >> With the samples you shared and with a random lbr-in-wav mono sample i found >> in the wild i get the following when i try to do a codec copy.

Re: [FFmpeg-devel] [PATCH] web/download: Change main repository URL to use https

2016-05-02 Thread Hendrik Leppkes
On Mon, May 2, 2016 at 2:54 AM, Michael Niedermayer wrote: > This also switches the public git URL to use the same server as ffmpeg-web & > fateserver git > That way all git server URLs are also much more consistent LGTM, since videolan switched to the "smart" http git

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-02 Thread Michael Niedermayer
On Mon, May 02, 2016 at 06:50:18PM +0200, Piotr Bandurski wrote: > > > > > VIDC.m702=digivcap.dll Matrox Offline HD > > > > It looks like I managed to create samples for this: > > > > "This codec renders video to a proxy HD video format for video editing > > purposes. > > > > Resolutions

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: add avpriv_dca_compute_crc()

2016-05-02 Thread James Almer
On 5/2/2016 5:47 PM, foo86 wrote: > --- > libavcodec/dca.c | 18 ++ > libavcodec/dca.h | 5 + > libavcodec/dcadec.c | 20 +--- > libavcodec/version.h | 2 +- > 4 files changed, 25 insertions(+), 20 deletions(-) > > diff --git a/libavcodec/dca.c

[FFmpeg-devel] [PATCH] avcodec/dca: fix sync word search error condition

2016-05-02 Thread foo86
This didn't actually check if sync word was found and always errored out with "-err_detect explode" option enabled. --- libavcodec/dca_core.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/libavcodec/dca_core.c b/libavcodec/dca_core.c index fc4fdf4..58c78f7

[FFmpeg-devel] [PATCH 2/3] avcodec/dca: inline ff_dca_check_crc()

2016-05-02 Thread foo86
--- libavcodec/dca_core.c | 25 ++--- libavcodec/dca_exss.c | 3 +-- libavcodec/dca_xll.c | 9 +++-- libavcodec/dcadec.c | 9 - libavcodec/dcadec.h | 12 ++-- 5 files changed, 24 insertions(+), 34 deletions(-) diff --git a/libavcodec/dca_core.c

[FFmpeg-devel] [PATCH 3/3] avformat/dtsdec: detect core-less streams

2016-05-02 Thread foo86
--- libavformat/dtsdec.c | 42 +++--- 1 file changed, 35 insertions(+), 7 deletions(-) diff --git a/libavformat/dtsdec.c b/libavformat/dtsdec.c index ef28391..b54b6ef 100644 --- a/libavformat/dtsdec.c +++ b/libavformat/dtsdec.c @@ -32,23 +32,48 @@ static int

[FFmpeg-devel] [PATCH 1/3] avcodec: add avpriv_dca_compute_crc()

2016-05-02 Thread foo86
--- libavcodec/dca.c | 18 ++ libavcodec/dca.h | 5 + libavcodec/dcadec.c | 20 +--- libavcodec/version.h | 2 +- 4 files changed, 25 insertions(+), 20 deletions(-) diff --git a/libavcodec/dca.c b/libavcodec/dca.c index 714509b..be89d0d 100644 ---

Re: [FFmpeg-devel] [PATCH] avcodec/dca: fix sync word search error condition

2016-05-02 Thread Michael Niedermayer
On Mon, May 02, 2016 at 11:44:47PM +0300, foo86 wrote: > This didn't actually check if sync word was found and always errored out > with "-err_detect explode" option enabled. > --- > libavcodec/dca_core.c | 15 +-- > 1 file changed, 9 insertions(+), 6 deletions(-) applied thanks

Re: [FFmpeg-devel] [PATCH] rtpenc: packetizer for VC-2 HQ RTP payload format (draft v1)

2016-05-02 Thread Thomas Volkert
On 01.05.2016 15:08, Moritz Barsnick wrote: On Sun, May 01, 2016 at 13:42:58 +0200, Thomas Volkert wrote: + "Please set -f_strict experimental in order to enable it.\n"); "-f_strict"? "deprecated; use strict, save via avconv" is how it's documented (not that I understand

Re: [FFmpeg-devel] [PATCH]lavf/webpenc: Do not set alpha flag for yuv420p

2016-05-02 Thread Paul B Mahol
On 5/2/16, Carl Eugen Hoyos wrote: > On Wednesday 13 April 2016 10:27:03 am you wrote: >> On Tue, 12 Apr 2016 19:53:13 + (UTC) >> >> Carl Eugen Hoyos wrote: >> > wm4 googlemail.com> writes: >> > > > -flags |= 2 + 16; >> > > > +flags |= 2; >>

Re: [FFmpeg-devel] [PATCH v2 0/3] DTS Express (LBR) decoder

2016-05-02 Thread foo86
On Mon, May 02, 2016 at 10:35:29AM +0200, Hendrik Leppkes wrote: > On Sun, May 1, 2016 at 11:43 PM, Hendrik Leppkes wrote: > > On Sun, May 1, 2016 at 8:30 PM, James Almer wrote: > >> With the samples you shared and with a random lbr-in-wav mono sample i >

Re: [FFmpeg-devel] [PATCH v2 1/3] avcodec/dca: move huffman data into separate object file

2016-05-02 Thread foo86
On Sun, May 01, 2016 at 08:18:45PM -0300, James Almer wrote: > On 5/1/2016 12:41 PM, foo86 wrote: > > --- > > libavcodec/Makefile |2 +- > > libavcodec/dca_core.c | 70 +--- > > libavcodec/dca_core.h |2 +- > > libavcodec/dcadata.c |9 - > > libavcodec/dcadata.h |2 - > >

[FFmpeg-devel] [PATCH v2 1/2] avcodec/dca: convert to AVCRC

2016-05-02 Thread foo86
--- libavcodec/dca_core.c | 25 ++--- libavcodec/dca_exss.c | 3 +-- libavcodec/dca_xll.c | 9 +++-- libavcodec/dcadec.c | 29 ++--- libavcodec/dcadec.h | 19 +-- 5 files changed, 33 insertions(+), 52 deletions(-) diff --git

[FFmpeg-devel] Error while setting fps filter value

2016-05-02 Thread Przemysław Sobala
Hello, I'm writing here because no one has answered me on libav-user mailing list. I'm trying to add the fps video filter to my filter chain. I set the 'fps' option using: av_opt_set_video_rate(fps_filter_ctx, "fps", (AVRational) {15,1}, AV_OPT_SEARCH_CHILDREN); but after that I can see an

Re: [FFmpeg-devel] [PATCH]lavf/webpenc: Do not set alpha flag for yuv420p

2016-05-02 Thread Carl Eugen Hoyos
On Wednesday 13 April 2016 10:27:03 am you wrote: > On Tue, 12 Apr 2016 19:53:13 + (UTC) > > Carl Eugen Hoyos wrote: > > wm4 googlemail.com> writes: > > > > -flags |= 2 + 16; > > > > +flags |= 2; > > > > +if (st->codecpar->format !=

Re: [FFmpeg-devel] [PATCH]lavf/webpenc: Do not set alpha flag for yuv420p

2016-05-02 Thread Ronald S. Bultje
Hi, On Mon, May 2, 2016 at 5:49 AM, Carl Eugen Hoyos wrote: > On Wednesday 13 April 2016 10:27:03 am you wrote: > > Maybe it should just check whether the pixel format is alpha, instead > > of whether it's not yuv420p. (Just a suggestion.) > > Isn't that exactly equivalent to