Re: [FFmpeg-devel] [libav-devel] [RFC] Cineform HD questions

2015-12-30 Thread Kieran Kunhya
On 31 December 2015 at 04:28, Ganesh Ajjanagadde wrote: > On Wed, Dec 30, 2015 at 8:13 PM, Kieran Kunhya wrote: >> >> This patch is the first attempt at getting a working Cineform HD decoder >> into avcodec >> It supports YUV422P10 files which are the majority

Re: [FFmpeg-devel] [libav-devel] [RFC] Cineform HD questions

2015-12-30 Thread Ganesh Ajjanagadde
On Wed, Dec 30, 2015 at 8:13 PM, Kieran Kunhya wrote: > > This patch is the first attempt at getting a working Cineform HD decoder into > avcodec > It supports YUV422P10 files which are the majority of files in the wild > There are some files not supported such as those from film

[FFmpeg-devel] [RFC][PATCH] avcodec: First hacky attempt at supporting CFHD

2015-12-30 Thread Kieran Kunhya
--- libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/cfhd.c | 707 libavcodec/cfhd.h | 97 +++ libavcodec/cfhddata.c | 466 +++

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

2015-12-30 Thread Ganesh Ajjanagadde
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 --enable-hardcoded-tables. Signed-off-by: Ganesh Ajjanagadde ---

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

2015-12-30 Thread Ganesh Ajjanagadde
This gets rid of some branches to speed up table generation slightly (impact higher on mulaw than alaw). Tables are identical to before, tested with FATE. Sample benchmark (Haswell, GNU/Linux+gcc): old: 313494 decicycles in build_alaw_table,4094 runs, 2 skips 315959 decicycles in

Re: [FFmpeg-devel] [PATCH] configure: make gcc 2 detection more robust

2015-12-30 Thread Michael Niedermayer
On Tue, Dec 29, 2015 at 07:05:20PM -0800, Ganesh Ajjanagadde wrote: > Solves an issue that will get triggered when gcc 20 rolls in. > > Found-by: Hendrik Leppkes > Signed-off-by: Ganesh Ajjanagadde > --- > configure | 3 ++- > 1 file changed, 2

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

2015-12-30 Thread Ganesh Ajjanagadde
On Tue, Dec 29, 2015 at 8:56 AM, Ganesh Ajjanagadde wrote: > Previous commit has sped up dsd_tablegen, and now table generation takes > ~ 40k cycles. Thus, these tables can always be generated at runtime. > > Tested with/without --enable-hardcoded-tables. > >

Re: [FFmpeg-devel] support for reading / writing encrypted MP4 files

2015-12-30 Thread Michael Niedermayer
On Tue, Dec 29, 2015 at 08:05:28PM +, Eran Kornblau wrote: > > > > > +case MKTAG('e','n','c','v'):// encrypted video > > > > > +case MKTAG('e','n','c','a'):// encrypted audio > > > > > +id = mov_codec_id(st, format); > > > > > +st->codec->codec_id = id;

[FFmpeg-devel] SSH ed25519 host key update

2015-12-30 Thread Michael Niedermayer
Hi due to http://wiki.hetzner.de/index.php/Ed25519/en we have updated the Ed25519 host key from ffmpeg.org we are not entirely sure our box was affected but I decided to regenerate the key to be double sure new key fingerprint is: 54:c8:05:df:29:70:3d:77:81:b1:d6:be:c8:4e:68:73 you may or may

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

2015-12-30 Thread Sasi Inguva
Pass through color range to vp9 encoder, and parse it in libvpxdec Signed-off-by: Sasi Inguva --- libavcodec/libvpxdec.c | 4 libavcodec/libvpxenc.c | 20 2 files changed, 24 insertions(+) diff --git a/libavcodec/libvpxdec.c b/libavcodec/libvpxdec.c

Re: [FFmpeg-devel] [PATCH] configure: make gcc 2 detection more robust

2015-12-30 Thread Ganesh Ajjanagadde
On Wed, Dec 30, 2015 at 7:45 AM, Michael Niedermayer wrote: > On Tue, Dec 29, 2015 at 07:05:20PM -0800, Ganesh Ajjanagadde wrote: >> Solves an issue that will get triggered when gcc 20 rolls in. >> >> Found-by: Hendrik Leppkes >> Signed-off-by: Ganesh

Re: [FFmpeg-devel] [PATCH 1/2] lavc/dsd_tablegen: speed up table generation

