Re: [FFmpeg-devel] [PATCH 2/5] avcodec/dcaenc: move channel reordering tables to dcaenc.h

2016-04-30 Thread James Almer
On 4/27/2016 2:20 PM, foo86 wrote: > DCA core decoder no longer uses fixed tables for channel reordering. > Move them into private encoder header (and drop ff_dca_ prefix). > --- > libavcodec/dcadata.c | 42 -- > libavcodec/dcadata.h | 5 - > libavcodec

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/dcaenc: reuse shared quant levels table

2016-04-30 Thread James Almer
On 4/27/2016 2:19 PM, foo86 wrote: > --- > libavcodec/dcaenc.c | 8 > libavcodec/dcaenc.h | 7 --- > 2 files changed, 4 insertions(+), 11 deletions(-) Applied. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman

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

2016-04-30 Thread Michael Niedermayer
On Sun, May 01, 2016 at 12:16:57AM +0200, Piotr Bandurski wrote: > > On Tue, 26 Apr 2016 17:11:26 +0300 > > Александр Слободенюк wrote: > > > > > { AV_CODEC_ID_MPEG2VIDEO, MKTAG('M', '7', '0', '1') }, > > > +{ AV_CODEC_ID_MPEG2VIDEO, MKTAG('M', '7', '0', '3') }, > > > { AV_CODEC

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

2016-04-30 Thread Piotr Bandurski
> On Tue, 26 Apr 2016 17:11:26 +0300 > Александр Слободенюк wrote: > > > { AV_CODEC_ID_MPEG2VIDEO, MKTAG('M', '7', '0', '1') }, > > +{ AV_CODEC_ID_MPEG2VIDEO, MKTAG('M', '7', '0', '3') }, > > { AV_CODEC_ID_MPEG2VIDEO, MKTAG('M', '7', '0', '5') }, > > i wonder if we can add m7

Re: [FFmpeg-devel] [PATCH] avformat/riff: assign g721 and g723codec tags to g726 decoder

2016-04-30 Thread Piotr Bandurski
> > On Wed, 27 Apr 2016 20:39:34 +0200 > Piotr Bandurski wrote: > > > Subject: [PATCH] avformat/riff: assign g721 and g723 codec tags to > > g726 decoder > > > + { AV_CODEC_ID_ADPCM_G726, 0x0014 }, > > + { AV_CODEC_ID_ADPCM_G726, 0x0040 }, > > i wonder if we should make a comment that these are

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread Carl Eugen Hoyos
On Friday 29 April 2016 09:10:08 am Christoph Gisquet wrote: > Hi, > > 2016-04-27 23:58 GMT+02:00 Carl Eugen Hoyos : > > Mark Thompson jkqxz.net> writes: > >> Unless someone can show what created this file and that > >> it might make more, I suggest that the hack workaround > >> should be removed.

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread Marton Balint
On Sat, 30 Apr 2016, Michael Niedermayer wrote: On Sat, Apr 30, 2016 at 10:12:11AM -0400, Ronald S. Bultje wrote: Please revert it. I would like to see the oppinion of the people who maintain the mpeg ts demuxer before its reverted. MAINTAINERs lists Marton for mpegts.c in practice i did tr

[FFmpeg-devel] [PATCH] ffplay: force setting alsa buffer size

2016-04-30 Thread Marton Balint
Signed-off-by: Marton Balint --- ffplay.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/ffplay.c b/ffplay.c index 804bcbc..d5fcde8 100644 --- a/ffplay.c +++ b/ffplay.c @@ -3781,6 +3781,7 @@ int main(int argc, char **argv) int flags; VideoState *is; char dummy_videodriv

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread Michael Niedermayer
On Sat, Apr 30, 2016 at 10:12:11AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Apr 30, 2016 at 9:02 AM, Carl Eugen Hoyos wrote: > > > > cane please revert this patch? > > > I don't think there's wide support for it. > > > > Two developers seem to wait for an explanation why it should be re

Re: [FFmpeg-devel] [PATCH] ffplay: use 48 kHz for ALSA output

2016-04-30 Thread Marton Balint
On Thu, 21 Apr 2016, Reimar Döffinger wrote: On 20.04.2016, at 23:59, Marton Balint wrote: Signed-off-by: Marton Balint --- ffplay.c | 5 + 1 file changed, 5 insertions(+) diff --git a/ffplay.c b/ffplay.c index 804bcbc..89a34d2 100644 --- a/ffplay.c +++ b/ffplay.c @@ -2578,12 +2578,17 @

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

2016-04-30 Thread Christophe Gisquet
--- tests/fate/lossless-audio.mak | 5 - tests/ref/fate/lossless-wma24-1 | 1 + tests/ref/fate/lossless-wma24-2 | 1 + 3 files changed, 6 insertions(+), 1 deletion(-) create mode 100644 tests/ref/fate/lossless-wma24-1 create mode 100644 tests/ref/fate/lossless-wma24-2 diff --git a/tests/f

[FFmpeg-devel] [PATCH 2/5] wmalossless: allow calling madd_int16

2016-04-30 Thread Christophe Gisquet
This is done by actually handling the cascaded LMS data as if it were int16_t, thus requiring switching at various locations the computations. --- libavcodec/wmalosslessdec.c | 146 +--- 1 file changed, 84 insertions(+), 62 deletions(-) diff --git a/libavco

[FFmpeg-devel] [PATCH 5/5] wmalossless: silence a sample request

2016-04-30 Thread Christophe Gisquet
16bits samples with CDLMS orders of 8 are currently unsupported, but have never been encountered before. However, 8 seems to be the most frequent, if not the only order used for 24bits. In that case, the dsp functions are fine with handling order that are multiples of 8, so silence the warning. --

[FFmpeg-devel] [PATCH 3/5] x86: lossless audio: SSE4 madd 32bits

2016-04-30 Thread Christophe Gisquet
The unique user so far is wmalossless 24bits. The few samples tested show an order of 8, so more unrolling or an avx2 version do not make sense. Timings: 72 -> 49 cycles --- libavcodec/x86/lossless_audiodsp.asm| 31 +-- libavcodec/x86/lossless_audiodsp_init.c | 7

[FFmpeg-devel] [PATCH 4/5] lossless audio dsp: unroll

2016-04-30 Thread Christophe Gisquet
The loops are guaranteed to be at least multiples of 8, so this unrolling is safe but allows exploiting execution ports. For int32 version: 72 -> 57c. --- libavcodec/lossless_audiodsp.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/libavcodec/lossless_audiodsp.c

[FFmpeg-devel] [PATCH 0/5] wmalossless: fix 16bits speed regression v2

2016-04-30 Thread Christophe Gisquet
Patch 2 is the squashing of several previous commits, as there were no opinion on their contents nor the way to go. The SSE4 one is the final version from its last thread. The last patch in this set is new, and silences a warning that's only meaningful for 16bits content. Christophe Gisquet (5):

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread James Almer
On 4/30/2016 11:12 AM, Ronald S. Bultje wrote: > Hi, > > On Sat, Apr 30, 2016 at 9:02 AM, Carl Eugen Hoyos wrote: > >>> cane please revert this patch? >>> I don't think there's wide support for it. >> >> Two developers seem to wait for an explanation why it should be reverted. > > > Last time

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread Ronald S. Bultje
Hi, On Sat, Apr 30, 2016 at 9:02 AM, Carl Eugen Hoyos wrote: > > cane please revert this patch? > > I don't think there's wide support for it. > > Two developers seem to wait for an explanation why it should be reverted. Last time I checked, commits were not free-for-all, but should be done on

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread Nicolas George
Le duodi 12 floréal, an CCXXIV, Carl Eugen Hoyos a écrit : > Why? > Does it brake something? It increases the maintenance burden. Fixing the file itself would have no such consequence, and make it work with other software too. Also, could you please fix your mail software so that it does not brea

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread Carl Eugen Hoyos
Hi! > cane please revert this patch? Why? Does it brake something? > I don't think there's wide support for it. Two developers seem to wait for an explanation why it should be reverted. > If we find out more about the origins of the file, we can revisit this > issue. I thought the file is

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread Ronald S. Bultje
Hi, On Fri, Apr 29, 2016 at 9:08 AM, Michael Niedermayer wrote: > On Fri, Apr 29, 2016 at 08:09:34AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Fri, Apr 29, 2016 at 7:52 AM, Carl Eugen Hoyos > wrote: > > > > > I was under the impression that the only reason that demuxers and > decoders >

Re: [FFmpeg-devel] [PATCH 0/6] wmalossless: fix 16bits speed regression

2016-04-30 Thread Christophe Gisquet
2016-04-29 10:50 GMT+02:00 Paul B Mahol : > Should be OK if it doesn't break anything. I'll resend the current state of this patchset for easier testing & applying. Michael ran this under valgrind with nothing popping up, and fate passes. I think the remaining thing is: is the first version (with