Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 5:33 PM, Michael Niedermayer wrote: > On Fri, Nov 13, 2015 at 07:38:31AM -0500, Ganesh Ajjanagadde wrote: >> On Fri, Nov 13, 2015 at 6:40 AM, Michael Niedermayer >> wrote: >> > On Thu, Nov 12, 2015 at 09:46:04PM -0500, Ganesh Ajjanagadde wrote: >> >> The rationale for this

Re: [FFmpeg-devel] [PATCHv4] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 7:17 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Nov 13, 2015 at 6:16 PM, Ganesh Ajjanagadde > wrote: > >> On Fri, Nov 13, 2015 at 4:52 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Fri, Nov 13, 2015 at 4:28 PM, Ganesh Ajjanagadde >> > wrote: >> > >> >> On Fri,

Re: [FFmpeg-devel] [libav-devel] [PATCH 3/4] dds: make sure pallete frame buffer exists before use

2015-11-13 Thread Vittorio Giovara
On Fri, Nov 13, 2015 at 10:01 PM, Andreas Cadhalpun wrote: > On 13.11.2015 02:08, Vittorio Giovara wrote: >> oh I see, that can happen for a special crafted file, DDPF_FOURCC has >> been introduced recently while DDPF_PALETTE has been removed, so a >> normal file should not have both set. > > OK,

[FFmpeg-devel] [PATCH] Added VideoToolbox H.264 encoder.

2015-11-13 Thread Rick Kern
--- MAINTAINERS|1 + configure | 14 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/vtenc.c | 1089 5 files changed, 1106 insertions(+) create mode 100644 libavcodec/vtenc.c diff -

[FFmpeg-devel] [PATCH] VideoToolbox H.264 Encoder

2015-11-13 Thread Rick Kern
This adds support for H.264 encoding using the VideoToolbox API. Signed-off-by: Rick Kern Rick Kern (1): Added VideoToolbox H.264 encoder. MAINTAINERS|1 + configure | 14 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/vtenc.c

Re: [FFmpeg-devel] [PATCHv4] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Fri, Nov 13, 2015 at 6:16 PM, Ganesh Ajjanagadde wrote: > On Fri, Nov 13, 2015 at 4:52 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Fri, Nov 13, 2015 at 4:28 PM, Ganesh Ajjanagadde > > wrote: > > > >> On Fri, Nov 13, 2015 at 1:28 PM, Ronald S. Bultje > >> wrote: > >> > Hi, > >> > >

[FFmpeg-devel] Error in ATRAC1 decoder?

2015-11-13 Thread Даниил Чередник
Hello! I have noticed if decode https://samples.ffmpeg.org/A-codecs/ATRAC1/Test%20tones%20disc%20-%20Chirp.aea file by ffmpeg we got aliasing near 11025Hz. Screenshots: https://yadi.sk/i/r-95jZkKkSnbu https://yadi.sk/i/PV92LNESkSnby I was tried to solve it: ATRAC1 - hybrid codec, has two stacked

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/faandct: use more accurate constants

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 6:17 PM, Michael Niedermayer wrote: > On Fri, Nov 13, 2015 at 11:42:31AM -0500, Ganesh Ajjanagadde wrote: >> This guarantees a "best effort precision". >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/faandct.c | 29 + >> 1 file cha

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/faandct: use more accurate constants

2015-11-13 Thread Michael Niedermayer
On Fri, Nov 13, 2015 at 11:42:31AM -0500, Ganesh Ajjanagadde wrote: > This guarantees a "best effort precision". > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/faandct.c | 29 + > 1 file changed, 13 insertions(+), 16 deletions(-) > > diff --git a/libavcodec

