Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ganesh Ajjanagadde
On Sat, Dec 19, 2015 at 1:56 PM, Ganesh Ajjanagadde wrote: > On Sat, Dec 19, 2015 at 1:31 PM, Paul B Mahol wrote: > [...] >> C90 is more than 20 years old. > > Of course, but this was raised by Hendrik and Ronald regarding > compliance and FFmpeg's coding guidelines. > This also highlights how lu

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ganesh Ajjanagadde
On Sat, Dec 19, 2015 at 1:31 PM, Paul B Mahol wrote: [...] > C90 is more than 20 years old. Of course, but this was raised by Hendrik and Ronald regarding compliance and FFmpeg's coding guidelines. This also highlights how ludicruous it is in 2015, soon to be 2016. >

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ganesh Ajjanagadde
On Sat, Dec 19, 2015 at 1:33 PM, Hendrik Leppkes wrote: > On Sat, Dec 19, 2015 at 10:28 PM, Ganesh Ajjanagadde wrote: >> On Sat, Dec 19, 2015 at 12:33 PM, Ronald S. Bultje >> wrote: >>> Hi, >>> >>> On Sat, Dec 19, 2015 at 2:25 PM, Ganesh Ajjanagadde >>> wrote: >>> On Sat, Dec 19, 2015 at

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Hendrik Leppkes
On Sat, Dec 19, 2015 at 10:28 PM, Ganesh Ajjanagadde wrote: > On Sat, Dec 19, 2015 at 12:33 PM, Ronald S. Bultje wrote: >> Hi, >> >> On Sat, Dec 19, 2015 at 2:25 PM, Ganesh Ajjanagadde >> wrote: >> >>> On Sat, Dec 19, 2015 at 11:05 AM, Hendrik Leppkes >>> wrote: >>> > On Sat, Dec 19, 2015 at 6:

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Paul B Mahol
On 12/19/15, Ganesh Ajjanagadde wrote: > On Sat, Dec 19, 2015 at 12:33 PM, Ronald S. Bultje > wrote: >> Hi, >> >> On Sat, Dec 19, 2015 at 2:25 PM, Ganesh Ajjanagadde >> wrote: >> >>> On Sat, Dec 19, 2015 at 11:05 AM, Hendrik Leppkes >>> wrote: >>> > On Sat, Dec 19, 2015 at 6:54 PM, Ganesh Ajjan

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ganesh Ajjanagadde
On Sat, Dec 19, 2015 at 12:33 PM, Ronald S. Bultje wrote: > Hi, > > On Sat, Dec 19, 2015 at 2:25 PM, Ganesh Ajjanagadde > wrote: > >> On Sat, Dec 19, 2015 at 11:05 AM, Hendrik Leppkes >> wrote: >> > On Sat, Dec 19, 2015 at 6:54 PM, Ganesh Ajjanagadde >> > wrote: >> >> On Sat, Dec 19, 2015 at 8:

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ganesh Ajjanagadde
On Sat, Dec 19, 2015 at 12:33 PM, Ronald S. Bultje wrote: > Hi, > > On Sat, Dec 19, 2015 at 2:25 PM, Ganesh Ajjanagadde > wrote: > >> On Sat, Dec 19, 2015 at 11:05 AM, Hendrik Leppkes >> wrote: >> > On Sat, Dec 19, 2015 at 6:54 PM, Ganesh Ajjanagadde >> > wrote: >> >> On Sat, Dec 19, 2015 at 8:

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ronald S. Bultje
Hi, On Sat, Dec 19, 2015 at 2:25 PM, Ganesh Ajjanagadde wrote: > On Sat, Dec 19, 2015 at 11:05 AM, Hendrik Leppkes > wrote: > > On Sat, Dec 19, 2015 at 6:54 PM, Ganesh Ajjanagadde > > wrote: > >> On Sat, Dec 19, 2015 at 8:06 AM, Ganesh Ajjanagadde > >> wrote: > >>> On Sat, Dec 19, 2015 at 6:2

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ganesh Ajjanagadde
On Sat, Dec 19, 2015 at 11:05 AM, Hendrik Leppkes wrote: > On Sat, Dec 19, 2015 at 6:54 PM, Ganesh Ajjanagadde > wrote: >> On Sat, Dec 19, 2015 at 8:06 AM, Ganesh Ajjanagadde >> wrote: >>> On Sat, Dec 19, 2015 at 6:26 AM, Ronald S. Bultje >>> wrote: Hi, On Fri, Dec 18, 2015 at 3

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Hendrik Leppkes
On Sat, Dec 19, 2015 at 6:54 PM, Ganesh Ajjanagadde wrote: > On Sat, Dec 19, 2015 at 8:06 AM, Ganesh Ajjanagadde > wrote: >> On Sat, Dec 19, 2015 at 6:26 AM, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Fri, Dec 18, 2015 at 3:47 PM, Ganesh Ajjanagadde >>> wrote: On Fri, Dec 18, 2015 at

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ganesh Ajjanagadde
On Sat, Dec 19, 2015 at 8:06 AM, Ganesh Ajjanagadde wrote: > On Sat, Dec 19, 2015 at 6:26 AM, Ronald S. Bultje wrote: >> Hi, >> >> On Fri, Dec 18, 2015 at 3:47 PM, Ganesh Ajjanagadde >> wrote: >>> >>> On Fri, Dec 18, 2015 at 12:41 PM, Ronald S. Bultje >>> wrote: >>> > Hi, >>> > >>> > On Fri, De

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ganesh Ajjanagadde
On Sat, Dec 19, 2015 at 6:26 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, Dec 18, 2015 at 3:47 PM, Ganesh Ajjanagadde > wrote: >> >> On Fri, Dec 18, 2015 at 12:41 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Fri, Dec 18, 2015 at 2:18 PM, Ganesh Ajjanagadde >> > >> > wrote: >> >> >> >>

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-19 Thread Ronald S. Bultje
Hi, On Fri, Dec 18, 2015 at 3:47 PM, Ganesh Ajjanagadde wrote: > On Fri, Dec 18, 2015 at 12:41 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Fri, Dec 18, 2015 at 2:18 PM, Ganesh Ajjanagadde < > gajjanaga...@gmail.com> > > wrote: > >> > >> For systems with broken libms. > >> Tested with NAN,

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-18 Thread Ganesh Ajjanagadde
On Fri, Dec 18, 2015 at 12:41 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Dec 18, 2015 at 2:18 PM, Ganesh Ajjanagadde > wrote: >> >> For systems with broken libms. >> Tested with NAN, -NAN, INFINITY, -INFINITY, +/-x for regular double x and >> combinations of these. >> >> Signed-off-by: Ganesh

Re: [FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-18 Thread Ronald S. Bultje
Hi, On Fri, Dec 18, 2015 at 2:18 PM, Ganesh Ajjanagadde wrote: > For systems with broken libms. > Tested with NAN, -NAN, INFINITY, -INFINITY, +/-x for regular double x and > combinations of these. > > Signed-off-by: Ganesh Ajjanagadde > --- > configure| 2 +- > libavutil/libm.h | 9 +++

[FFmpeg-devel] [PATCH] lavu/libm: add copysign hack

2015-12-18 Thread Ganesh Ajjanagadde
For systems with broken libms. Tested with NAN, -NAN, INFINITY, -INFINITY, +/-x for regular double x and combinations of these. Signed-off-by: Ganesh Ajjanagadde --- configure| 2 +- libavutil/libm.h | 9 + 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/configure