Re: [FFmpeg-devel] [PATCH] alsdec: error out if channel number changes

2016-01-04 Thread Andreas Cadhalpun
On 04.01.2016 21:18, Thilo Borgmann wrote: > Am 04.01.16 um 18:31 schrieb Derek Buitenhuis: >> On 1/4/2016 3:18 PM, Andreas Cadhalpun wrote: >>> Supporting this would require re-initialization to change buffer sizes. >> >> I may be mistaken, but don't we already support some codecs which do >>

Re: [FFmpeg-devel] [PATCH] libvpx: Support setting color range for vp9.

2016-01-04 Thread James Almer
On 1/4/2016 5:01 PM, Sasi Inguva wrote: > Would it be a lot easier and correct if I just update the IMAGE_ABI_VERSION > to 4 in libvpx HEAD and check here in the decoder IMAGE_ABI_VERSION > 3 ? Yes, bumping any of the defines would probably be best. It will not apply to libvpxdec 1.5.0, though,

Re: [FFmpeg-devel] [PATCH]lavf/decimate: Fix total difference for the first frame

2016-01-04 Thread Clément Bœsch
On Mon, Jan 04, 2016 at 05:32:32PM +, Derek Buitenhuis wrote: > On 1/4/2016 5:17 PM, Carl Eugen Hoyos wrote: > > I wish I would understand;-( > > > > Since this fixes a reported issue here, I will > > commit my patch if nobody objects. > > [17:30] * Daemon404 pokes ubitux > [17:30]

[FFmpeg-devel] [PATCH] lavf/matroskadec: A_QUICKTIME and AV_CODEC_ID_NONE

2016-01-04 Thread Mats Peterson
In many older QuickTime files, the audio format, or "fourcc", is 0x (AV_CODEC_ID_NONE). The QuickTime File Format Specification states the following regarding this situation: "This format descriptor should not be used, but may be found in some files. Samples are assumed to be stored in

Re: [FFmpeg-devel] [PATCH] Enable dash output to work when the output isn't a local file

2016-01-04 Thread Hendrik Leppkes
On 04.01.2016 20:10, Derek Buitenhuis wrote: On 1/4/2016 6:05 PM, Ganesh Ajjanagadde wrote: Personally, I think it should be ok to use rename here for now, especially since even projects like Python had trouble with this aspect: https://bugs.python.org/issue8828. Someone with greater Windows

Re: [FFmpeg-devel] [PATCH]lavf/decimate: Fix total difference for the first frame

2016-01-04 Thread Clément Bœsch
On Mon, Jan 04, 2016 at 08:53:24PM +, Carl Eugen Hoyos wrote: > Clément Bœsch pkh.me> writes: > > > +if (cyclestart == 0) { > > +vdm->vmi[0].maxbdiff = vdm->vmi[1].maxbdiff; > > +vdm->vmi[0].totdiff = vdm->scthresh + 1; > > +} > > + > > > I suggest to "cherry-pick"

[FFmpeg-devel] [PATCH] lavf/mov: Audio and AV_CODEC_ID_NONE

2016-01-04 Thread Mats Peterson
In many older QuickTime files, the audio format, or "fourcc", is 0x (AV_CODEC_ID_NONE). The QuickTime File Format Specification states the following regarding this situation: "This format descriptor should not be used, but may be found in some files. Samples are assumed to be stored in

Re: [FFmpeg-devel] [PATCH] libvpx: Support setting color range for vp9.

2016-01-04 Thread Sasi Inguva
Would it be a lot easier and correct if I just update the IMAGE_ABI_VERSION to 4 in libvpx HEAD and check here in the decoder IMAGE_ABI_VERSION > 3 ? On Mon, Jan 4, 2016 at 11:03 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Jan 4, 2016 at 1:48 PM, Sasi Inguva

Re: [FFmpeg-devel] [PATCH] libvpx: Support setting color range for vp9.

2016-01-04 Thread Ronald S. Bultje
Hi Sasi, On Mon, Jan 4, 2016 at 3:01 PM, Sasi Inguva wrote: > Would it be a lot easier and correct if I just update the IMAGE_ABI_VERSION > to 4 in libvpx HEAD and check here in the decoder IMAGE_ABI_VERSION > 3 ? Sure, that's fine also. Ronald

Re: [FFmpeg-devel] [PATCH]lavf/decimate: Fix total difference for the first frame

2016-01-04 Thread Carl Eugen Hoyos
Clément Bœsch pkh.me> writes: > +if (cyclestart == 0) { > +vdm->vmi[0].maxbdiff = vdm->vmi[1].maxbdiff; > +vdm->vmi[0].totdiff = vdm->scthresh + 1; > +} > + > I suggest to "cherry-pick" this if it works. Nicolas has already found a very similar solution that only

Re: [FFmpeg-devel] [PATCH] libvpx: Support setting color range for vp9.

2016-01-04 Thread Sasi Inguva
Ping! I could not find any other fields for version other than the IMAGE_ABI_VERSION , CODEC_ABI_VERSION, DECODER_ABI_VERSION which remain unchanged . On Wed, Dec 30, 2015 at 2:21 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Dec 30, 2015 at 3:46 PM, Sasi Inguva

Re: [FFmpeg-devel] [PATCH] Enable dash output to work when the output isn't a local file

2016-01-04 Thread Derek Buitenhuis
On 1/4/2016 6:05 PM, Ganesh Ajjanagadde wrote: > Personally, I think it should be ok to use rename here for now, > especially since even projects like Python had trouble with this > aspect: https://bugs.python.org/issue8828. Someone with greater > Windows expertise can then examine the validity of

Re: [FFmpeg-devel] [PATCH] Enable dash output to work when the output isn't a local file

2016-01-04 Thread Derek Buitenhuis
On 1/4/2016 7:54 PM, Hendrik Leppkes wrote: > rename is used unconditionally in ff_rename as far as I can tell, so it > would probably be OK to remove the unistd.h check in file.c for rename? I would git blame and ask the author if possible first. - Derek

Re: [FFmpeg-devel] [PATCH] Add frame side data when SEI green metadata are detected

2016-01-04 Thread Michael Niedermayer
On Mon, Jan 04, 2016 at 02:50:44PM +0100, Nicolas DEROUINEAU wrote: > --- > libavcodec/h264.c | 21 + > libavcodec/h264.h | 1 + > libavcodec/h264_sei.c | 3 +++ > libavutil/frame.h | 8 this should be in a seperate patch and libavutil/version.h should

Re: [FFmpeg-devel] [PATCH] alsdec: error out if channel number changes

2016-01-04 Thread Derek Buitenhuis
On 1/4/2016 8:18 PM, Thilo Borgmann wrote: > I'm quite sure that changing the number of channels is invalid for ALS > so there is no sense in implementing it. Ah, OK. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 1/2] lavc/pcm_tablegen: slight speedup of table generation

2016-01-04 Thread Clément Bœsch
On Sun, Jan 03, 2016 at 09:49:33PM -0800, Ganesh Ajjanagadde wrote: > On Sun, Jan 3, 2016 at 1:30 PM, Carl Eugen Hoyos wrote: > > Carl Eugen Hoyos ag.or.at> writes: > > > >> Ganesh Ajjanagadde mit.edu> writes: > >> > >> > No one has told me what is interesting > >> > >> Did

Re: [FFmpeg-devel] [PATCH] avformat/concatdec: get bit_rate from internal format

2016-01-04 Thread Nicolas George
> When concatenated files are streamed via HTTP, > opening all of them is not a good idea. Exactly. > If we've known that all files share same codec parameters, > maybe first file's bit_rate would be the best guess. Not at all. Having the same codec parameters, to some extent, is necessary for

Re: [FFmpeg-devel] [PATCH] avformat/concatdec: get bit_rate from internal format

2016-01-04 Thread Zhang Rui
>> av_dump_format() could show something better than 0. > > Yes, but what would be the point? How is it actually useful to know the > average bitrate of a concatenated stream? Actually, I use custom IO in my own application to prefetch data via HTTP. I'm trying to calculate duration from bytes

Re: [FFmpeg-devel] [PATCH 1/2] lavc/pcm_tablegen: slight speedup of table generation

2016-01-04 Thread Michael Niedermayer
On Sun, Jan 03, 2016 at 09:11:28PM -0800, Ganesh Ajjanagadde wrote: > On Sun, Jan 3, 2016 at 7:32 PM, Michael Niedermayer > wrote: > > On Mon, Jan 04, 2016 at 04:04:02AM +0100, Michael Niedermayer wrote: > >> On Wed, Dec 30, 2015 at 08:34:55PM -0800, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH 2/3] vorbisdec: replace get_bits with get_bitsz where n can be 0

