-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Karl Schultz wrote:
> Am getting these warnings in the Windows build:
> 
> 2>Compiling...
> 2>lex.yy.c
> 2>program_lexer.l(327) : warning C4244: '=' : conversion from 'double'
> to 'float', possible loss of data
> 2>program_lexer.l(331) : warning C4244: '=' : conversion from 'double'
> to 'float', possible loss of data
> 2>program_lexer.l(335) : warning C4244: '=' : conversion from 'double'
> to 'float', possible loss of data
> 2>program_lexer.l(339) : warning C4244: '=' : conversion from 'double'
> to 'float', possible loss of data

[snip]

> If it is not a huge hassle, can someone add these casts to 7.8 and
> master?  The warnings are no big deal, but the casts do have some value
> in documenting the implicit conversion.

Absolutely not.  All the casts do is clutter the code.  This is a
completely worthless warning from Visual Studio.  It generates only
noise.  I have never seen a case where it was actually useful.  There
are pragmas and command line options to disable specific warnings.  You
should do that.

Brian, could you please revert those patches.  I don't want that crap in
code that I maintain.  Seriously.

Thanks.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAkuanyEACgkQX1gOwKyEAw/PJwCgjZDqwdLrkC5hZ28hCgLpdmvH
anYAnAm2rMb2fSgnOjsqRSybuhQ1ZmOA
=cRwl
-----END PGP SIGNATURE-----

------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to