[FFmpeg-devel] [PATCH 2/3] x86/vf_blend: make all functions work on x86_32

Signed-off-by: James Almer --- libavfilter/x86/vf_blend.asm| 103 +++- libavfilter/x86/vf_blend_init.c | 4 +- 2 files changed, 52 insertions(+), 55 deletions(-) diff --git a/libavfilter/x86/vf_blend.asm b/libavfilter/x86/vf_blend.asm index d079b79..5f02

[FFmpeg-devel] [PATCH 3/3] x86/vf_blend: add sse2 versions of blend_difference and blend_negation

Signed-off-by: James Almer --- libavfilter/x86/vf_blend.asm| 12 +--- libavfilter/x86/vf_blend_init.c | 4 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/libavfilter/x86/vf_blend.asm b/libavfilter/x86/vf_blend.asm index 5f0271a..730be77 100644 --- a/libavfilter/

[FFmpeg-devel] [PATCH 1/3] x86/vf_blend: simplify using macros

Signed-off-by: James Almer --- libavfilter/x86/vf_blend.asm| 276 +--- libavfilter/x86/vf_blend_init.c | 102 +++ 2 files changed, 53 insertions(+), 325 deletions(-) diff --git a/libavfilter/x86/vf_blend.asm b/libavfilter/x86/vf_blend.asm index

Re: [FFmpeg-devel] [PATCH] lavu: add pthread asserts if ASSERT_LEVEL>1

On Wed, Dec 23, 2015 at 7:59 PM, Ganesh Ajjanagadde wrote: > On Tue, Dec 22, 2015 at 12:05 PM, Clément Bœsch wrote: >> --- >> libavutil/thread.h | 96 >> ++ >> 1 file changed, 96 insertions(+) >> >> diff --git a/libavutil/thread.h b/libavutil/

Re: [FFmpeg-devel] [PATCH] lavu: add pthread asserts if ASSERT_LEVEL>1

On Tue, Dec 22, 2015 at 12:05 PM, Clément Bœsch wrote: > --- > libavutil/thread.h | 96 > ++ > 1 file changed, 96 insertions(+) > > diff --git a/libavutil/thread.h b/libavutil/thread.h > index 3d15737..0bb745e 100644 > --- a/libavutil/thread.h

Re: [FFmpeg-devel] [PATCH] fate: replace custom md5 tests with those from RFC 1321

On 11/3/2015 2:18 PM, James Almer wrote: > On 11/3/2015 4:44 AM, Clément Bœsch wrote: >> On Tue, Nov 03, 2015 at 04:09:43AM -0300, James Almer wrote: >>> Signed-off-by: James Almer >>> --- >>> libavutil/md5.c| 23 ++- >>> tests/ref/fate/md5 | 12 +++- >>> 2 files c

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: add flag to embed an AC-3/E-AC-3 ES the DVB way

On Sun, Dec 13, 2015 at 11:54:32AM +0100, Stefan Pöschel wrote: > So far an AC-3 elementary stream is refered to in the PMT according to > System A (ATSC). An E-AC-3 ES in contrast is embedded the System B (DVB) way. > To fix this inconsistency, this commit changes the default E-AC-3 behaviour to >

Re: [FFmpeg-devel] [PATCH] swr/resample: use av_clip_int16 instead of av_clip

On Wed, Dec 23, 2015 at 02:50:33PM -0800, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libswresample/resample.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many

Re: [FFmpeg-devel] [PATCH 00/15] replace pow(10, x) by exp10(x) across FFmpeg

On Wed, Dec 23, 2015 at 4:39 PM, Michael Niedermayer wrote: > On Wed, Dec 23, 2015 at 03:52:50PM -0800, Ganesh Ajjanagadde wrote: >> On Wed, Dec 23, 2015 at 3:38 PM, Michael Niedermayer >> wrote: >> > On Wed, Dec 23, 2015 at 10:47:20AM -0800, Ganesh Ajjanagadde wrote: >> >> exp10(x) is superior t

Re: [FFmpeg-devel] [PATCH 00/15] replace pow(10, x) by exp10(x) across FFmpeg

On Wed, Dec 23, 2015 at 03:52:50PM -0800, Ganesh Ajjanagadde wrote: > On Wed, Dec 23, 2015 at 3:38 PM, Michael Niedermayer > wrote: > > On Wed, Dec 23, 2015 at 10:47:20AM -0800, Ganesh Ajjanagadde wrote: > >> exp10(x) is superior to pow(10,x). > >> Note that in some cases, this may affect integers

Re: [FFmpeg-devel] [PATCH 00/15] replace pow(10, x) by exp10(x) across FFmpeg