2016-01-04 Thread Andreas Cadhalpun
On 04.01.2016 02:31, Michael Niedermayer wrote: > On Sun, Jan 03, 2016 at 11:44:02PM +0100, Andreas Cadhalpun wrote: >> vorbisdec.c |5 + >> 1 file changed, 5 insertions(+) >> cb124c1de2cdbe783b2217af90f082cbc6e0a29f >> 0001-vorbisdec-reject-rangebits-0-with-non-0-partitions.patch >>

[FFmpeg-devel] [PATCH 1/2] brstm: make sure an ADPC chunk was read for adpcm_thp

2016-01-04 Thread Andreas Cadhalpun
This fixes NULL pointer dereferencing. Signed-off-by: Andreas Cadhalpun --- libavformat/brstm.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/brstm.c b/libavformat/brstm.c index bbdbcef..6ec4d89 100644 --- a/libavformat/brstm.c +++

[FFmpeg-devel] [PATCH 2/2] brstm: also allocate b->table in read_packet

2016-01-04 Thread Andreas Cadhalpun
This fixes NULL pointer dereferencing if the codec is forced to adpcm_thp even though a different one was detected. Signed-off-by: Andreas Cadhalpun --- libavformat/brstm.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/brstm.c

Re: [FFmpeg-devel] [PATCH] avformat/concatdec: get bit_rate from internal format

