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

2015-11-02 Thread Ganesh Ajjanagadde
On Mon, Nov 2, 2015 at 1:59 PM, Daniel Serpell wrote: > Hi, > > On Sun, Nov 01, 2015 at 01:03:35PM -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. >> T

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

2015-11-02 Thread Daniel Serpell
Hi, On Sun, Nov 01, 2015 at 01:03:35PM -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 i

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

2015-11-02 Thread Daniel Serpell
Hi, On Sun, Nov 01, 2015 at 01:03:35PM -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 i

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

2015-11-01 Thread Ganesh Ajjanagadde
On Sun, Nov 1, 2015 at 6:09 PM, Ronald S. Bultje wrote: > Hi, > > On Sun, Nov 1, 2015 at 1:03 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. >> Thi

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

2015-11-01 Thread Ronald S. Bultje
Hi, On Sun, Nov 1, 2015 at 1:03 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 >

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

2015-11-01 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