On Wed, Dec 23, 2015 at 3:38 PM, Michael Niedermayer wrote: > On Wed, Dec 23, 2015 at 10:47:20AM -0800, Ganesh Ajjanagadde wrote: >> exp10(x) is superior to pow(10,x). >> Note that in some cases, this may affect integers derived from pow calls. >> When >> spotted, this remark has been added to th

[FFmpeg-devel] [PATCH] x86/vf_maskedmerge: make ff_maskedmerge8_sse2 work on x86_32

Signed-off-by: James Almer --- libavfilter/x86/vf_maskedmerge.asm| 29 ++--- libavfilter/x86/vf_maskedmerge_init.c | 2 +- 2 files changed, 19 insertions(+), 12 deletions(-) diff --git a/libavfilter/x86/vf_maskedmerge.asm b/libavfilter/x86/vf_maskedmerge.asm index 1

Re: [FFmpeg-devel] [PATCH 00/15] replace pow(10, x) by exp10(x) across FFmpeg

On Wed, Dec 23, 2015 at 10:47:20AM -0800, Ganesh Ajjanagadde wrote: > exp10(x) is superior to pow(10,x). > Note that in some cases, this may affect integers derived from pow calls. When > spotted, this remark has been added to the relevant commit message. > > Note that if such a thing is troubleso

Re: [FFmpeg-devel] [PATCH] Changelog: add note on dynaudnorm availability on all platforms

Am 23.12.2015 19:47 schrieb "Ganesh Ajjanagadde" : > > On Wed, Dec 23, 2015 at 10:02 AM, Hendrik Leppkes wrote: > > Am 23.12.2015 18:30 schrieb "Ganesh Ajjanagadde" : > >> > >> Commits 062e3e23824ba3d25594ea966d7834bcf34db49b and > >> dd68cde28a44bbf5307d29ee6cb8ebd14985dea5 added support for erf

[FFmpeg-devel] [PATCH] swr/resample: use av_clip_int16 instead of av_clip

Signed-off-by: Ganesh Ajjanagadde --- libswresample/resample.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libswresample/resample.c b/libswresample/resample.c index 6c83eef..7e3bda5 100644 --- a/libswresample/resample.c +++ b/libswresample/resample.c @@ -200,7 +200,7 @

Re: [FFmpeg-devel] [PATCH 12/15] lavc/opus: replace pow(10, x) by exp10(x)

On Wed, Dec 23, 2015 at 12:40 PM, Rostislav Pehlivanov wrote: > On Wed, 2015-12-23 at 10:47 -0800, Ganesh Ajjanagadde wrote: >> exp10, introduced recently, is superior for the purpose. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/opus.c | 3 ++- >> 1 file changed, 2 insertions(+)

Re: [FFmpeg-devel] [PATCH 12/15] lavc/opus: replace pow(10, x) by exp10(x)

On Wed, 2015-12-23 at 10:47 -0800, Ganesh Ajjanagadde wrote: > exp10, introduced recently, is superior for the purpose. > > Signed-off-by: Ganesh Ajjanagadde > --- >  libavcodec/opus.c | 3 ++- >  1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/opus.c b/libavcodec/opus.

Re: [FFmpeg-devel] [PATCH 06/15] lavc/aacpsy: replace pow(10, x) by exp10(x)

On Wed, 2015-12-23 at 10:47 -0800, Ganesh Ajjanagadde wrote: > exp10, recently introduced, is superior for the purpose. > > Signed-off-by: Ganesh Ajjanagadde > --- >  libavcodec/aacpsy.c | 8 >  1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/aacpsy.c b/libavc

[FFmpeg-devel] [PATCH] lavc/acelp_pitch_delay: replace exp2f(M_LOG2_10 *x) by exp10f(x)

Suggested-by: Ronald S. Bultje Signed-off-by: Ganesh Ajjanagadde --- libavcodec/acelp_pitch_delay.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/acelp_pitch_delay.c b/libavcodec/acelp_pitch_delay.c index 3ecec01..95272d9 100644 --- a/libavcodec/acelp_pitch_delay

Re: [FFmpeg-devel] [PATCH 01/15] lavu/eval: replace pow(10, x) by exp10

On Wed, Dec 23, 2015 at 11:05 AM, Ronald S. Bultje wrote: > Hi, > > On Wed, Dec 23, 2015 at 1:47 PM, Ganesh Ajjanagadde > wrote: >> >> exp10, recently introduced, is superior for the purpose. > > > Code change itself is fine. But I find this extra line in the commit message > somewhat redundant.

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 12/23/2015 07:44 PM, Lou Logan wrote: On Wed, Dec 23, 2015, at 03:05 AM, Mats Peterson wrote: OK Clement, I have changed some of the stuff you complained about. Those were not complaints, but a comments from a review. Here's a real complaint: I find it increasingly hard to follow your stre