2016-01-04 Thread Nicolas George
Sorry for delaying my reply. Zhang Rui: > --- > libavformat/concatdec.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c > index d21805f..4c7217c 100644 > --- a/libavformat/concatdec.c > +++ b/libavformat/concatdec.c > @@ -310,6 +310,7

[FFmpeg-devel] [PATCH v2] lavf/matroskadec: A_QUICKTIME and AV_CODEC_ID_NONE

2016-01-04 Thread Mats Peterson
In many older QuickTime files, the audio format, or "fourcc", is 0x (AV_CODEC_ID_NONE). The QuickTime File Format Specification states the following regarding this situation: "This format descriptor should not be used, but may be found in some files. Samples are assumed to be stored in

[FFmpeg-devel] [PATCH] parser: error out if the codec doesn't match

2016-01-04 Thread Andreas Cadhalpun
Otherwise this can have some surprising effects (crashes), so let's better not allow it. Signed-off-by: Andreas Cadhalpun --- libavcodec/parser.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/libavcodec/parser.c b/libavcodec/parser.c index

Re: [FFmpeg-devel] [PATCH] parser: error out if the codec doesn't match

2016-01-04 Thread Andreas Cadhalpun
On 05.01.2016 00:51, Ganesh Ajjanagadde wrote: > On Mon, Jan 4, 2016 at 3:44 PM, Andreas Cadhalpun > wrote: >> Otherwise this can have some surprising effects (crashes), so let's >> better not allow it. >> >> Signed-off-by: Andreas Cadhalpun

Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: A_QUICKTIME and AV_CODEC_ID_NONE

2016-01-04 Thread Mats Peterson
On 01/04/2016 10:34 PM, Hendrik Leppkes wrote: On Mon, Jan 4, 2016 at 10:17 PM, Mats Peterson wrote: In many older QuickTime files, the audio format, or "fourcc", is 0x (AV_CODEC_ID_NONE). The QuickTime File Format Specification states the following

