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

2015-11-14 Thread Ganesh Ajjanagadde
On Sat, Nov 14, 2015 at 6:30 PM, Hendrik Leppkes wrote: > On Sat, Nov 14, 2015 at 10:38 PM, Ganesh Ajjanagadde wrote: >> On Sat, Nov 14, 2015 at 4:30 PM, Hendrik Leppkes wrote: >>> On Sat, Nov 14, 2015 at 10:27 PM, Ganesh Ajjanagadde >>> wrote: On Sat, Nov 14, 2015 at 4:03 PM, Ganesh Ajja

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

2015-11-14 Thread Hendrik Leppkes
On Sat, Nov 14, 2015 at 10:38 PM, Ganesh Ajjanagadde wrote: > On Sat, Nov 14, 2015 at 4:30 PM, Hendrik Leppkes wrote: >> On Sat, Nov 14, 2015 at 10:27 PM, Ganesh Ajjanagadde >> wrote: >>> On Sat, Nov 14, 2015 at 4:03 PM, Ganesh Ajjanagadde >>> wrote: On Sat, Nov 14, 2015 at 3:28 PM, Hend

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

2015-11-14 Thread Ganesh Ajjanagadde
On Sat, Nov 14, 2015 at 4:30 PM, Hendrik Leppkes wrote: > On Sat, Nov 14, 2015 at 10:27 PM, Ganesh Ajjanagadde wrote: >> On Sat, Nov 14, 2015 at 4:03 PM, Ganesh Ajjanagadde wrote: >>> On Sat, Nov 14, 2015 at 3:28 PM, Hendrik Leppkes >>> wrote: On Sat, Nov 14, 2015 at 3:51 AM, Ganesh Ajjan

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

2015-11-14 Thread Ganesh Ajjanagadde
On Sat, Nov 14, 2015 at 4:35 PM, James Almer wrote: > On 11/14/2015 6:30 PM, Hendrik Leppkes wrote: >> On Sat, Nov 14, 2015 at 10:27 PM, Ganesh Ajjanagadde >> wrote: >>> On Sat, Nov 14, 2015 at 4:03 PM, Ganesh Ajjanagadde >>> wrote: On Sat, Nov 14, 2015 at 3:28 PM, Hendrik Leppkes

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

2015-11-14 Thread James Almer
On 11/14/2015 6:30 PM, Hendrik Leppkes wrote: > On Sat, Nov 14, 2015 at 10:27 PM, Ganesh Ajjanagadde wrote: >> On Sat, Nov 14, 2015 at 4:03 PM, Ganesh Ajjanagadde wrote: >>> On Sat, Nov 14, 2015 at 3:28 PM, Hendrik Leppkes >>> wrote: On Sat, Nov 14, 2015 at 3:51 AM, Ganesh Ajjanagadde >>

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

2015-11-14 Thread Hendrik Leppkes
On Sat, Nov 14, 2015 at 10:27 PM, Ganesh Ajjanagadde wrote: > On Sat, Nov 14, 2015 at 4:03 PM, Ganesh Ajjanagadde wrote: >> On Sat, Nov 14, 2015 at 3:28 PM, Hendrik Leppkes wrote: >>> On Sat, Nov 14, 2015 at 3:51 AM, Ganesh Ajjanagadde >>> wrote: On Fri, Nov 13, 2015 at 7:17 PM, Ronald S.

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

2015-11-14 Thread Ganesh Ajjanagadde
On Sat, Nov 14, 2015 at 4:03 PM, Ganesh Ajjanagadde wrote: > On Sat, Nov 14, 2015 at 3:28 PM, Hendrik Leppkes wrote: >> On Sat, Nov 14, 2015 at 3:51 AM, Ganesh Ajjanagadde wrote: >>> On Fri, Nov 13, 2015 at 7:17 PM, Ronald S. Bultje >>> wrote: Hi, On Fri, Nov 13, 2015 at 6:16 PM

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

2015-11-14 Thread Ganesh Ajjanagadde
On Sat, Nov 14, 2015 at 3:28 PM, Hendrik Leppkes wrote: > On Sat, Nov 14, 2015 at 3:51 AM, Ganesh Ajjanagadde wrote: >> 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 P

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

2015-11-14 Thread Hendrik Leppkes
On Sat, Nov 14, 2015 at 3:51 AM, Ganesh Ajjanagadde wrote: > 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

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

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