Re: [FFmpeg-devel] [PATCH 1/7] lavfi: remove astreamsync.

2015-10-24 Thread Paul B Mahol
Dana 24. 10. 2015. 19:06 osoba "Nicolas George" napisala je: > > It was only useful for very specific testing purposes > and appears to be currently partially broken. What's broken? > > Signed-off-by: Nicolas George > --- > MAINTAINERS | 1 - > doc/filters.texi |

Re: [FFmpeg-devel] Wanted to contribute

2015-10-24 Thread rishi shah
Hi wm4, I went through that link but its all about just coding convention and patch submission methodology. Nothing about dev setup or nothing about list of projects, modules and bugs. Thanks, Rishi On Sat, Oct 24, 2015 at 6:48 PM, wm4 wrote: > On Sat, 24 Oct 2015 18:21:45 +0530 > rishi shah

Re: [FFmpeg-devel] [PATCH 1/7] lavfi: remove astreamsync.

2015-10-24 Thread Nicolas George
Le tridi 3 brumaire, an CCXXIV, Paul B Mahol a écrit : > What's broken? EOF handling. I am not sure it did ever work. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org htt

Re: [FFmpeg-devel] Can we drop OpenJPEG 1.5 in favor of 2.x?

2015-10-24 Thread Dominik 'Rathann' Mierzejewski
On Saturday, 24 October 2015 at 18:59, Carl Eugen Hoyos wrote: > Michael Bradshaw gmail.com> writes: > > > tl;dr: I've got a patch that updates OpenJPEG to 2.0/2.1. > > Currently, I've opted to drop OpenJPEG 1.5. Should I > > proceed with preparing this patch for submission, or > > should I al

Re: [FFmpeg-devel] [PATCH] Don't needlessly reinitialize ff_cos_## tables.

2015-10-24 Thread Michael Niedermayer
On Sat, Oct 24, 2015 at 03:10:58PM +0200, wm4 wrote: > On Sat, 24 Oct 2015 04:13:18 +0200 > Michael Niedermayer wrote: > > > On Thu, Oct 22, 2015 at 09:48:30PM -0700, Dale Curtis wrote: > > > On Wed, Oct 21, 2015 at 6:52 PM, Dale Curtis > > > wrote: > > > > > > > On Tue, Oct 20, 2015 at 11:50

Re: [FFmpeg-devel] [PATCH] x86/intmath: allow the source operand for icc/icl ff_ctzll to be a memory location

2015-10-24 Thread Ganesh Ajjanagadde
On Sat, Oct 24, 2015 at 6:08 PM, James Almer wrote: > This gives the compiler some flexibility > > Signed-off-by: James Almer > --- > libavutil/x86/intmath.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/x86/intmath.h b/libavutil/x86/intmath.h > index 7881e3c

Re: [FFmpeg-devel] Wanted to contribute

2015-10-24 Thread Ganesh Ajjanagadde
On Sat, Oct 24, 2015 at 6:22 PM, rishi shah wrote: > Hi wm4, > I went through that link but its all about just coding convention and patch > submission methodology. Nothing about dev setup or nothing about list of > projects, modules and bugs. May be cliched, but the first rule is not to top post

Re: [FFmpeg-devel] [PATCH] x86/intmath: allow the source operand for icc/icl ff_ctzll to be a memory location

2015-10-24 Thread James Almer
On 10/24/2015 7:48 PM, Ganesh Ajjanagadde wrote: > On Sat, Oct 24, 2015 at 6:08 PM, James Almer wrote: >> This gives the compiler some flexibility >> >> Signed-off-by: James Almer >> --- >> libavutil/x86/intmath.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavu

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/vf_deshake: use a void * comparator for consistency

2015-10-24 Thread wm4
On Sat, 24 Oct 2015 18:02:36 -0400 Ganesh Ajjanagadde wrote: > For generality, qsort uses a comparator whose elements are void *. This > makes the comparator have such a form, and thus makes the void * cast of > the comparator pointer useless. Furthermore, this makes the code more > consistent wi

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/vf_deshake: use a void * comparator for consistency

