[Boost-commit] svn:boost r57494 - trunk/libs/utility

Subject: [Boost-commit] svn:boost r57494 - trunk/libs/utility
From: john_at_[hidden]
Date: 2009-11-08 13:54:00

Author: johnmaddock
Date: 2009-11-08 13:53:59 EST (Sun, 08 Nov 2009)
New Revision: 57494
URL: http://svn.boost.org/trac/boost/changeset/57494

Log:
Suppress/fix some msvc and gcc compiler warnings.
Text files modified: 
   trunk/libs/utility/base_from_member_test.cpp |     2 +-                                      
   trunk/libs/utility/binary_test.cpp           |     4 ++++                                    
   trunk/libs/utility/call_traits_test.cpp      |    11 +++++++++--                             
   trunk/libs/utility/ref_test.cpp              |     2 +-                                      
   trunk/libs/utility/value_init_test.cpp       |     6 +++---                                  
   5 files changed, 18 insertions(+), 7 deletions(-)

Modified: trunk/libs/utility/base_from_member_test.cpp
==============================================================================
--- trunk/libs/utility/base_from_member_test.cpp	(original)
+++ trunk/libs/utility/base_from_member_test.cpp	2009-11-08 13:53:59 EST (Sun, 08 Nov 2009)
@@ -281,7 +281,7 @@
         }
         else
         {
-            return a.second->before( *b.second );
+            return a.second->before( *b.second ) != 0;
         }
     }
 }

Modified: trunk/libs/utility/binary_test.cpp
==============================================================================
--- trunk/libs/utility/binary_test.cpp	(original)
+++ trunk/libs/utility/binary_test.cpp	2009-11-08 13:53:59 EST (Sun, 08 Nov 2009)
@@ -11,6 +11,10 @@
 #include <algorithm>
 #include <cstddef>
 
+#ifdef BOOST_MSVC
+#pragma warning(disable:4996) // warning C4996: 'std::equal': Function call with parameters that may be unsafe - this call relies on the caller to check that the passed values are correct. To disable this warning, use -D_SCL_SECURE_NO_WARNINGS. See documentation on how to use Visual C++ 'Checked Iterators'
+#endif
+
 /*
 Note: This file tests every single valid bit-grouping on its own, and some
       random combinations of bit-groupings.

Modified: trunk/libs/utility/call_traits_test.cpp
==============================================================================
--- trunk/libs/utility/call_traits_test.cpp	(original)
+++ trunk/libs/utility/call_traits_test.cpp	2009-11-08 13:53:59 EST (Sun, 08 Nov 2009)
@@ -21,6 +21,10 @@
 #include <libs/type_traits/test/test.hpp>
 #include <libs/type_traits/test/check_type.hpp>
 
+#ifdef BOOST_MSVC
+#pragma warning(disable:4181) // : warning C4181: qualifier applied to reference type; ignored
+#endif
+
 // a way prevent warnings for unused variables
 template<class T> inline void unused_variable(const T&) {}
 
@@ -52,7 +56,8 @@
    const_reference const_get()const { return v_; }
    // pass value:
    void call(param_type){}
-
+private:
+   contained& operator=(const contained&);
 };
 
 #ifndef BOOST_NO_TEMPLATE_PARTIAL_SPECIALIZATION
@@ -77,6 +82,8 @@
    reference get() { return v_; }
    const_reference const_get()const { return v_; }
    void call(param_type){}
+private:
+   contained& operator=(const contained&);
 };
 #endif
 
@@ -197,7 +204,7 @@
    bool operator == (const comparible_UDT& v){ return v.i_ == i_; }
 };
 
-int main(int argc, char *argv[ ])
+int main()
 {
    call_traits_checker<comparible_UDT> c1;
    comparible_UDT u;

Modified: trunk/libs/utility/ref_test.cpp
==============================================================================
--- trunk/libs/utility/ref_test.cpp	(original)
+++ trunk/libs/utility/ref_test.cpp	2009-11-08 13:53:59 EST (Sun, 08 Nov 2009)
@@ -70,7 +70,7 @@
 
 struct copy_counter {
   static int count_;
-  copy_counter(copy_counter const& other) {
+  copy_counter(copy_counter const& /*other*/) {
     ++count_;
   }
   copy_counter() {}

Modified: trunk/libs/utility/value_init_test.cpp
==============================================================================
--- trunk/libs/utility/value_init_test.cpp	(original)
+++ trunk/libs/utility/value_init_test.cpp	2009-11-08 13:53:59 EST (Sun, 08 Nov 2009)
@@ -29,9 +29,9 @@
 //
 struct POD
 {
-  POD () : c(0), i(0), f(0) {}
+  POD () : f(0), c(0), i(0){}
 
-  POD ( char c_, int i_, float f_ ) : c(c_), i(i_), f(f_) {}
+  POD ( char c_, int i_, float f_ ) : f(f_), c(c_), i(i_) {}
 
   friend std::ostream& operator << ( std::ostream& os, POD const& pod )
     { return os << '(' << pod.c << ',' << pod.i << ',' << pod.f << ')' ; }
@@ -291,7 +291,7 @@
 {
   BOOST_CHECK ( test( 0,1234 ) ) ;
   BOOST_CHECK ( test( 0.0,12.34 ) ) ;
-  BOOST_CHECK ( test( POD(0,0,0.0), POD('a',1234,56.78) ) ) ;
+  BOOST_CHECK ( test( POD(0,0,0.0), POD('a',1234,56.78f) ) ) ;
   BOOST_CHECK ( test( NonPOD( std::string() ), NonPOD( std::string("something") ) ) ) ;
 
   NonPOD NonPOD_object( std::string("NonPOD_object") );