2015-12-30 Thread Ganesh Ajjanagadde
On Tue, Dec 29, 2015 at 4:09 PM, Michael Niedermayer wrote: > On Tue, Dec 29, 2015 at 08:56:48AM -0800, Ganesh Ajjanagadde wrote: >> Tables are bit identical. >> Sample benchmark (Haswell, GNU/Linux+gcc): >> old: >> 814485 decicycles in dsd_ctables_tableinit, 512

Re: [FFmpeg-devel] [PATCH] lavc/cook: get rid of wasteful pow in init_pow2table

2015-12-30 Thread Ganesh Ajjanagadde
On Tue, Dec 29, 2015 at 1:12 PM, Ganesh Ajjanagadde wrote: > On Tue, Dec 29, 2015 at 11:29 AM, Clément Bœsch wrote: >> On Tue, Dec 29, 2015 at 09:28:34AM -0800, Ganesh Ajjanagadde wrote: >>> The table is highly structured, so pow (or exp2 for that matter) can >>>

Re: [FFmpeg-devel] [PATCH 1/3] lavd/avfoundation: Switch from run-time approach for format detection to format query and Organize frame buffer in buffer queue.

2015-12-30 Thread Michael Niedermayer
On Tue, Nov 24, 2015 at 07:31:20PM +0100, Thilo Borgmann wrote: > > avfoundation.m | 385 > - > 1 file changed, 166 insertions(+), 219 deletions(-) > 71ff751504ec88ba4527fcf40535249d671880ca >

Re: [FFmpeg-devel] [PATCH 1/2] lavc/aac_ac3_parser: avoid zeroing codec parameters if we haven't read a frame

2015-12-30 Thread Michael Niedermayer
On Wed, Nov 25, 2015 at 02:54:59PM -0600, Rodger Combs wrote: > This caused issues when seeking in some unusual MPEGTS files > --- > libavcodec/aac_ac3_parser.c | 4 > 1 file changed, 4 insertions(+) LGTM thanks PS: would adding a fate test be easy ? [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 2/2] lavc/aac_ac3_parser: reindent

2015-12-30 Thread Michael Niedermayer
On Wed, Nov 25, 2015 at 02:55:00PM -0600, Rodger Combs wrote: > --- > libavcodec/aac_ac3_parser.c | 45 > +++-- > 1 file changed, 23 insertions(+), 22 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH v4 11/14] mips: do not disable any feature for generic cores

2015-12-30 Thread Michael Niedermayer
On Fri, Dec 04, 2015 at 03:48:27PM +, Vicente Olivert Riera wrote: > We don't know which features are available when the user selects a > generic core, so don't disable anything by default and let the user > decide. > > Signed-off-by: Vicente Olivert Riera > --- >

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

2015-12-30 Thread Sasi Inguva
VPX_IMAGE_ABI_VERSION was not changed when the color_range enum was introduced so I am afraid I would have to drop libvpxdec changes. Sent the new patch On Wed, Dec 30, 2015 at 10:56 AM, James Almer wrote: > On 12/30/2015 3:47 PM, James Almer wrote: > > On 12/30/2015 3:23 PM,

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

2015-12-30 Thread James Almer
On 12/30/2015 3:23 PM, Sasi Inguva wrote: > Pass through color range to vp9 encoder, and parse it in libvpxdec > > Signed-off-by: Sasi Inguva > --- > libavcodec/libvpxdec.c | 4 > libavcodec/libvpxenc.c | 20 > 2 files changed, 24 insertions(+) > >

[FFmpeg-devel] [PATCH] lavf/qtpalette: Ignore greyscale bit in 1-bit video

2015-12-30 Thread Mats Peterson
Michael, can you apply this one in the meantime, just to get rid of it, and if it seems sensible, until someone discovers how to solve the 1-bit palettized qtrle issue? Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 60798a04ef1204dadf5395cbf7205ea3957fafbb Mon Sep 17 00:00:00

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

2015-12-30 Thread Sasi Inguva
Pass through color range to vp9 encoder. Signed-off-by: Sasi Inguva --- libavcodec/libvpxenc.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index 8588241..58da30d 100644 ---

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

2015-12-30 Thread James Almer
On 12/30/2015 3:47 PM, James Almer wrote: > On 12/30/2015 3:23 PM, Sasi Inguva wrote: >> Pass through color range to vp9 encoder, and parse it in libvpxdec >> >> Signed-off-by: Sasi Inguva >> --- >> libavcodec/libvpxdec.c | 4 >> libavcodec/libvpxenc.c | 20

Re: [FFmpeg-devel] [PATCH] lavf/qtpalette: Ignore greyscale bit in 1-bit video

