Re: [FFmpeg-devel] [PATCH 11/11] avutil/rational: use frexp rather than ad-hoc log to get floating point exponent

2015-10-30 Thread Ganesh Ajjanagadde
On Fri, Oct 30, 2015 at 7:14 PM, Michael Niedermayer wrote: > On Thu, Oct 29, 2015 at 12:20:09AM -0400, Ganesh Ajjanagadde wrote: >> This simplifies and cleans up the code. >> Furthermore, it is much faster due to absence of the slow log computation. >> >> Signed-off-by: Ganesh Ajjanagadde >> ---

Re: [FFmpeg-devel] [PATCH 11/11] avutil/rational: use frexp rather than ad-hoc log to get floating point exponent

2015-10-30 Thread Michael Niedermayer
On Thu, Oct 29, 2015 at 12:20:09AM -0400, Ganesh Ajjanagadde wrote: > This simplifies and cleans up the code. > Furthermore, it is much faster due to absence of the slow log computation. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/rational.c | 4 ++-- > 1 file changed, 2 insertions(+

Re: [FFmpeg-devel] [PATCH 11/11] avutil/rational: use frexp rather than ad-hoc log to get floating point exponent

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 12:20 AM, Ganesh Ajjanagadde wrote: > This simplifies and cleans up the code. > Furthermore, it is much faster due to absence of the slow log computation. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/rational.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 del

[FFmpeg-devel] [PATCH 11/11] avutil/rational: use frexp rather than ad-hoc log to get floating point exponent

2015-10-28 Thread Ganesh Ajjanagadde
This simplifies and cleans up the code. Furthermore, it is much faster due to absence of the slow log computation. Signed-off-by: Ganesh Ajjanagadde --- libavutil/rational.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavutil/rational.c b/libavutil/rational.c index