Re: [FFmpeg-devel] [PATCHv4] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 4:52 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Nov 13, 2015 at 4:28 PM, Ganesh Ajjanagadde > wrote: > >> On Fri, Nov 13, 2015 at 1:28 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Fri, Nov 13, 2015 at 12:17 PM, Ganesh Ajjanagadde >> > wrote: >> > >> >> On Fri,

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Michael Niedermayer
On Fri, Nov 13, 2015 at 07:38:31AM -0500, Ganesh Ajjanagadde wrote: > On Fri, Nov 13, 2015 at 6:40 AM, Michael Niedermayer > wrote: > > On Thu, Nov 12, 2015 at 09:46:04PM -0500, Ganesh Ajjanagadde wrote: > >> The rationale for this function is reflected in the documentation for > >> it, and is cop

Re: [FFmpeg-devel] [PATCH 1/3] aacsbr_fixed: avoid division by zero in sbr_gain_calc

2015-11-13 Thread Andreas Cadhalpun
On 13.11.2015 04:21, Michael Niedermayer wrote: > On Thu, Nov 12, 2015 at 08:51:28PM +0100, Andreas Cadhalpun wrote: >> On 11.11.2015 23:19, Michael Niedermayer wrote: >>> On Wed, Nov 11, 2015 at 09:31:18PM +0100, Andreas Cadhalpun wrote: I'm not sure, but it can happen, when q_mapped is very

Re: [FFmpeg-devel] [PATCHv4] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Fri, Nov 13, 2015 at 4:28 PM, Ganesh Ajjanagadde wrote: > On Fri, Nov 13, 2015 at 1:28 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Fri, Nov 13, 2015 at 12:17 PM, Ganesh Ajjanagadde > > wrote: > > > >> On Fri, Nov 13, 2015 at 12:10 PM, Ronald S. Bultje > >> wrote: > >> > Hi Ganesh,

Re: [FFmpeg-devel] [PATCH] aac_fixed: fix overflow in sbr_sum_square_c

2015-11-13 Thread Andreas Cadhalpun
On 13.11.2015 04:15, Michael Niedermayer wrote: > On Thu, Nov 12, 2015 at 08:43:42PM +0100, Andreas Cadhalpun wrote: >> Considering that the aac float decoder can decode such samples, I tend >> to think that the aac fixed decoder should be able to do that, too. > > IMO this reasoning is flawed >

Re: [FFmpeg-devel] [PATCHv4] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 1:28 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Nov 13, 2015 at 12:17 PM, Ganesh Ajjanagadde > wrote: > >> On Fri, Nov 13, 2015 at 12:10 PM, Ronald S. Bultje >> wrote: >> > Hi Ganesh, >> > On Nov 13, 2015 12:02 PM, "Ganesh Ajjanagadde" >> > wrote: >> >> >> >> The rati

Re: [FFmpeg-devel] [PATCH 3/4] avcodec/ppc/fdctdsp: use more accurate constants

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 2:58 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Nov 13, 2015 at 11:42 AM, Ganesh Ajjanagadde > wrote: >> >> Whoever wrote this stuff had a pretty bad libm - digits differ pretty >> quickly. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/ppc/fdctdsp.c | 1

Re: [FFmpeg-devel] [libav-devel] [PATCH 3/4] dds: make sure pallete frame buffer exists before use

2015-11-13 Thread Andreas Cadhalpun
On 13.11.2015 02:08, Vittorio Giovara wrote: > oh I see, that can happen for a special crafted file, DDPF_FOURCC has > been introduced recently while DDPF_PALETTE has been removed, so a > normal file should not have both set. OK, that makes sense. > Because of that, and how rare palette dds are,

[FFmpeg-devel] [PATCH] avformat: add VPK demuxer

2015-11-13 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/vpk.c| 79 3 files changed, 81 insertions(+) create mode 100644 libavformat/vpk.c diff --git a/libavformat/Makefile b/libavforma

Re: [FFmpeg-devel] [PATCH 3/4] avcodec/ppc/fdctdsp: use more accurate constants

2015-11-13 Thread Ronald S. Bultje
Hi, On Fri, Nov 13, 2015 at 11:42 AM, Ganesh Ajjanagadde wrote: > Whoever wrote this stuff had a pretty bad libm - digits differ pretty > quickly. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/ppc/fdctdsp.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) > > d