Re: [FFmpeg-devel] [PATCH 03/15] lavfi/f_ebur128: replace pow(10, x) by exp10(x)

On Wed, Dec 23, 2015 at 11:08 AM, Ronald S. Bultje wrote: > Hi, > > On Wed, Dec 23, 2015 at 2:00 PM, Ganesh Ajjanagadde > wrote: > >> On Wed, Dec 23, 2015 at 10:54 AM, Clément Bœsch wrote: >> > On Wed, Dec 23, 2015 at 10:47:23AM -0800, Ganesh Ajjanagadde wrote: >> >> exp10, recently introduced,

Re: [FFmpeg-devel] [PATCH 03/15] lavfi/f_ebur128: replace pow(10, x) by exp10(x)

Hi, On Wed, Dec 23, 2015 at 2:00 PM, Ganesh Ajjanagadde wrote: > On Wed, Dec 23, 2015 at 10:54 AM, Clément Bœsch wrote: > > On Wed, Dec 23, 2015 at 10:47:23AM -0800, Ganesh Ajjanagadde wrote: > >> exp10, recently introduced, is superior for the purpose. > >> > >> Signed-off-by: Ganesh Ajjanagad

Re: [FFmpeg-devel] [PATCH 01/15] lavu/eval: replace pow(10, x) by exp10

Hi, On Wed, Dec 23, 2015 at 1:47 PM, Ganesh Ajjanagadde wrote: > exp10, recently introduced, is superior for the purpose. Code change itself is fine. But I find this extra line in the commit message somewhat redundant. :) Ronald ___ ffmpeg-devel mai

Re: [FFmpeg-devel] [PATCH 03/15] lavfi/f_ebur128: replace pow(10, x) by exp10(x)

On Wed, Dec 23, 2015 at 10:54 AM, Clément Bœsch wrote: > On Wed, Dec 23, 2015 at 10:47:23AM -0800, Ganesh Ajjanagadde wrote: >> exp10, recently introduced, is superior for the purpose. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/f_ebur128.c | 2 +- >> 1 file changed, 1 insertio

[FFmpeg-devel] [PATCH 05/15] lavfi/af_compand: replace pow(10, x) by exp10(x)

exp10, recently introduced, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavfilter/af_compand.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/af_compand.c b/libavfilter/af_compand.c index 68b1fae..9c18da6 100644 --- a/libavfilter/af_compa

Re: [FFmpeg-devel] [PATCH 03/15] lavfi/f_ebur128: replace pow(10, x) by exp10(x)

On Wed, Dec 23, 2015 at 10:47:23AM -0800, Ganesh Ajjanagadde wrote: > exp10, recently introduced, is superior for the purpose. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/f_ebur128.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/f_ebur128.c b

[FFmpeg-devel] [PATCH 12/15] lavc/opus: replace pow(10, x) by exp10(x)

exp10, introduced recently, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/opus.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/opus.c b/libavcodec/opus.c index 6b3d3c3..cac9f85 100644 --- a/libavcodec/opus.c +++ b/libavcodec/opus

[FFmpeg-devel] [PATCH 15/15] lavc/wmaprodec: replace pow(10, x) by exp10(x)

exp10, introduced recently, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/wmaprodec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/wmaprodec.c b/libavcodec/wmaprodec.c index f8f901c..cbda9e9 100644 --- a/libavcodec/wmaprodec.c +

[FFmpeg-devel] [PATCH 14/15] lavc/wmaenc: replace pow(10, x) by exp10(x)

exp10, introduced recently, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/wmaenc.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/wmaenc.c b/libavcodec/wmaenc.c index faf0cb5..c2d2236 100644 --- a/libavcodec/wmaenc.c +++ b/liba

[FFmpeg-devel] [PATCH 13/15] lavc/wmadec: replace pow(10, x) by exp10(x)

exp10, introduced recently, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/wmadec.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/wmadec.c b/libavcodec/wmadec.c index e814007..b48cac7 100644 --- a/libavcodec/wmadec.c +++ b/liba

[FFmpeg-devel] [PATCH 09/15] lavc/imc: replace pow(10, x) by exp10(x)

exp10, introduced recently, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/imc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/imc.c b/libavcodec/imc.c index 14f9fa3..2335ae1e 100644 --- a/libavcodec/imc.c +++ b/libavcodec/imc.c

