[functional/hash] VC8.0 compiler warning

[Richard Webb <richard.webb@…>]

When i build my project in VC8.0 with warning level 4 and code analysis enabled, i get the following warning:

44>e:\development\safemail32\external\boost\boost\functional\detail\hash_float.hpp(121) : warning C6294: Ill-defined for-loop: initial condition does not satisfy test. Loop body not executed
44>        E:\Development\SAFEmail32\External\Boost\boost/functional/detail/hash_float.hpp(178) : see reference to function template instantiation 'size_t boost::hash_detail::float_hash_impl<T>(T)' being compiled
44>        with
44>        [
44>            T=float
44>        ]
44>        E:\Development\SAFEmail32\External\Boost\boost/functional/hash/hash.hpp(304) : see reference to function template instantiation 'size_t boost::hash_detail::float_hash_value<float>(T)' being compiled
44>        with
44>        [
44>            T=float
44>        ]

[Daniel James]

(In [41864]) Try to suppress a Visual C++ warning. Refs #1509

[Daniel James]

Thanks for reporting this. Please can you check if the above change has fixed it. Also, can you let me know which flags are required to get this warning.

[Richard Webb <richard.webb@…>]

C6294 is a code analysis warning. I can get it by simply creating a new project, including functional/hash.hpp, and building with the code analysis option enabled (only available in some versions of the compiler - you might not be able to reproduce it with the Express edition).

The change has fixed the warning on VC8, but has also resulted in a new warning on VC7.1 (as C6294 doesnt exist in that version of the compiler).

I think that it only needs to be disabled for VC8/9?

[Daniel James]

(In [41866]) Only suppress warning for Visual C++ 8 and later. Refs #1509

[Eric Niebler]

(In [41916]) Merged revisions 41816-41915 via svnmerge from ​https://svn.boost.org/svn/boost/trunk

........

r41819 | anthonyw | 2007-12-07 00:11:11 -0800 (Fri, 07 Dec 2007) | 1 line

fixed order of comparison in timeout check

........

r41823 | chris_kohlhoff | 2007-12-07 04:53:39 -0800 (Fri, 07 Dec 2007) | 2 lines

Try to fix stall when sending large amounts of data over SSL.

........

r41830 | andreas_huber69 | 2007-12-07 11:03:46 -0800 (Fri, 07 Dec 2007) | 1 line

Added markup for various statechart failures.

........

r41842 | dave | 2007-12-07 15:23:39 -0800 (Fri, 07 Dec 2007) | 2 lines

Fix msvc auto-configuration (yikes!)

........

r41845 | chris_kohlhoff | 2007-12-07 16:18:59 -0800 (Fri, 07 Dec 2007) | 2 lines

Documentation fixes.

........

r41863 | andreas_huber69 | 2007-12-08 02:05:41 -0800 (Sat, 08 Dec 2007) | 1 line

Added markup for various statechart failures.

........

r41864 | danieljames | 2007-12-08 03:04:40 -0800 (Sat, 08 Dec 2007) | 2 lines

Try to suppress a Visual C++ warning. Refs #1509

........

r41866 | danieljames | 2007-12-08 04:24:36 -0800 (Sat, 08 Dec 2007) | 2 lines

Only suppress warning for Visual C++ 8 and later. Refs #1509

........

r41867 | chris_kohlhoff | 2007-12-08 05:00:45 -0800 (Sat, 08 Dec 2007) | 2 lines

Documentation fixes.

........

r41868 | chris_kohlhoff | 2007-12-08 05:48:52 -0800 (Sat, 08 Dec 2007) | 2 lines

Suppress signed/unsigned warning.

........

r41870 | chris_kohlhoff | 2007-12-08 06:03:40 -0800 (Sat, 08 Dec 2007) | 2 lines

Ensure asio header comes before boost.thread header.

........

r41889 | johnmaddock | 2007-12-08 08:04:54 -0800 (Sat, 08 Dec 2007) | 2 lines

Removed double quoting of Doxygen title. Fixes: ​http://lists.boost.org/boost-users/2007/12/32523.php

........

r41907 | jhunold | 2007-12-08 12:02:33 -0800 (Sat, 08 Dec 2007) | 2 lines

Add support for QtScript library.

........

r41908 | jhunold | 2007-12-08 12:08:29 -0800 (Sat, 08 Dec 2007) | 2 lines

Add dependency scanner for Qt Resource Files (.qrc).

........

[Richard Webb <richard.webb@…>]

With the latest change the original warning is gone in VC8/9, and there is no longer an issue with VC7.1.

Thanks for the fix.

[Daniel James]

(In [41920]) Merge fix to suppress a Visual C++ warning. Fixes #1509