Re: [FFmpeg-devel] [PATCH] parser: error out if the codec doesn't match

2016-01-04 Thread Ganesh Ajjanagadde
On Mon, Jan 4, 2016 at 3:44 PM, Andreas Cadhalpun wrote: > Otherwise this can have some surprising effects (crashes), so let's > better not allow it. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/parser.c | 11

Re: [FFmpeg-devel] [RFC v5] libavcodec: add a native Daala decoder

2016-01-04 Thread Rostislav Pehlivanov
>This can still overflow to -256 causing the following line to crash. >A first step towards fixing this would be to replace '2*(*ex) >> 8' >with '*ex >> 7'. That has the same result whenever the first expression >is not undefined and should be faster, as well. ;) >This prevents most of these

Re: [FFmpeg-devel] [PATCH] ffmpeg: replace log2 by faster variant

2016-01-04 Thread Ganesh Ajjanagadde
On Sat, Jan 2, 2016 at 7:59 AM, Ganesh Ajjanagadde wrote: > On Sat, Jan 2, 2016 at 6:24 AM, Michael Niedermayer > wrote: >> On Fri, Jan 01, 2016 at 05:55:31PM -0800, Ganesh Ajjanagadde wrote: >>> On Wed, Dec 30, 2015 at 1:01 AM, Hendrik Leppkes

[FFmpeg-devel] [PATCHv2] avformat/mp3dec, rmdec: check return value of ffio_ensure_seekback

2016-01-04 Thread Ganesh Ajjanagadde
ffio_ensure_seekback can fail due to e.g ENOMEM. This return value is checked here and a diagnostic is logged. All usage of ffio_ensure_seekback in the codebase now has the return value checked. Reviewed-by: wm4 Reviewed-by: Ronald S. Bultje

Re: [FFmpeg-devel] [PATCH] avformat/concatdec: get bit_rate from internal format

2016-01-04 Thread Zhang Rui
2016-01-05 2:12 GMT+08:00 Nicolas George : > Le quintidi 15 nivôse, an CCXXIV, Zhang Rui a écrit : >> Actually, I use custom IO in my own application to prefetch data via HTTP. >> I'm trying to calculate duration from bytes with bit_rate, for: >> 1. Calculate necessary download

Re: [FFmpeg-devel] [PATCH 1/2] lavc/pcm_tablegen: slight speedup of table generation

2016-01-04 Thread Ganesh Ajjanagadde
On Mon, Jan 4, 2016 at 2:45 AM, Michael Niedermayer wrote: > On Sun, Jan 03, 2016 at 09:11:28PM -0800, Ganesh Ajjanagadde wrote: >> On Sun, Jan 3, 2016 at 7:32 PM, Michael Niedermayer >> wrote: >> > On Mon, Jan 04, 2016 at 04:04:02AM +0100, Michael

Re: [FFmpeg-devel] [PATCH 2/2] lavc/pcm_tablegen: always generate tables at runtime.

2016-01-04 Thread Ganesh Ajjanagadde
On Wed, Dec 30, 2015 at 8:34 PM, Ganesh Ajjanagadde wrote: > Previous commit has sped up pcm_tablegen slightly, and table generation > of the alaw and mulaw tables is ~ 20k cycles. Thus, these tables can > always be generated at runtime. > > Tested with/without

[FFmpeg-devel] [PATCHv2] lavc/cbrt_tablegen: speed up tablegen

2016-01-04 Thread Ganesh Ajjanagadde
This exploits an approach based on the sieve of Eratosthenes, a popular method for generating prime numbers. Tables are identical to previous ones. Tested with FATE with/without --enable-hardcoded-tables. Sample benchmark (Haswell, GNU/Linux+gcc): prev: 7860100 decicycles in cbrt_tableinit,

[FFmpeg-devel] [PATCH 3/3] libavcodec/ccaption_dec: rewrite packet handler as case statement; remove COR3 macro