[FFmpeg-devel] [PATCH 10/15] lavc/libopusdec: replace pow(10, x) by exp10(x)

exp10, introduced recently, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/libopusdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/libopusdec.c b/libavcodec/libopusdec.c index 4cbd14a..4c663fe 100644 --- a/libavcodec/libopusdec

[FFmpeg-devel] [PATCH 08/15] lavc/dcaenc: replace pow(10, x) by exp10(x)

exp10, recently introduced, is superior for the purpose. We may want to change the exp10 fallback to pow(10,) based; value may change slightly here. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/dcaenc.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/dcae

[FFmpeg-devel] [PATCH 11/15] lavc/on2avc: replace pow(10, x) by exp10(x)

exp10, introduced recently, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/on2avc.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/on2avc.c b/libavcodec/on2avc.c index 04c8e41..0409b3e 100644 --- a/libavcodec/on2avc.c +++ b/liba

[FFmpeg-devel] [PATCH 07/15] lavc/cngdec: replace pow(10, x) by exp10(x)

exp10, recently introduced, is superior for the purpose. We may want to change the exp10 fallback to pow(10,) based; value may change slightly here. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/cngdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/cngdec.

[FFmpeg-devel] [PATCH 04/15] lavfi/af_volume: replace pow(10, x) by exp10(x)

exp10, recently introduced, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavfilter/af_volume.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/af_volume.c b/libavfilter/af_volume.c index 3ba356a..6c4528a 100644 --- a/libavfilter/af_volume.c

[FFmpeg-devel] [PATCH 06/15] lavc/aacpsy: replace pow(10, x) by exp10(x)

exp10, recently introduced, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/aacpsy.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/aacpsy.c b/libavcodec/aacpsy.c index 71eeb3e..163ee43 100644 --- a/libavcodec/aacpsy.c +++ b/l

[FFmpeg-devel] [PATCH 02/15] lavfi/vsrc_testsrc: replace pow(10, x) by exp10(x)

exp10, recently introduced, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavfilter/vsrc_testsrc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/vsrc_testsrc.c b/libavfilter/vsrc_testsrc.c index 1fca3e7..0eff642 100644 --- a/libavfilter/vs

[FFmpeg-devel] [PATCH 00/15] replace pow(10, x) by exp10(x) across FFmpeg

exp10(x) is superior to pow(10,x). Note that in some cases, this may affect integers derived from pow calls. When spotted, this remark has been added to the relevant commit message. Note that if such a thing is troublesome, one can do one of two things: 1. leave the pow(10,x) call as is in such pl

[FFmpeg-devel] [PATCH 01/15] lavu/eval: replace pow(10, x) by exp10

exp10, recently introduced, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavutil/eval.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/eval.c b/libavutil/eval.c index dcb8c56..936e6e6 100644 --- a/libavutil/eval.c +++ b/libavutil/eval.c @@ -

[FFmpeg-devel] [PATCH 03/15] lavfi/f_ebur128: replace pow(10, x) by exp10(x)

exp10, recently introduced, is superior for the purpose. Signed-off-by: Ganesh Ajjanagadde --- libavfilter/f_ebur128.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/f_ebur128.c b/libavfilter/f_ebur128.c index b9577c5..6749bcd 100644 --- a/libavfilter/f_ebur128.c

Re: [FFmpeg-devel] [PATCH] Changelog: add note on dynaudnorm availability on all platforms

On Wed, Dec 23, 2015 at 10:02 AM, Hendrik Leppkes wrote: > Am 23.12.2015 18:30 schrieb "Ganesh Ajjanagadde" : >> >> Commits 062e3e23824ba3d25594ea966d7834bcf34db49b and >> dd68cde28a44bbf5307d29ee6cb8ebd14985dea5 added support for erf and >> copysign on broken libms, and so dynaudnorm is now avail

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On Wed, Dec 23, 2015, at 03:05 AM, Mats Peterson wrote: > OK Clement, I have changed some of the stuff you complained about. Those were not complaints, but a comments from a review. Here's a real complaint: I find it increasingly hard to follow your stream-of-consciousness style of replies. You

Re: [FFmpeg-devel] [PATCH] avfilter: add audio parametric multi band equalizer filter

On Wed, Dec 23, 2015 at 10:05 AM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > > I'm happy with feature set so I will apply this soon. > > --- > configure| 1 + > doc/filters.texi | 76 + > libavfilter/Makefile | 1 + > libavfilter/af

[FFmpeg-devel] [PATCH] avfilter: add audio parametric multi band equalizer filter

