Re: [FFmpeg-devel] [PATCH 3/3] lavu/rational: add more info regarding floor(x+0.5) usage

2016-02-29 Thread Ganesh Ajjanagadde
On Thu, Feb 25, 2016 at 1:11 PM, Michael Niedermayer wrote: > On Wed, Feb 24, 2016 at 09:20:11PM -0500, Ganesh Ajjanagadde wrote: >> Add some more verbose info regarding why the imprecise and slow floor(x+0.5) >> hack >> is used; helpful for future maintenance. >> >>

Re: [FFmpeg-devel] [PATCH 3/3] lavu/rational: add more info regarding floor(x+0.5) usage

2016-02-25 Thread Michael Niedermayer
On Wed, Feb 24, 2016 at 09:20:11PM -0500, Ganesh Ajjanagadde wrote: > Add some more verbose info regarding why the imprecise and slow floor(x+0.5) > hack > is used; helpful for future maintenance. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/rational.c | 3 ++- >

Re: [FFmpeg-devel] [PATCH 3/3] lavu/rational: add more info regarding floor(x+0.5) usage

2016-02-25 Thread Derek Buitenhuis
On 2/25/2016 2:20 AM, Ganesh Ajjanagadde wrote: > -// (int64_t)rint() and llrint() do not work with gcc on ia64 and sparc64 > +// (int64_t)rint() and llrint() do not work with gcc on ia64 and sparc64, > +// see Ticket2713 for affected gcc/glibc versions I've never seen this comment

[FFmpeg-devel] [PATCH 3/3] lavu/rational: add more info regarding floor(x+0.5) usage

2016-02-24 Thread Ganesh Ajjanagadde
Add some more verbose info regarding why the imprecise and slow floor(x+0.5) hack is used; helpful for future maintenance. Signed-off-by: Ganesh Ajjanagadde --- libavutil/rational.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavutil/rational.c