2016-01-04 Thread Aman Gupta
From: Aman Gupta --- libavcodec/ccaption_dec.c | 92 +++ 1 file changed, 53 insertions(+), 39 deletions(-) diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c index 96f0ccf..788e96a 100644 --- a/libavcodec/ccaption_dec.c

[FFmpeg-devel] [PATCH 2/3] libavcodec/ccaption_dec: clean up and standardize white space

2016-01-04 Thread Aman Gupta
From: Aman Gupta --- libavcodec/ccaption_dec.c | 98 ++- 1 file changed, 45 insertions(+), 53 deletions(-) diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c index 4e478e0..96f0ccf 100644 --- a/libavcodec/ccaption_dec.c

[FFmpeg-devel] [PATCH] ffmpeg: check return value of avio_closep for progress report

2016-01-04 Thread Ganesh Ajjanagadde
avio_closep is not guaranteed to succeed, and its return value can contain information regarding failure of preceding writes and silent loss of data (man 2 close, man fclose). Users should know when the progress was not successfully logged, and so a diagnostic is printed here. The reason only one

[FFmpeg-devel] [PATCH 1/3] libavcodec/ccaption_dec: remove unnecessary include

2016-01-04 Thread Aman Gupta
From: Aman Gupta --- libavcodec/ccaption_dec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c index 9f67caa..4e478e0 100644 --- a/libavcodec/ccaption_dec.c +++ b/libavcodec/ccaption_dec.c @@ -21,7 +21,6 @@ #include

[FFmpeg-devel] [PATCH] lavfi/avf_showspectrum: replace pow(x, 0.25) by sqrt(sqrt(x))

2016-01-04 Thread Ganesh Ajjanagadde
This is faster; precision assured as result is a float. Signed-off-by: Ganesh Ajjanagadde --- libavfilter/avf_showspectrum.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/avf_showspectrum.c b/libavfilter/avf_showspectrum.c index

[FFmpeg-devel] [PATCH] lavfi/af_compensationdelay: replace pow(x, 0.5) by sqrt(x)

2016-01-04 Thread Ganesh Ajjanagadde
sqrt is faster, and is sometimes more accurate depending on the libm. Signed-off-by: Ganesh Ajjanagadde --- libavfilter/af_compensationdelay.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/af_compensationdelay.c

Re: [FFmpeg-devel] [PATCH 2/2] brstm: also allocate b->table in read_packet

2016-01-04 Thread Paul B Mahol
On 1/4/16, Andreas Cadhalpun wrote: > This fixes NULL pointer dereferencing if the codec is forced to > adpcm_thp even though a different one was detected. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/brstm.c | 4

Re: [FFmpeg-devel] [PATCH] Adding frame side data about green metadata

2016-01-04 Thread Nicolas Derouineau
Same patch encoded as base64. I hope this time it won't get trashed. Nicolas DEROUINEAU Research Engineer VITEC T. +33 1 46 73 06 06 E. nicolas.derouin...@vitec.com W. www.vitec.com De : ffmpeg-devel de la part de

[FFmpeg-devel] [PATCH 2/2] swscale/arm/yuv2rgb: add ff_yuv422p_to_{argb, rgba, abgr, bgra}_neon_{16, 32}

2016-01-04 Thread Matthieu Bouron
From: Matthieu Bouron --- libswscale/arm/swscale_unscaled.c | 26 ++- libswscale/arm/yuv2rgb_neon.S | 93 --- 2 files changed, 101 insertions(+), 18 deletions(-) diff --git a/libswscale/arm/swscale_unscaled.c

[FFmpeg-devel] [PATCH 1/2] swscale/arm/yuv2rgb: fix typo

2016-01-04 Thread Matthieu Bouron
--- libswscale/arm/yuv2rgb_neon.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libswscale/arm/yuv2rgb_neon.S b/libswscale/arm/yuv2rgb_neon.S index dd00246..d497dd4 100644 --- a/libswscale/arm/yuv2rgb_neon.S +++ b/libswscale/arm/yuv2rgb_neon.S @@ -299,7 +299,7 @@ function