Signed-off-by: Paul B Mahol --- I'm happy with feature set so I will apply this soon. --- configure| 1 + doc/filters.texi | 76 + libavfilter/Makefile | 1 + libavfilter/af_anequalizer.c | 679 +++ libavfi

Re: [FFmpeg-devel] [PATCH] Changelog: add note on dynaudnorm availability on all platforms

On 12/23/15, Hendrik Leppkes wrote: > Am 23.12.2015 18:30 schrieb "Ganesh Ajjanagadde" : >> >> Commits 062e3e23824ba3d25594ea966d7834bcf34db49b and >> dd68cde28a44bbf5307d29ee6cb8ebd14985dea5 added support for erf and >> copysign on broken libms, and so dynaudnorm is now available on all >> platfo

Re: [FFmpeg-devel] [PATCH] Changelog: add note on dynaudnorm availability on all platforms

Am 23.12.2015 18:30 schrieb "Ganesh Ajjanagadde" : > > Commits 062e3e23824ba3d25594ea966d7834bcf34db49b and > dd68cde28a44bbf5307d29ee6cb8ebd14985dea5 added support for erf and > copysign on broken libms, and so dynaudnorm is now available on all > platforms. > > This is user facing; hence the Chan

[FFmpeg-devel] [PATCH] ffprobe: avoid unnecessary pow and exp2 calls

These are just for prefixes and may be hardcoded easily; see lavu/eval for this approach. Signed-off-by: Ganesh Ajjanagadde --- ffprobe.c | 27 +++ 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/ffprobe.c b/ffprobe.c index 0b672ff..c352b44 100644 --- a/ffp

[FFmpeg-devel] [PATCH] Changelog: add note on dynaudnorm availability on all platforms

Commits 062e3e23824ba3d25594ea966d7834bcf34db49b and dd68cde28a44bbf5307d29ee6cb8ebd14985dea5 added support for erf and copysign on broken libms, and so dynaudnorm is now available on all platforms. This is user facing; hence the Changelog entry. Signed-off-by: Ganesh Ajjanagadde --- Changelog