2015-10-24 Thread Henrik Gramner
On Sun, Oct 25, 2015 at 12:02 AM, Ganesh Ajjanagadde wrote: > -static int cmp(const double *a, const double *b) > +static int cmp(const void *a, const void *b) > { > -return *a < *b ? -1 : ( *a > *b ? 1 : 0 ); > +double va = *(double *)a, vb = *(double *)b; > +return va < vb ? -1 : (

Re: [FFmpeg-devel] [PATCH 3/3] avutil/tree: add additional const qualifier to the comparator

2015-10-24 Thread Henrik Gramner
On Sun, Oct 25, 2015 at 12:02 AM, Ganesh Ajjanagadde wrote: > -static int cmp(void *key, const void *node) > +static int cmp(const void *key, const void *node) > { > return (*(int64_t *) key) - ((const CacheEntry *) node)->logical_pos; > } > -int ff_nut_sp_pos_cmp(const Syncpoint *a, const

Re: [FFmpeg-devel] [PATCH] Don't needlessly reinitialize ff_cos_## tables.

2015-10-24 Thread Ronald S. Bultje
On Sat, Oct 24, 2015 at 6:45 PM, Michael Niedermayer wrote: > either way, this subject is rather uninterresting ill try to do > something more usefull for FFmpeg than arguing about these > hypothetical cases ... +100. Many of these tools are like -Wpedantic. Reminds me of my rants about helgri

[FFmpeg-devel] [DECISION] Include more developers in the voting committee [#2]

2015-10-24 Thread Michael Niedermayer
Hi all Heres another suggested addition to the voting committee also off topic but i wonder if "[DECISSION]" in the subject is the best choice for this I Suggest to add all people who pushed or pull-requested 20 or more commits according to the commiter field in the last year (the initial voting

Re: [FFmpeg-devel] Can we drop OpenJPEG 1.5 in favor of 2.x?

2015-10-24 Thread Ben Boeckel
On Sat, 24 Oct, 2015 at 21:37:57 GMT, Michael Bradshaw wrote: > Crap, you're right. Before I started this I checked if Ubuntu had openjpeg2 > and they do[1] but it turns out it's really just openjpeg 1.3 (if anyone > knows why they made a separate package named libopenjpeg2 when it's really > openj

[FFmpeg-devel] [PATCH] all: simplify qsort comparators, and add const-correctness

2015-10-24 Thread Ganesh Ajjanagadde
All the comparator API needs is > 0, < 0, or = 0 signalling: it does not need +1, -1, 0. This avoids some useless branching. This also adds const-correctness when needed for the comparators. Signed-off-by: Ganesh Ajjanagadde --- cmdutils_opencl.c | 2 +- ffmpeg.c

Re: [FFmpeg-devel] [PATCH 3/3] avutil/tree: add additional const qualifier to the comparator

2015-10-24 Thread Ganesh Ajjanagadde
On Sat, Oct 24, 2015 at 7:17 PM, Henrik Gramner wrote: > On Sun, Oct 25, 2015 at 12:02 AM, Ganesh Ajjanagadde > wrote: >> -static int cmp(void *key, const void *node) >> +static int cmp(const void *key, const void *node) >> { >> return (*(int64_t *) key) - ((const CacheEntry *) node)->logic

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/vf_deshake: use a void * comparator for consistency

2015-10-24 Thread Ganesh Ajjanagadde
On Sat, Oct 24, 2015 at 7:07 PM, Henrik Gramner wrote: > On Sun, Oct 25, 2015 at 12:02 AM, Ganesh Ajjanagadde > wrote: >> -static int cmp(const double *a, const double *b) >> +static int cmp(const void *a, const void *b) >> { >> -return *a < *b ? -1 : ( *a > *b ? 1 : 0 ); >> +double va =

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/vf_deshake: use a void * comparator for consistency

2015-10-24 Thread Ganesh Ajjanagadde
On Sat, Oct 24, 2015 at 7:04 PM, wm4 wrote: > On Sat, 24 Oct 2015 18:02:36 -0400 > Ganesh Ajjanagadde wrote: > >> For generality, qsort uses a comparator whose elements are void *. This >> makes the comparator have such a form, and thus makes the void * cast of >> the comparator pointer useless.

Re: [FFmpeg-devel] [PATCH] version.sh: add note that ffversion.h is auto-generated

2015-10-24 Thread Ganesh Ajjanagadde
On Sat, Oct 24, 2015 at 9:05 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 7:04 PM, Ganesh Ajjanagadde > wrote: >> Signed-off-by: Ganesh Ajjanagadde >> --- >> version.sh | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/version.sh b/version.sh >> index f9754eb..a9d7e39 10075

Re: [FFmpeg-devel] [PATCH] x86/intmath: allow the source operand for icc/icl ff_ctzll to be a memory location

2015-10-24 Thread Ganesh Ajjanagadde
On Sat, Oct 24, 2015 at 7:03 PM, James Almer wrote: > On 10/24/2015 7:48 PM, Ganesh Ajjanagadde wrote: >> On Sat, Oct 24, 2015 at 6:08 PM, James Almer wrote: >>> This gives the compiler some flexibility >>> >>> Signed-off-by: James Almer >>> --- >>> libavutil/x86/intmath.h | 2 +- >>> 1 file ch

Re: [FFmpeg-devel] [PATCH] version.sh: add note that ffversion.h is auto-generated

2015-10-24 Thread Ronald S. Bultje
Hi, On Sat, Oct 24, 2015 at 9:13 PM, Ganesh Ajjanagadde wrote: > On Sat, Oct 24, 2015 at 9:05 AM, Ganesh Ajjanagadde > wrote: > > On Thu, Oct 15, 2015 at 7:04 PM, Ganesh Ajjanagadde > > wrote: > >> Signed-off-by: Ganesh Ajjanagadde > >> --- > >> version.sh | 1 + > >> 1 file changed, 1 inser

Re: [FFmpeg-devel] [PATCH] all: simplify qsort comparators, and add const-correctness

2015-10-24 Thread Ronald S. Bultje
Hi, On Sat, Oct 24, 2015 at 9:02 PM, Ganesh Ajjanagadde wrote: > All the comparator API needs is > 0, < 0, or = 0 signalling: it does not > need +1, -1, 0. This avoids some useless branching. > [..] diff --git a/cmdutils_opencl.c b/cmdutils_opencl.c > index 61478e2..d9095b6 100644 > --- a/cmdut

Re: [FFmpeg-devel] [DECISION] Include more developers in the voting committee [#2]

2015-10-24 Thread Ronald S. Bultje
Hi, On Sat, Oct 24, 2015 at 7:53 PM, Michael Niedermayer wrote: > Hi all > > Heres another suggested addition to the voting committee > also off topic but i wonder if "[DECISSION]" in the subject is the best > choice for this > > I Suggest to add all people who pushed or pull-requested 20 or mor

Re: [FFmpeg-devel] [PATCH] all: simplify qsort comparators, and add const-correctness

2015-10-24 Thread Mark Harris
>> >> static int compare_int64(const void *a, const void *b) >> { >> -int64_t va = *(int64_t *)a, vb = *(int64_t *)b; >> -return va < vb ? -1 : va > vb ? +1 : 0; >> +return *(const int64_t *)a - *(const int64_t *)b; >> } >> > > What if the result doesn't fit in int? The input is not

Re: [FFmpeg-devel] [PATCH] avfilter/avf_showcqt: rewrite showcqt and add features

2015-10-24 Thread Muhammad Faiz
On Sat, Oct 24, 2015 at 1:31 AM, Muhammad Faiz wrote: > add yuv444p, yuv422p, and yuv420p output format (lower cpu usage > on ffplay playback because it does not do format conversion) > custom size with size/s option (fullhd option is deprecated) > custom layout with bar_h, axis_h, and sono_h opti

Re: [FFmpeg-devel] [PATCH] avfilter/avf_showcqt: rewrite showcqt and add features

2015-10-24 Thread Michael Niedermayer
On Sun, Oct 25, 2015 at 08:43:07AM +0700, Muhammad Faiz wrote: > On Sat, Oct 24, 2015 at 1:31 AM, Muhammad Faiz wrote: > > add yuv444p, yuv422p, and yuv420p output format (lower cpu usage > > on ffplay playback because it does not do format conversion) > > custom size with size/s option (fullhd op

[FFmpeg-devel] [RFC] mem: facilitate imports into GPU memory space.

2015-10-24 Thread Gwenole Beauchesne
Allow for av_malloc() to allocate memory that could be mapped into the GPU address space. This requires allocations on page boundaries. On the video memory buffers side, this requires minimal alignment of strides to 64 bytes. Option 1: use heuristics in av_malloc() - break down into mem, frame, an

Re: [FFmpeg-devel] [PATCH] vf_lut: Add support for RGB48 and RGBA64

2015-10-24 Thread Steven Robertson
Done, thanks. On Tue, Oct 13, 2015 at 12:51 AM, Paul B Mahol wrote: > On 10/11/15, Steven Robertson wrote: > > Thanks for taking a look! > > > > Steve > > > > lgtm, do you need to update fate? > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.

<    1   2