Re: [FFmpeg-devel] [PATCHv4] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Fri, Nov 13, 2015 at 12:17 PM, Ganesh Ajjanagadde wrote: > On Fri, Nov 13, 2015 at 12:10 PM, Ronald S. Bultje > wrote: > > Hi Ganesh, > > On Nov 13, 2015 12:02 PM, "Ganesh Ajjanagadde" > > wrote: > >> > >> The rationale for this function is reflected in the documentation for > >> it, an

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: implement wrapped_avframe decoder

2015-11-13 Thread Muhammad Faiz
On Wed, Nov 11, 2015 at 4:53 AM, wm4 wrote: > On Tue, 10 Nov 2015 22:42:31 +0100 > Michael Niedermayer wrote: > >> On Tue, Nov 10, 2015 at 03:15:50PM +0100, wm4 wrote: >> > On Tue, 10 Nov 2015 14:31:26 +0100 >> > Michael Niedermayer wrote: >> > >> > > On Tue, Nov 10, 2015 at 04:54:17PM +0700, Mu

Re: [FFmpeg-devel] [PATCHv4] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 12:10 PM, Ronald S. Bultje wrote: > Hi Ganesh, > On Nov 13, 2015 12:02 PM, "Ganesh Ajjanagadde" > wrote: >> >> The rationale for this function is reflected in the documentation for >> it, and is copied here: >> >> Clip a double value into the long long amin-amax range. >>

Re: [FFmpeg-devel] [PATCHv4] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi Ganesh, On Nov 13, 2015 12:02 PM, "Ganesh Ajjanagadde" wrote: > > The rationale for this function is reflected in the documentation for > it, and is copied here: > > Clip a double value into the long long amin-amax range. > This function is needed because conversion of floating point to integer

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/faandct: use typedef instead of #define

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 11:58 AM, Nicolas George wrote: > Le tridi 23 brumaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >> See e.g >> https://stackoverflow.com/questions/1666353/are-typedef-and-define-the-same-in-c >> for rationale. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/f

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 9:59 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, Nov 13, 2015 at 9:23 AM, Ganesh Ajjanagadde > wrote: >> >> The rationale for this function is reflected in the documentation for >> it, and is copied here: >> >> Clip a double value into the long long amin-amax range. >> T

[FFmpeg-devel] [PATCHv4] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
The rationale for this function is reflected in the documentation for it, and is copied here: Clip a double value into the long long amin-amax range. This function is needed because conversion of floating point to integers when it does not fit in the integer's representation does not necessarily s

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/faandct: use typedef instead of #define

2015-11-13 Thread Nicolas George
Le tridi 23 brumaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > See e.g > https://stackoverflow.com/questions/1666353/are-typedef-and-define-the-same-in-c > for rationale. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/faandct.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [FFmpeg-devel] [PATCH] all: use predefined mathematics macros

2015-11-13 Thread Ganesh Ajjanagadde
On Wed, Nov 11, 2015 at 6:53 AM, Michael Niedermayer wrote: > On Tue, Nov 10, 2015 at 10:35:23PM -0500, Ganesh Ajjanagadde wrote: >> This uses M_SQRT2, M_PI, and M_E instead of the actual literals. >> Benefits include: >> 1. Reduced scope for copy/paste errors and improved readability. >> 2. Consi

[FFmpeg-devel] [PATCH 4/4] avcodec/faandct: use more accurate constants

2015-11-13 Thread Ganesh Ajjanagadde
This guarantees a "best effort precision". Signed-off-by: Ganesh Ajjanagadde --- libavcodec/faandct.c | 29 + 1 file changed, 13 insertions(+), 16 deletions(-) diff --git a/libavcodec/faandct.c b/libavcodec/faandct.c index 473bc1d..c8ea328 100644 --- a/libavcodec/faa

[FFmpeg-devel] [PATCH 3/4] avcodec/ppc/fdctdsp: use more accurate constants

2015-11-13 Thread Ganesh Ajjanagadde
Whoever wrote this stuff had a pretty bad libm - digits differ pretty quickly. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/ppc/fdctdsp.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/libavcodec/ppc/fdctdsp.c b/libavcodec/ppc/fdctdsp.c index 924d12c..9286