2015-12-30 Thread Mats Peterson
On 12/30/2015 07:52 PM, Mats Peterson wrote: Michael, can you apply this one in the meantime, just to get rid of it, and if it seems sensible, until someone discovers how to solve the 1-bit palettized qtrle issue? Mats ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH v4 11/14] mips: do not disable any feature for generic cores

2015-12-30 Thread Andreas Cadhalpun
On 30.12.2015 20:35, Michael Niedermayer wrote: > On Fri, Dec 04, 2015 at 03:48:27PM +, Vicente Olivert Riera wrote: >> We don't know which features are available when the user selects a >> generic core, so don't disable anything by default and let the user >> decide. >> >> Signed-off-by:

Re: [FFmpeg-devel] [PATCH] avfilter: add ahistogram multimedia filter

2015-12-30 Thread Lou Logan
On Wed, 30 Dec 2015 13:49:07 +0100, Paul B Mahol wrote: > diff --git a/doc/filters.texi b/doc/filters.texi [...] > +@item ascale > +Set amplitude scale. > + > +It accepts the following values: > +@table @samp > +@item log > +logarithmic > +@item lin > +linear > +@end table Default is

Re: [FFmpeg-devel] [RFC v2 3/3] daaladec: Implement a native Daala decoder

2015-12-30 Thread Andreas Cadhalpun
Hi, On 30.12.2015 02:09, Ronald S. Bultje wrote: > On Tue, Dec 29, 2015 at 6:54 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: >> I prefer that input for such performance-sensitive dsp code gets sanitized, >> so that overflows are (nearly) impossible. > > > I don't think it

Re: [FFmpeg-devel] [PATCH v3 09/14] mips: add mips32r6 architecture variant

2015-12-30 Thread Michael Niedermayer
On Thu, Dec 03, 2015 at 01:22:24PM +, Vicente Olivert Riera wrote: > Signed-off-by: Vicente Olivert Riera > --- > Changes v2 -> v3: > - Nothing. > > Changes v1 -> v2: > - Nothing. > > configure | 12 > 1 file changed, 12 insertions(+) applied

Re: [FFmpeg-devel] [PATCH v3 10/14] mips: put "disable mipsfpu" in a better place for loongson

2015-12-30 Thread Michael Niedermayer
On Thu, Dec 03, 2015 at 01:22:25PM +, Vicente Olivert Riera wrote: > Let's disable the ISAs first, and then the core capabilities, as we do > for the rest of the cores. This way the code is better organized. > > Signed-off-by: Vicente Olivert Riera > --- > Changes

[FFmpeg-devel] [PATCH] jpegls: allocate large enough zero buffer

2015-12-30 Thread Andreas Cadhalpun
It is read up to length s->width * stride, which can be larger than the linesize. (stride = (s->nb_components > 1) ? 3 : 1) This fixes an out of bounds read. Signed-off-by: Andreas Cadhalpun --- libavcodec/jpeglsdec.c | 2 +- 1 file changed, 1 insertion(+), 1

[FFmpeg-devel] [PATCH] avfilter/vf_scale: set proper out frame color range

2015-12-30 Thread Thomas Mundt
Prevents that following scalers in the filter chain will do unintentional color range conversions. Fixes Ticket #5096 Signed-off-by: Thomas Mundt --- libavfilter/vf_scale.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavfilter/vf_scale.c b/libavfilter/vf_scale.c

Re: [FFmpeg-devel] support for reading / writing encrypted MP4 files

