[Ffmpeg-cvslog] r6324 - trunk/libavcodec/eval.c

[Panagiotis Issaris]

Hi,

On za, 2006-09-23 at 10:56 +0200, Reimar D?ffinger wrote:
> Hello,
> On Sat, Sep 23, 2006 at 10:50:50AM +0200, takis wrote:
> >  #ifndef NAN
> > -  #define NAN 0
> > +  #define NAN 0.0/0.0
> 
> As a better-safe-than-sorry approach I'd suggest adding () around
> that...
I initially wanted to do that... but then I figured using NAN in any
other way then direct assignments or returns makes no sense/is
undefined.

I mean stuff like this, in which operator precendence would matter,
makes no sense:
double f = ++NAN; 
or
return ++NAN;

Ofcourse if NAN turns up in a variable after operators, it could happen,
but then there's no issue with the NAN define:
double f;
... /* f turns into NAN somewhere */
return ++f;

With friendly regards,
Takis