Re: [FFmpeg-devel] [PATCHv2] avutil/libm: correct isnan, isinf compat hacks

2015-11-24 Thread Ganesh Ajjanagadde
On Tue, Nov 24, 2015 at 8:48 PM, Ganesh Ajjanagadde wrote: > On Tue, Nov 24, 2015 at 6:53 PM, Michael Niedermayer wrote: >> On Tue, Nov 24, 2015 at 09:17:18AM +0100, Hendrik Leppkes wrote: >>> On Sat, Nov 21, 2015 at 2:53 PM, Ganesh Ajjanagadde >>> wrote: >>> > On Wed, Nov 18, 2015 at 3:04 PM,

Re: [FFmpeg-devel] [PATCHv2] avutil/libm: correct isnan, isinf compat hacks

2015-11-24 Thread Ganesh Ajjanagadde
On Tue, Nov 24, 2015 at 6:53 PM, Michael Niedermayer wrote: > On Tue, Nov 24, 2015 at 09:17:18AM +0100, Hendrik Leppkes wrote: >> On Sat, Nov 21, 2015 at 2:53 PM, Ganesh Ajjanagadde wrote: >> > On Wed, Nov 18, 2015 at 3:04 PM, Ganesh Ajjanagadde >> > wrote: >> >> On Wed, Nov 18, 2015 at 2:58 PM

Re: [FFmpeg-devel] [PATCHv2] avutil/libm: correct isnan, isinf compat hacks

2015-11-24 Thread Michael Niedermayer
On Tue, Nov 24, 2015 at 09:17:18AM +0100, Hendrik Leppkes wrote: > On Sat, Nov 21, 2015 at 2:53 PM, Ganesh Ajjanagadde wrote: > > On Wed, Nov 18, 2015 at 3:04 PM, Ganesh Ajjanagadde > > wrote: > >> On Wed, Nov 18, 2015 at 2:58 PM, Michael Niedermayer > >> wrote: > >>> On Tue, Nov 17, 2015 at 0

Re: [FFmpeg-devel] [PATCHv2] avutil/libm: correct isnan, isinf compat hacks

2015-11-24 Thread Hendrik Leppkes
On Sat, Nov 21, 2015 at 2:53 PM, Ganesh Ajjanagadde wrote: > On Wed, Nov 18, 2015 at 3:04 PM, Ganesh Ajjanagadde wrote: >> On Wed, Nov 18, 2015 at 2:58 PM, Michael Niedermayer >> wrote: >>> On Tue, Nov 17, 2015 at 04:54:10PM -0500, Ganesh Ajjanagadde wrote: isnan and isinf are actually mac

Re: [FFmpeg-devel] [PATCHv2] avutil/libm: correct isnan, isinf compat hacks

2015-11-21 Thread Ganesh Ajjanagadde
On Wed, Nov 18, 2015 at 3:04 PM, Ganesh Ajjanagadde wrote: > On Wed, Nov 18, 2015 at 2:58 PM, Michael Niedermayer wrote: >> On Tue, Nov 17, 2015 at 04:54:10PM -0500, Ganesh Ajjanagadde wrote: >>> isnan and isinf are actually macros as per the standard. In particular, >>> the existing implementati

Re: [FFmpeg-devel] [PATCHv2] avutil/libm: correct isnan, isinf compat hacks

2015-11-18 Thread Ganesh Ajjanagadde
On Wed, Nov 18, 2015 at 2:58 PM, Michael Niedermayer wrote: > On Tue, Nov 17, 2015 at 04:54:10PM -0500, Ganesh Ajjanagadde wrote: >> isnan and isinf are actually macros as per the standard. In particular, >> the existing implementation has incorrect signature. Furthermore, this >> results in undef

Re: [FFmpeg-devel] [PATCHv2] avutil/libm: correct isnan, isinf compat hacks

2015-11-18 Thread Michael Niedermayer
On Tue, Nov 17, 2015 at 04:54:10PM -0500, Ganesh Ajjanagadde wrote: > isnan and isinf are actually macros as per the standard. In particular, > the existing implementation has incorrect signature. Furthermore, this > results in undefined behavior for e.g double values outside float range > as per t