2015-12-30 Thread Eran Kornblau
> > Please let me know if you think that is ok, and I will resubmit the patch > > with all fixes. > > should be ok > Updated patch attached, diff from previous patch is: --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -4026,6 +4026,14 @@ static int mov_read_frma(MOVContext *c, AVIOContext

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

2015-12-30 Thread Ronald S. Bultje
Hi, On Wed, Dec 30, 2015 at 3:46 PM, Sasi Inguva wrote: > VPX_IMAGE_ABI_VERSION was not changed when the color_range enum was > introduced so I am afraid I would have to drop libvpxdec changes. It should work in 1.5.0, no? So can't you do a version check? Ronald

Re: [FFmpeg-devel] Configure with --cpu=host and Clang

2015-12-30 Thread Michael Niedermayer
On Mon, Dec 14, 2015 at 06:11:00PM +0300, yukari yakumo wrote: > Hi everyone! > I'm Gentoo Linux user and currently I play with Clang as default compiler. > And currently I tried to build ffmpeg 2.3.8 with Clang 3.7.0 and > -march=native in C(XX)FLAGS (ffmpeg-2.3.8.ebuild automatically convert

Re: [FFmpeg-devel] [PATCH] avfilter/vf_scale: set proper out frame color range

2015-12-30 Thread Michael Niedermayer
On Thu, Dec 31, 2015 at 12:01:21AM +0100, Thomas Mundt wrote: > Prevents that following scalers in the filter chain will do unintentional > color range conversions. > Fixes Ticket #5096 > > Signed-off-by: Thomas Mundt > --- > libavfilter/vf_scale.c | 2 ++ > 1 file changed, 2

Re: [FFmpeg-devel] [PATCH] AAC encoder: fix assertion error with prediction

2015-12-30 Thread Claudio Freire
On Tue, Dec 29, 2015 at 1:12 PM, Claudio Freire wrote: > On Tue, Dec 29, 2015 at 6:18 AM, Rostislav Pehlivanov > wrote: >> Wouldn't it be simpler to just check if the maximum codebook was 0 after >> calculating cost1 and skipping the rest of the code

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

2015-12-30 Thread Ganesh Ajjanagadde
On Wed, Dec 30, 2015 at 8:44 AM, Ganesh Ajjanagadde wrote: > On Tue, Dec 29, 2015 at 8:56 AM, Ganesh Ajjanagadde > wrote: >> Previous commit has sped up dsd_tablegen, and now table generation takes >> ~ 40k cycles. Thus, these tables can always be

[FFmpeg-devel] [RFC] Cineform HD questions

2015-12-30 Thread Kieran Kunhya
This patch is the first attempt at getting a working Cineform HD decoder into avcodec It supports YUV422P10 files which are the majority of files in the wild There are some files not supported such as those from film scanners and some older files which do something unusual with chroma and the

[FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/trim.c | 27 +++ 1 file changed, 27 insertions(+) diff --git a/libavfilter/trim.c b/libavfilter/trim.c index b2d9c6f..9d040dd 100644 --- a/libavfilter/trim.c +++ b/libavfilter/trim.c @@ -345,6 +345,32 @@ drop:

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Clément Bœsch
On Wed, Dec 30, 2015 at 09:17:56AM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/trim.c | 27 +++ > 1 file changed, 27 insertions(+) > > diff --git a/libavfilter/trim.c b/libavfilter/trim.c > index b2d9c6f..9d040dd 100644

Re: [FFmpeg-devel] [PATCH] lavc/qtrle: Use AV_PIX_FMT_PAL8 for 1-bit video

2015-12-30 Thread Mats Peterson
On 12/30/2015 08:35 AM, Mats Peterson wrote: On 12/30/2015 08:17 AM, Mats Peterson wrote: When comparing the output in QuickTime and FFmpeg, I noticed that the background is blue in FFmpeg, while it is white (or transparent, if you will) in QuickTime. Hold your horses on this patch. Mats

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

2015-12-30 Thread Hendrik Leppkes
On Wed, Dec 30, 2015 at 4:39 AM, Ganesh Ajjanagadde wrote: > The log is anyway rounded to an integer, so one may use an frexp > based approach. Note that this may be made frexpf; if arguments are less than > 2^24 there is no loss. Kept as double precision for simplicity;

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Paul B Mahol
On 12/30/15, Clement Boesch wrote: > On Wed, Dec 30, 2015 at 09:17:56AM +0100, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> libavfilter/trim.c | 27 +++ >> 1 file changed, 27 insertions(+) >> >> diff --git

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Nicolas George
Le decadi 10 nivôse, an CCXXIV, Clement Boesch a écrit : > Yes; why ff_default_query_formats() is doing a query with > all_channel_layouts and is not actually ff_query_formats_all (which is > doing ff_all_channel_counts)? Carefulness. Filters from the fork use ff_default_query_formats() but still

[FFmpeg-devel] [PATCH] avfilter: add ahistogram multimedia filter

2015-12-30 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/filters.texi | 70 + libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/avf_ahistogram.c | 358 +++ 4 files changed, 430 insertions(+) create

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Clément Bœsch
On Wed, Dec 30, 2015 at 09:14:23AM +, Paul B Mahol wrote: > > Q: why is it not the default? > > > > The all channel counts layout? > Yes; why ff_default_query_formats() is doing a query with all_channel_layouts and is not actually ff_query_formats_all (which is doing ff_all_channel_counts)?

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Ganesh Ajjanagadde
On Wed, Dec 30, 2015 at 12:17 AM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol [...] > > +static int atrim_query_formats(AVFilterContext *ctx) > +{ > +AVFilterFormats *formats; > +AVFilterChannelLayouts *layouts; > +int ret; > + > +layouts