[FFmpeg-devel] [PATCH 2/4] avcodec/faandct: use typedef instead of #define

2015-11-13 Thread Ganesh Ajjanagadde
See e.g https://stackoverflow.com/questions/1666353/are-typedef-and-define-the-same-in-c for rationale. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/faandct.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/faandct.c b/libavcodec/faandct.c index 4053d69..473bc

[FFmpeg-devel] [PATCH 1/4] all: use predefined mathematics macros

2015-11-13 Thread Ganesh Ajjanagadde
This uses M_SQRT1_2, M_SQRT2, M_PI, and M_E instead of the actual literals. Benefits include: 1. Reduced scope for copy/paste errors and improved readability. 2. Consistency across the codebase. 3. Actually fixes an incorrect sqrt(2) in avcodec/ppc. 4. Greater precision in avcodec/ac3. Patch teste

[FFmpeg-devel] [PATCH 0/4] more accurate constants

2015-11-13 Thread Ganesh Ajjanagadde
1/4 uses predefined mathematics macros whenever possible to improve readability and in some cases accuracy. 2/4 is a minor "pedantic" patch for something noticed while working on this stuff. 3/4 is a more important patch, since the precomputed cosines on ppc are terrible, with accuracy off even

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Fri, Nov 13, 2015 at 10:42 AM, Ganesh Ajjanagadde wrote: > On Fri, Nov 13, 2015 at 10:37 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Fri, Nov 13, 2015 at 10:17 AM, Ganesh Ajjanagadde > > wrote: > >> > >> On Fri, Nov 13, 2015 at 9:59 AM, Ronald S. Bultje > >> wrote: > >> > Hi, > >>

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 10:37 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, Nov 13, 2015 at 10:17 AM, Ganesh Ajjanagadde > wrote: >> >> On Fri, Nov 13, 2015 at 9:59 AM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Fri, Nov 13, 2015 at 9:23 AM, Ganesh Ajjanagadde >> > >> > wrote: >> >> >> >>

Re: [FFmpeg-devel] Outreachy 2015-12

2015-11-13 Thread Nicolas George
Le tridi 23 vendémiaire, an CCXXIV, Michael Niedermayer a écrit : > Ive copied the may 2015 outreachy page for the next round > https://trac.ffmpeg.org/wiki/SponsoringPrograms/Outreachy/2015-12 > please help updating the page so everything on it is correct for > december > links, dates, available m

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Fri, Nov 13, 2015 at 10:17 AM, Ganesh Ajjanagadde wrote: > On Fri, Nov 13, 2015 at 9:59 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Fri, Nov 13, 2015 at 9:23 AM, Ganesh Ajjanagadde < > gajjanaga...@gmail.com> > > wrote: > >> > >> The rationale for this function is reflected in the d

Re: [FFmpeg-devel] [PATCH 3/5] concatdec: simplify duration calculation in open_next_file

2015-11-13 Thread Nicolas George
Le decadi 20 brumaire, an CCXXIV, Marton Balint a écrit : > If duration is still AV_NOPTS_VALUE when opening the next file, we can assume > that outpoint is not set. > > Signed-off-by: Marton Balint > --- > libavformat/concatdec.c | 9 ++--- > 1 file changed, 2 insertions(+), 7 deletions(-)