[FFmpeg-devel] [PATCH] Add frame side data when SEI green metadata are detected

2016-01-04 Thread Nicolas DEROUINEAU
--- libavcodec/h264.c | 21 + libavcodec/h264.h | 1 + libavcodec/h264_sei.c | 3 +++ libavutil/frame.h | 8 4 files changed, 33 insertions(+) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 089a86f..e90bcc0 100644 --- a/libavcodec/h264.c +++

Re: [FFmpeg-devel] [PATCH 1/2] swscale/arm/yuv2rgb: fix typo

2016-01-04 Thread Michael Niedermayer
On Mon, Jan 04, 2016 at 02:33:46PM +0100, Matthieu Bouron wrote: > --- > libswscale/arm/yuv2rgb_neon.S | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No snowflake in an avalanche ever feels

Re: [FFmpeg-devel] [PATCH 1/2] lavc/pcm_tablegen: slight speedup of table generation

2016-01-04 Thread Andreas Cadhalpun
Hi Ganesh, On 04.01.2016 06:49, Ganesh Ajjanagadde wrote: > Generally, my strengths are in algorithmic/mathematical/numerical > improvements. I have a strong interest in security (both its > "practical" and "theoretical" variants), but with nowhere near the > same level of knowledge. > >

Re: [FFmpeg-devel] [PATCH 2/2] brstm: also allocate b->table in read_packet

2016-01-04 Thread Andreas Cadhalpun
On 04.01.2016 13:15, Paul B Mahol wrote: > On 1/4/16, Andreas Cadhalpun wrote: >> This fixes NULL pointer dereferencing if the codec is forced to >> adpcm_thp even though a different one was detected. >> >> Signed-off-by: Andreas Cadhalpun

Re: [FFmpeg-devel] [PATCH v2] lavf/mov: Audio and AV_CODEC_ID_NONE

2016-01-04 Thread Ganesh Ajjanagadde
On Mon, Jan 4, 2016 at 9:45 PM, Mats Peterson wrote: > Simplified the patch somewhat. Description follows: > > In many older QuickTime files, the audio format, or "fourcc", is > 0x (AV_CODEC_ID_NONE). The QuickTime File Format Specification > states the

Re: [FFmpeg-devel] [PATCH v2] lavf/mov: Audio and AV_CODEC_ID_NONE

2016-01-04 Thread Ganesh Ajjanagadde
On Mon, Jan 4, 2016 at 10:09 PM, Mats Peterson wrote: > On 01/05/2016 07:05 AM, Ganesh Ajjanagadde wrote: >> >> On Mon, Jan 4, 2016 at 9:45 PM, Mats Peterson >> wrote: >>> >>> Simplified the patch somewhat. Description follows:

Re: [FFmpeg-devel] [PATCH v2] lavf/mov: Audio and AV_CODEC_ID_NONE

2016-01-04 Thread Mats Peterson
On 01/05/2016 07:16 AM, Ganesh Ajjanagadde wrote: Sorry, my gmail webmail does not seem to handle this correctly. Ah, OK! Mats ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2] lavf/mov: Audio and AV_CODEC_ID_NONE

2016-01-04 Thread Mats Peterson
On 01/05/2016 07:05 AM, Ganesh Ajjanagadde wrote: On Mon, Jan 4, 2016 at 9:45 PM, Mats Peterson wrote: Simplified the patch somewhat. Description follows: In many older QuickTime files, the audio format, or "fourcc", is 0x (AV_CODEC_ID_NONE). The

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libstagefright

2016-01-04 Thread Matthieu Bouron
On Sun, Jan 03, 2016 at 01:37:18PM -0500, Derek Buitenhuis wrote: > It serves absolutely no purpose other than to confuse potentional > Android developers about how to use hardware acceleration properly > on the the platform. Both stagefright itself, and MediaCodec, have > avcodec backends

[FFmpeg-devel] [PATCH] alsdec: error out if channel number changes