Re: [FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

On Wed, Dec 23, 2015 at 8:04 AM, Ganesh Ajjanagadde wrote: > On Tue, Dec 22, 2015 at 11:32 PM, Paul B Mahol wrote: >> On 12/22/15, Ganesh Ajjanagadde wrote: >>> On Tue, Dec 22, 2015 at 2:10 PM, Paul B Mahol wrote: On 12/22/15, Ganesh Ajjanagadde wrote: > On Tue, Dec 22, 2015 at 12:46

Re: [FFmpeg-devel] [PATCH] lavu/libm: add exp10 support

On Tue, Dec 22, 2015 at 4:31 PM, Ganesh Ajjanagadde wrote: > On Tue, Dec 22, 2015 at 4:17 PM, Hendrik Leppkes wrote: >> On Wed, Dec 23, 2015 at 12:29 AM, Ganesh Ajjanagadde >> wrote: >>> exp10 is a function available in GNU libm. Looks like no other common >>> libm has it. This adds support for

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 12/23/2015 05:26 PM, Mats Peterson wrote: On 12/23/2015 05:24 PM, Mats Peterson wrote: On 12/23/2015 05:17 PM, Alexander Strasser wrote: On 2015-12-23 13:33 +0100, Hendrik Leppkes wrote: Am 23.12.2015 13:28 schrieb "Mats Peterson" < matsp888-at-yahoo@ffmpeg.org>: On 12/23/2015 01:15 P

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 12/23/2015 05:24 PM, Mats Peterson wrote: On 12/23/2015 05:17 PM, Alexander Strasser wrote: On 2015-12-23 13:33 +0100, Hendrik Leppkes wrote: Am 23.12.2015 13:28 schrieb "Mats Peterson" < matsp888-at-yahoo@ffmpeg.org>: On 12/23/2015 01:15 PM, Mats Peterson wrote: On 12/23/2015 01:13

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 12/23/2015 05:17 PM, Alexander Strasser wrote: On 2015-12-23 13:33 +0100, Hendrik Leppkes wrote: Am 23.12.2015 13:28 schrieb "Mats Peterson" < matsp888-at-yahoo@ffmpeg.org>: On 12/23/2015 01:15 PM, Mats Peterson wrote: On 12/23/2015 01:13 PM, Carl Eugen Hoyos wrote: Mats Peterson f

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 2015-12-23 13:33 +0100, Hendrik Leppkes wrote: > Am 23.12.2015 13:28 schrieb "Mats Peterson" < > matsp888-at-yahoo@ffmpeg.org>: > > > > On 12/23/2015 01:15 PM, Mats Peterson wrote: > >> > >> On 12/23/2015 01:13 PM, Carl Eugen Hoyos wrote: > >>> > >>> Mats Peterson ffmpeg.org> writes: > >>>

Re: [FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

On Tue, Dec 22, 2015 at 11:32 PM, Paul B Mahol wrote: > On 12/22/15, Ganesh Ajjanagadde wrote: >> On Tue, Dec 22, 2015 at 2:10 PM, Paul B Mahol wrote: >>> On 12/22/15, Ganesh Ajjanagadde wrote: On Tue, Dec 22, 2015 at 12:46 AM, Paul B Mahol wrote: > On 12/22/15, Ganesh Ajjanagadde wr

Re: [FFmpeg-devel] [PATCH v6] lavf: palettized QuickTime video in Matroska

On 12/23/2015 03:37 PM, Mats Peterson wrote: I mistakenly used 'extradata' rather than 'st->codec->extradata', naturally leading to a segfault. Here's an updated patch. An explanation of the patch follows: Palettized QuickTime video in Matroska has hitherto not been recognized whatsoever, and t

[FFmpeg-devel] [PATCH v6] lavf: palettized QuickTime video in Matroska

I mistakenly used 'extradata' rather than 'st->codec->extradata', naturally leading to a segfault. Here's an updated patch. An explanation of the patch follows: Palettized QuickTime video in Matroska has hitherto not been recognized whatsoever, and the "palette" used has been completely random

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 12/23/2015 01:37 PM, Mats Peterson wrote: On 12/23/2015 01:33 PM, Hendrik Leppkes wrote: Am 23.12.2015 13:28 schrieb "Mats Peterson" < matsp888-at-yahoo@ffmpeg.org>: On 12/23/2015 01:15 PM, Mats Peterson wrote: On 12/23/2015 01:13 PM, Carl Eugen Hoyos wrote: Mats Peterson ffmpeg.or

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

On Wed, Dec 23, 2015 at 12:43:48PM +, Nicolas Derouineau wrote: > > Sorry if I missed something: > > What is green metadata? > > Greenmetadata is a set of metrics specified in ISO/IEC 23001-11. > > They are used to predict the decoding complexity (in terms of cycles) of AVC > frames. It is u

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

On Wed, Dec 23, 2015 at 12:28 PM, Nicolas Derouineau wrote: > Hello, > Please find here enclosed a patch adding frame side data information about > green metadata. The logic is ok, but the introduction of av_greenmetadata_set is a no-go for me. This function is exposed to every user (with a non-

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

> Sorry if I missed something: > What is green metadata? Greenmetadata is a set of metrics specified in ISO/IEC 23001-11. They are used to predict the decoding complexity (in terms of cycles) of AVC frames. It is useful to perform DVFS (ie: CPU frequency scaling) in order to reduce decoder powe

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 12/23/2015 01:33 PM, Hendrik Leppkes wrote: Am 23.12.2015 13:28 schrieb "Mats Peterson" < matsp888-at-yahoo@ffmpeg.org>: On 12/23/2015 01:15 PM, Mats Peterson wrote: On 12/23/2015 01:13 PM, Carl Eugen Hoyos wrote: Mats Peterson ffmpeg.org> writes: In matroskadec.c, I'm also puttin

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

Am 23.12.2015 13:28 schrieb "Mats Peterson" < matsp888-at-yahoo@ffmpeg.org>: > > On 12/23/2015 01:15 PM, Mats Peterson wrote: >> >> On 12/23/2015 01:13 PM, Carl Eugen Hoyos wrote: >>> >>> Mats Peterson ffmpeg.org> writes: >>> >> In matroskadec.c, I'm also putting the palette in 'extradata'

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 12/23/2015 01:15 PM, Mats Peterson wrote: On 12/23/2015 01:13 PM, Carl Eugen Hoyos wrote: Mats Peterson ffmpeg.org> writes: In matroskadec.c, I'm also putting the palette in 'extradata', like it's done for V_MS/VFW/FOURCC; this is a requirement in order for MPlayer to recognize the palette

Re: [FFmpeg-devel] can't build the ffmpeg for iOS using the newest gas-preprocessor.pl

On Wed, Dec 23, 2015 at 07:30:05PM +0800, zhuhb wrote: > > > aarch64 2.7.2+ build on osx should work with the options listed at: > > > ple-clang-n2.7> > http://fate.ffmpeg.org/report.cgi?time=20151221193018&slot=ar

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 12/23/2015 01:13 PM, Carl Eugen Hoyos wrote: Mats Peterson ffmpeg.org> writes: In matroskadec.c, I'm also putting the palette in 'extradata', like it's done for V_MS/VFW/FOURCC; this is a requirement in order for MPlayer to recognize the palette. This is wrong, please remove this hunk.

Re: [FFmpeg-devel] Merge commit 'b9ece15a01782b4f301c0c139d1d7b20f848914c'

Hendrik Leppkes gmail.com> writes: > > $ ffmpeg -f lavfi -i color -f yuv4mpegpipe -vcodec rawvideo /dev/null > Just skip the vcodec entirely and it'll work magically Thank you for this useful information, I would never have guessed! > and faster as well - which is the reason for the change,

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

Mats Peterson ffmpeg.org> writes: > >> In matroskadec.c, I'm also putting the palette in 'extradata', > >> like it's done for V_MS/VFW/FOURCC; this is a requirement in > >> order for MPlayer to recognize the palette. > > > > This is wrong, please remove this hunk. > The reason for this is becaus

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 12/23/2015 01:07 PM, Carl Eugen Hoyos wrote: Mats Peterson ffmpeg.org> writes: In matroskadec.c, I'm also putting the palette in 'extradata', like it's done for V_MS/VFW/FOURCC; this is a requirement in order for MPlayer to recognize the palette. This is wrong, please remove this hunk. T

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

Mats Peterson ffmpeg.org> writes: > In matroskadec.c, I'm also putting the palette in 'extradata', > like it's done for V_MS/VFW/FOURCC; this is a requirement in > order for MPlayer to recognize the palette. This is wrong, please remove this hunk. The reason it is needed for AVI is not MPlaye

[FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

OK Clement, I have changed some of the stuff you complained about. Regarding my being the sole author, well, the *idea* has been there before in the shape of that patch by, they say, Martin Storsjö (which he doesn't admit to be behind, I've talked to him). I don't know how his name came up what

Re: [FFmpeg-devel] Merge commit 'b9ece15a01782b4f301c0c139d1d7b20f848914c'

On Wed, Dec 23, 2015 at 12:47 PM, Carl Eugen Hoyos wrote: > Nicolas George nsup.org> writes: > >> >> Le tridi 3 nivôse, an CCXXIV, Carl Eugen Hoyos a écrit : >> > yuv4mpegpipe used to accept (only) rawvideo as codec. >> > Bug reports indicate that changing this (without any >> > explanation) brea

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: add flag to embed an AC-3/E-AC-3 ES the DVB way

Just wanted to ask if there is any further information needed regarding my patch. > Gesendet: Sonntag, 13. Dezember 2015 um 11:54 Uhr > Von: "Stefan Pöschel" > An: ffmpeg-devel@ffmpeg.org > Betreff: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: add flag to embed an > AC-3/E-AC-3 ES the DVB way > >

Re: [FFmpeg-devel] Merge commit 'b9ece15a01782b4f301c0c139d1d7b20f848914c'

Nicolas George nsup.org> writes: > > Le tridi 3 nivôse, an CCXXIV, Carl Eugen Hoyos a écrit : > > yuv4mpegpipe used to accept (only) rawvideo as codec. > > Bug reports indicate that changing this (without any > > explanation) breaks usage of the muxer. > > Seems to work fine here. $ ffmpeg -

Re: [FFmpeg-devel] Merge commit 'b9ece15a01782b4f301c0c139d1d7b20f848914c'

Le tridi 3 nivôse, an CCXXIV, Carl Eugen Hoyos a écrit : > yuv4mpegpipe used to accept (only) rawvideo as codec. > Bug reports indicate that changing this (without any > explanation) breaks usage of the muxer. Seems to work fine here. "Complete command line and console output missing." ;-Þ Rega

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

Nicolas Derouineau vitec.com> writes: > Please find here enclosed a patch adding frame > side data information about green metadata. Sorry if I missed something: What is green metadata? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.or

Re: [FFmpeg-devel] Merge commit 'b9ece15a01782b4f301c0c139d1d7b20f848914c'

Nicolas George nsup.org> writes: > Le tridi 3 nivôse, an CCXXIV, Carl Eugen Hoyos a écrit : > >> Unfortunately, from user's perspective this > >> introduces a regression: Could somebody explain > >> the advantages of this commit that justify the > >> pain? > > Ping. > > What are you talking a

Re: [FFmpeg-devel] [PATCH] avcodec/s302menc: set supported channel layouts by codec

On 22 December 2015 at 20:09, Paul B Mahol wrote: > On 12/22/15, Kieran Kunhya wrote: >> On 20 December 2015 at 00:14, Michael Niedermayer >> wrote: >>> On Sat, Dec 19, 2015 at 09:35:19PM +0100, Paul B Mahol wrote: Signed-off-by: Paul B Mahol --- libavcodec/s302menc.c | 5 +

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

Hello, Please find here enclosed a patch adding frame side data information about green metadata. Best regards, Nicolas DEROUINEAU From c2959f8482065dc400d2e09002363fc8295d5400 Mon Sep 17 00:00:00 2001 From: Nicolas DEROUINEAU Date: Wed, 23 Dec 2015 12:19:03 +0100 Subject: [PATCH 2

Re: [FFmpeg-devel] can't build the ffmpeg for iOS using the newest gas-preprocessor.pl

aarch64 2.7.2+ build on osx should work with the options listed at: http://fate.ffmpeg.org/report.cgi?time=20151221193018&slot=arm64-osx10.9-app le-clang-n2.7 arm:

Re: [FFmpeg-devel] Merge commit 'b9ece15a01782b4f301c0c139d1d7b20f848914c'

Le tridi 3 nivôse, an CCXXIV, Carl Eugen Hoyos a écrit : >> Unfortunately, from user's perspective this >> introduces a regression: Could somebody explain >> the advantages of this commit that justify the >> pain? > Ping. What are you talking about? Regards, -- Nicolas George signature.a

Re: [FFmpeg-devel] Comments about patch

On 12/23/2015 12:13 PM, Clément Bœsch wrote: On Wed, Dec 23, 2015 at 12:04:15PM +0100, Mats Peterson wrote: [...] I don't know what you mean by "a << 24" is undefined if a msb is set. I'm doing exactly as it was done in mov.c before. my bad, didn't see a was unsigned, you can ignore. [...]

Re: [FFmpeg-devel] Comments about patch

On 12/23/2015 12:09 PM, Clément Bœsch wrote: On Wed, Dec 23, 2015 at 12:04:15PM +0100, Mats Peterson wrote: [...] OK. Regarding being the sole author, well I'm the one who has made the patch. I don't understand what you mean. It was a real question; I remember Martin being mentioned in the ti

Re: [FFmpeg-devel] Comments about patch

On Wed, Dec 23, 2015 at 12:04:15PM +0100, Mats Peterson wrote: [...] > I don't know what you mean by "a << 24" is undefined if a msb is set. I'm > doing exactly as it was done in mov.c before. my bad, didn't see a was unsigned, you can ignore. [...] -- Clément B. signature.asc Description: PG

Re: [FFmpeg-devel] Comments about patch

On Wed, Dec 23, 2015 at 12:04:15PM +0100, Mats Peterson wrote: [...] > OK. Regarding being the sole author, well I'm the one who has made the > patch. I don't understand what you mean. > It was a real question; I remember Martin being mentioned in the ticket, and Carl Eugen working on a patch. Ma

Re: [FFmpeg-devel] Comments about patch

On 12/23/2015 11:55 AM, Clément Bœsch wrote: On Wed, Dec 23, 2015 at 11:54:28AM +0100, Mats Peterson wrote: Clement, can you please repeat what you said. I don't know where the hell your post went. http://ffmpeg.org/pipermail/ffmpeg-devel/2015-December/185664.html _

Re: [FFmpeg-devel] Comments about patch

On Wed, Dec 23, 2015 at 11:54:28AM +0100, Mats Peterson wrote: > Clement, can you please repeat what you said. I don't know where the hell > your post went. > http://ffmpeg.org/pipermail/ffmpeg-devel/2015-December/185664.html -- Clément B. signature.asc Description: PGP signature

[FFmpeg-devel] Comments about patch

Clement, can you please repeat what you said. I don't know where the hell your post went. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] libavformat: palettized QuickTime video in Matroska, round 4

On Wed, Dec 23, 2015 at 11:25:44AM +0100, Mats Peterson wrote: [...] > From: Mats Peterson So are you the sole author of this? > Date: Wed, 23 Dec 2015 11:19:06 +0100 > Subject: [PATCH] libavformat: palettized QuickTime video in Matroska, round 4 > Use "[PATCH v4]" instead of the suffix as you

[FFmpeg-devel] [PATCH] libavformat: palettized QuickTime video in Matroska, round 4

Here is another update of my patch for palettized QuickTime video in Matroska. Only read the first 86 - 16 (16 bytes are already read in ff_mov_read_stsd_entries()) bytes into the 'stsd' variable in mov_parse_stsd_video() in mov.c, since we are going to read the palette from the file in ff_ge