Re: [FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-13 Thread Nicolas George
Le tridi 23 brumaire, an CCXXIV, Paul B Mahol a écrit : > This monolithic is incorrect, it is easy to add support for loading > filters its just that it never get commited, there is even patch for this. I do not think you are right. Filters need to use internal APIs to process frames, dynamic load

Re: [FFmpeg-devel] [PATCH 5/5] lavfi/select: add support for concatdec_select option

2015-11-13 Thread Nicolas George
Le decadi 20 brumaire, an CCXXIV, Marton Balint a écrit : > This option can be used to select useful frames from an ffconcat file which is > using inpoints and outpoints but where the source files are not intra frame > only. > > Signed-off-by: Marton Balint > --- > doc/filters.texi | 17 ++

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 9:59 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, Nov 13, 2015 at 9:23 AM, Ganesh Ajjanagadde > wrote: >> >> The rationale for this function is reflected in the documentation for >> it, and is copied here: >> >> Clip a double value into the long long amin-amax range. >> T

Re: [FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-13 Thread Paul B Mahol
On 11/13/15, wm4 wrote: > On Fri, 13 Nov 2015 15:57:22 +0100 > Nicolas George wrote: > >> Le primidi 21 brumaire, an CCXXIV, Clement Boesch a ecrit : >> > You construct something like this: >> > >> > AVAsyncContext( >> > AVAsyncReader( >> > AVAsyncDecoder(...), >> > AVAsyncDecod

Re: [FFmpeg-devel] [PATCH 1/5] fate: add concat demuxer tests

2015-11-13 Thread Nicolas George
Le decadi 20 brumaire, an CCXXIV, Marton Balint a écrit : > Signed-off-by: Marton Balint > --- > tests/Makefile | 1 + > tests/extended.ffconcat| 114 > tests/fate-run.sh | 20 +++

Re: [FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-13 Thread Nicolas George
Le tridi 23 brumaire, an CCXXIV, wm4 a écrit : > libavfilter is too inflexible, brittle, monolithic. Care to elaborate? Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org h

Re: [FFmpeg-devel] [PATCH 4/5] concatdec: add option for adding segment start time and duration metadata

2015-11-13 Thread Nicolas George
Le decadi 20 brumaire, an CCXXIV, Marton Balint a écrit : > Signed-off-by: Marton Balint > --- > doc/demuxers.texi | 8 > libavformat/concatdec.c | 10 ++ > 2 files changed, 18 insertions(+) This one looks fine. Regards, -- Nicolas George signature.asc Description:

Re: [FFmpeg-devel] [PATCH 2/5] concatdec: calculate duration early if outpoint is known

2015-11-13 Thread Nicolas George
Le decadi 20 brumaire, an CCXXIV, Marton Balint a écrit : > Signed-off-by: Marton Balint > --- > libavformat/concatdec.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c > index 0180a7e..560aa64 100644 > --- a/libavformat/concatdec.c >

Re: [FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-13 Thread wm4
On Fri, 13 Nov 2015 15:57:22 +0100 Nicolas George wrote: > Le primidi 21 brumaire, an CCXXIV, Clement Boesch a écrit : > > You construct something like this: > > > > AVAsyncContext( > > AVAsyncReader( > > AVAsyncDecoder(...), > > AVAsyncDecoder(...), > > AVAsyncDecoder(..

Re: [FFmpeg-devel] FFmpeg 2.8.2

2015-11-13 Thread Przemysław Sobala
W dniu 08.11.2015 o 20:23, Michael Niedermayer pisze: Hi I will probably make 2.8.2 soon (days/week), if you want somethig backported, please backport sooner thx Can you include commit 6540fe04a3f9a11ba7084a49b3ee5fa2fc5b32ab libvpxenc: remove some unused ctrl id mappings Author: James Zern

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Fri, Nov 13, 2015 at 9:23 AM, Ganesh Ajjanagadde wrote: > The rationale for this function is reflected in the documentation for > it, and is copied here: > > Clip a double value into the long long amin-amax range. > This function is needed because conversion of floating point to integers

Re: [FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-13 Thread Nicolas George
Le primidi 21 brumaire, an CCXXIV, Clement Boesch a écrit : > You construct something like this: > > AVAsyncContext( > AVAsyncReader( > AVAsyncDecoder(...), > AVAsyncDecoder(...), > AVAsyncDecoder(...), > ... > ), > AVAsyncReader( > AVAsyncDecoder(...),

Re: [FFmpeg-devel] [PATCH] avformat/rtpdec_mpa_robust: change assignment to equality test in conditional

2015-11-13 Thread Ganesh Ajjanagadde
On Wed, Oct 28, 2015 at 10:19 PM, Michael Niedermayer wrote: > On Tue, Oct 27, 2015 at 07:58:03PM -0400, Ganesh Ajjanagadde wrote: >> In the spirit of commit 8199908fdf9b3797cceaea9d1e2fc09d02ef7309, likely >> typo originally. >> >> Found by enabling -Wparentheses on clang 3.7 and running a manual

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 9:33 AM, Hendrik Leppkes wrote: > On Fri, Nov 13, 2015 at 3:23 PM, Ganesh Ajjanagadde > wrote: >> The rationale for this function is reflected in the documentation for >> it, and is copied here: >> >> Clip a double value into the long long amin-amax range. >> This function

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread Hendrik Leppkes
On Fri, Nov 13, 2015 at 3:23 PM, Ganesh Ajjanagadde wrote: > The rationale for this function is reflected in the documentation for > it, and is copied here: > > Clip a double value into the long long amin-amax range. > This function is needed because conversion of floating point to integers when >

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread James Darnley
On 2015-11-13 15:23, Ganesh Ajjanagadde wrote: > diff --git a/libavutil/version.h b/libavutil/version.h > index 909f9a6..ea10ff0 100644 > --- a/libavutil/version.h > +++ b/libavutil/version.h > @@ -56,8 +56,8 @@ > */ > > #define LIBAVUTIL_VERSION_MAJOR 55 > -#define LIBAVUTIL_VERSION_MINOR

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 9:10 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, Nov 13, 2015 at 9:06 AM, Ganesh Ajjanagadde > wrote: >> >> On Fri, Nov 13, 2015 at 8:52 AM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Fri, Nov 13, 2015 at 8:08 AM, Ronald S. Bultje >> > wrote: >> >> >> >> Hi, >> >

Re: [FFmpeg-devel] [PATCH] fate: add mpdecimate test.

2015-11-13 Thread Nicolas George
Le decadi 20 brumaire, an CCXXIV, Nicolas George a écrit : > Thanks for the heads up, I will try to reproduce and understand (the filter > does not touch timestamps!) when time permits. Well, I built with the same options, and it did not fail, I do not know what can cause the difference. If nobody

[FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
The rationale for this function is reflected in the documentation for it, and is copied here: Clip a double value into the long long amin-amax range. This function is needed because conversion of floating point to integers when it does not fit in the integer's representation does not necessarily s

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Fri, Nov 13, 2015 at 9:06 AM, Ganesh Ajjanagadde wrote: > On Fri, Nov 13, 2015 at 8:52 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Fri, Nov 13, 2015 at 8:08 AM, Ronald S. Bultje > > wrote: > >> > >> Hi, > >> > >> On Thu, Nov 12, 2015 at 9:46 PM, Ganesh Ajjanagadde > >> wrote: > >>

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 8:52 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, Nov 13, 2015 at 8:08 AM, Ronald S. Bultje > wrote: >> >> Hi, >> >> On Thu, Nov 12, 2015 at 9:46 PM, Ganesh Ajjanagadde >> wrote: >>> >>> The rationale for this function is reflected in the documentation for >>> it, and is

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Fri, Nov 13, 2015 at 8:08 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, Nov 12, 2015 at 9:46 PM, Ganesh Ajjanagadde < > gajjanaga...@gmail.com> wrote: > >> The rationale for this function is reflected in the documentation for >> it, and is copied here: >> >> Clip a double value into the l

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Thu, Nov 12, 2015 at 9:46 PM, Ganesh Ajjanagadde wrote: > The rationale for this function is reflected in the documentation for > it, and is copied here: > > Clip a double value into the long long amin-amax range. > This function is needed because conversion of floating point to integers

Re: [FFmpeg-devel] [PATCH] avformat/async: Fix bug where async could not recover after seek to eof

2015-11-13 Thread Michael Niedermayer
On Fri, Nov 13, 2015 at 02:36:05AM -0800, Bryan Huh wrote: > Michael, can you merge this? sure, applied thanks > > On Wed, Nov 11, 2015 at 6:47 PM, Zhang Rui wrote: [...] > > Yes, this patch is correct with or without that option. > > I would prefer this patch to be merged first if no other ob

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 7:05 AM, Pedro Arthur wrote: > >> +static av_always_inline av_const int64_t av_rint64_clip_c(double a, >> int64_t amin, int64_t amax) >> +{ >> +#if defined(HAVE_AV_CONFIG_H) && defined(ASSERT_LEVEL) && ASSERT_LEVEL >= >> 2 >> +if (amin > amax) abort(); >> +#endif >> +

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 6:40 AM, Michael Niedermayer wrote: > On Thu, Nov 12, 2015 at 09:46:04PM -0500, Ganesh Ajjanagadde wrote: >> The rationale for this function is reflected in the documentation for >> it, and is copied here: >> >> Clip a double value into the long long amin-amax range. >> Thi

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ganesh Ajjanagadde
On Fri, Nov 13, 2015 at 7:29 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, Nov 12, 2015 at 9:46 PM, Ganesh Ajjanagadde > wrote: >> >> The rationale for this function is reflected in the documentation for >> it, and is copied here: >> >> Clip a double value into the long long amin-amax range. >> T

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Ronald S. Bultje
Hi, On Thu, Nov 12, 2015 at 9:46 PM, Ganesh Ajjanagadde wrote: > The rationale for this function is reflected in the documentation for > it, and is copied here: > > Clip a double value into the long long amin-amax range. > This function is needed because conversion of floating point to integers

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Pedro Arthur
> +static av_always_inline av_const int64_t av_rint64_clip_c(double a, > int64_t amin, int64_t amax) > +{ > +#if defined(HAVE_AV_CONFIG_H) && defined(ASSERT_LEVEL) && ASSERT_LEVEL >= > 2 > +if (amin > amax) abort(); > +#endif > +// INT64_MAX+1,INT64_MIN are exactly representable as IEEE dou

Re: [FFmpeg-devel] [PATCH 1/2] avutil/common: add av_rint64_clip

2015-11-13 Thread Michael Niedermayer
On Thu, Nov 12, 2015 at 09:46:04PM -0500, Ganesh Ajjanagadde wrote: > The rationale for this function is reflected in the documentation for > it, and is copied here: > > Clip a double value into the long long amin-amax range. > This function is needed because conversion of floating point to intege

Re: [FFmpeg-devel] [PATCH] avformat/async: Fix bug where async could not recover after seek to eof

2015-11-13 Thread Bryan Huh
Michael, can you merge this? On Wed, Nov 11, 2015 at 6:47 PM, Zhang Rui wrote: > 2015-11-12 9:45 GMT+08:00 Bryan Huh : > > > > > > On Wed, Nov 11, 2015 at 3:06 AM, Zhang Rui wrote: > >> > >> 2015-11-11 18:00 GMT+08:00 Bryan Huh : > >> > When async issues its inner seek via ffurl_seek, it treats

Re: [FFmpeg-devel] [PATCH] qsv.c: making qsv session routines common for libavcodec and libavfilter modules.

2015-11-13 Thread Ivan Uskov
Hello wm4, Friday, November 13, 2015, 11:28:46 AM, you wrote: >> HL> avutil is not a place we dump stuff that should be shared between >> HL> avcodec and avfilter, or similar, avutil should generally be public >> HL> useful API. >> I believe 3 of 4 functions here can be published as useful API.

Re: [FFmpeg-devel] [PATCH] qsv.c: making qsv session routines common for libavcodec and libavfilter modules.

2015-11-13 Thread wm4
On Fri, 13 Nov 2015 10:57:59 +0300 Ivan Uskov wrote: > Hello Hendrik, > > Friday, November 13, 2015, 12:55:38 AM, you wrote: > > HL> On Thu, Nov 12, 2015 at 10:15 PM, Ivan Uskov > wrote: > >> Hello All, > >> > >> As it turned out, a suggested recently libavfilter/vf_qsv_vpp.c may not >