2016-01-04 Thread Andreas Cadhalpun
Supporting this would require re-initialization to change buffer sizes. This fixes out of bounds reads. Signed-off-by: Andreas Cadhalpun --- libavcodec/alsdec.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavcodec/alsdec.c

[FFmpeg-devel] [PATCH] Enable dash output to work when the output isn't a local file

2016-01-04 Thread rhi
From: Raymond Hilseth Signed-off-by: Raymond Hilseth --- libavformat/dashenc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 4509ee4..378c4e4 100644 --- a/libavformat/dashenc.c +++

Re: [FFmpeg-devel] [libav-devel] [PATCH] Cineform HD Decoder:

2016-01-04 Thread Vittorio Giovara
On Mon, Jan 4, 2016 at 1:04 AM, Kieran Kunhya wrote: > Decodes YUV422P10 samples in AVI and MOV (Gopro Studio) > Older files with more subbands, skips, Bayer not supported > --- > +static av_cold int cfhd_init_decoder(AVCodecContext *avctx) > +{ > +CFHDContext *s =

Re: [FFmpeg-devel] [PATCH] alsdec: error out if channel number changes

2016-01-04 Thread Thilo Borgmann
Am 04.01.16 um 16:18 schrieb Andreas Cadhalpun: > Supporting this would require re-initialization to change buffer sizes. > > This fixes out of bounds reads. Can you upload a sample for this? -Thilo ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 2/2] swscale/arm/yuv2rgb: add ff_yuv422p_to_{argb, rgba, abgr, bgra}_neon_{16, 32}

2016-01-04 Thread Michael Niedermayer
On Mon, Jan 04, 2016 at 02:33:47PM +0100, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > libswscale/arm/swscale_unscaled.c | 26 ++- > libswscale/arm/yuv2rgb_neon.S | 93 > --- > 2 files changed, 101

Re: [FFmpeg-devel] [PATCH 1/2] lavc/pcm_tablegen: slight speedup of table generation

2016-01-04 Thread Ganesh Ajjanagadde
On Mon, Jan 4, 2016 at 1:29 AM, Clément Bœsch wrote: > On Sun, Jan 03, 2016 at 09:49:33PM -0800, Ganesh Ajjanagadde wrote: >> On Sun, Jan 3, 2016 at 1:30 PM, Carl Eugen Hoyos wrote: >> > Carl Eugen Hoyos ag.or.at> writes: >> > >> >> Ganesh Ajjanagadde mit.edu>

Re: [FFmpeg-devel] [PATCH] Cineform HD Decoder:

2016-01-04 Thread Michael Niedermayer
On Mon, Jan 04, 2016 at 12:04:08AM +, Kieran Kunhya wrote: > Decodes YUV422P10 samples in AVI and MOV (Gopro Studio) > Older files with more subbands, skips, Bayer not supported > --- > libavcodec/Makefile | 1 + > libavcodec/allcodecs.c | 1 + > libavcodec/avcodec.h| 1 + >

Re: [FFmpeg-devel] [PATCH]lavf/decimate: Fix total difference for the first frame

