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] [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] [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] [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] [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] [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