2016-01-04 Thread Derek Buitenhuis
On 1/4/2016 5:17 PM, Carl Eugen Hoyos wrote: > I wish I would understand;-( > > Since this fixes a reported issue here, I will > commit my patch if nobody objects. [17:30] * Daemon404 pokes ubitux [17:30] <@ubitux> yup? [17:30] <@Daemon404> do you have a comment on carl's decimate change

Re: [FFmpeg-devel] [PATCH] alsdec: error out if channel number changes

2016-01-04 Thread Derek Buitenhuis
On 1/4/2016 3:18 PM, Andreas Cadhalpun wrote: > Supporting this would require re-initialization to change buffer sizes. I may be mistaken, but don't we already support some codecs which do this, properly? - Derek ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] Enable dash output to work when the output isn't a local file

2016-01-04 Thread Derek Buitenhuis
On 1/4/2016 4:46 PM, Ganesh Ajjanagadde wrote: > Yes, this is strange. url_move points to file_move, and the only > system functionality it relies on is rename, available in stdio.h. The > semantics vary from system to system, with some details specified in > POSIX, but rename itself is standard

Re: [FFmpeg-devel] [PATCH]lavf/decimate: Fix total difference for the first frame

2016-01-04 Thread Carl Eugen Hoyos
Derek Buitenhuis gmail.com> writes: > > -dm->queue[dm->fid].totdiff = INT64_MAX; > > +dm->queue[dm->fid].totdiff = dm->scthresh; > > Upstream uses +1 here, but I'm not sure why I wish I would understand;-( Since this fixes a reported issue here, I will commit my patch if

Re: [FFmpeg-devel] [PATCH] Adding frame side data about green metadata

2016-01-04 Thread wm4
On Mon, 4 Jan 2016 13:33:02 + Nicolas Derouineau wrote: > Same patch encoded as base64. I hope this time it won't get trashed. > Well, now most likely nobody can comfortably read it. Does your mail client not allow to simply attach text files?

Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: A_QUICKTIME and AV_CODEC_ID_NONE

2016-01-04 Thread Hendrik Leppkes
On Mon, Jan 4, 2016 at 10:17 PM, Mats Peterson wrote: > In many older QuickTime files, the audio format, or "fourcc", is > 0x (AV_CODEC_ID_NONE). The QuickTime File Format Specification > states the following regarding this situation: > > "This format

Re: [FFmpeg-devel] [PATCH 1/2] swscale/arm/yuv2rgb: fix typo

2016-01-04 Thread Matthieu Bouron
On Mon, Jan 04, 2016 at 03:12:13PM +0100, Michael Niedermayer wrote: > On Mon, Jan 04, 2016 at 02:33:46PM +0100, Matthieu Bouron wrote: > > --- > > libswscale/arm/yuv2rgb_neon.S | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > LGTM Pushed, thanks. [...]

Re: [FFmpeg-devel] [PATCH] alsdec: error out if channel number changes

2016-01-04 Thread Andreas Cadhalpun
On 04.01.2016 16:29, Thilo Borgmann wrote: > Am 04.01.16 um 16:18 schrieb Andreas Cadhalpun: >> Supporting this would require re-initialization to change buffer sizes. >> >> This fixes out of bounds reads. > > Can you upload a sample for this? Unfortunately (or fortunately?) the sample doesn't

Re: [FFmpeg-devel] [PATCH] Enable dash output to work when the output isn't a local file

2016-01-04 Thread Hendrik Leppkes
On Mon, Jan 4, 2016 at 3:58 PM, wrote: > From: Raymond Hilseth > > Signed-off-by: Raymond Hilseth > --- > libavformat/dashenc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavformat/dashenc.c

Re: [FFmpeg-devel] [PATCH] Enable dash output to work when the output isn't a local file

2016-01-04 Thread Ganesh Ajjanagadde
On Mon, Jan 4, 2016 at 8:36 AM, Hendrik Leppkes wrote: > On Mon, Jan 4, 2016 at 3:58 PM, wrote: >> From: Raymond Hilseth >> >> Signed-off-by: Raymond Hilseth >> --- >> libavformat/dashenc.c | 4 ++-- >> 1 file changed, 2

Re: [FFmpeg-devel] [PATCH] avformat/concatdec: get bit_rate from internal format

2016-01-04 Thread Nicolas George
Le quintidi 15 nivôse, an CCXXIV, Zhang Rui a écrit : > Actually, I use custom IO in my own application to prefetch data via HTTP. > I'm trying to calculate duration from bytes with bit_rate, for: > 1. Calculate necessary download speed. > 2. Adjust prefetch buffer size. > 3. Show prefetched

Re: [FFmpeg-devel] [PATCH]lavf/decimate: Fix total difference for the first frame

2016-01-04 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > -dm->queue[dm->fid].totdiff = INT64_MAX; > +dm->queue[dm->fid].totdiff = dm->scthresh; This is not correct, I will try to find the actual issue. Sorry, Carl Eugen ___ ffmpeg-devel