Remove unused paramters.

jhunold · jhunold · commit 5cbb539ec583 · 2007-12-01T20:26:37.000Z
Add -Wextra to gcc flags to enable more warnings.


[SVN r41550]
diff --git a/include/boost/python/detail/result.hpp b/include/boost/python/detail/result.hpp
@@ -95,7 +95,7 @@ result(X const&, short = 0) { return 0; }
 # define N BOOST_PP_ITERATION()
 
 template <class R BOOST_PP_ENUM_TRAILING_PARAMS_Z(1, N, class A)>
-boost::type<R>* result(R (*pf)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)), int = 0)
+boost::type<R>* result(R (*)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)), int = 0)
 {
     return 0;
 }
@@ -120,7 +120,7 @@ boost::type<R>* result(R (*pf)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)), int = 0)
 # define Q BOOST_PYTHON_CV_QUALIFIER(BOOST_PP_RELATIVE_ITERATION(1))
 
 template <class R, class T BOOST_PP_ENUM_TRAILING_PARAMS_Z(1, N, class A)>
-boost::type<R>* result(R (T::*pmf)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)) Q, int = 0)
+boost::type<R>* result(R (T::*)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)) Q, int = 0)
 {
     return 0;
 }
diff --git a/include/boost/python/detail/unwind_type.hpp b/include/boost/python/detail/unwind_type.hpp
@@ -149,7 +149,7 @@ struct unwind_helper2<reference_to_pointer_>
 template <class Generator, class U>
 inline typename Generator::result_type
 #ifndef _MSC_VER
-unwind_type(boost::type<U>*p, Generator*)
+unwind_type(boost::type<U>*, Generator*)
 #else
 unwind_type(boost::type<U>*p =0, Generator* =0)
 #endif
diff --git a/include/boost/python/to_python_value.hpp b/include/boost/python/to_python_value.hpp
@@ -36,7 +36,7 @@ template <bool is_const_ref>
 struct object_manager_get_pytype
 {
    template <class U>
-   static PyTypeObject const* get( U& (*p)() =0)
+   static PyTypeObject const* get( U& (*)() =0)
    {
       return converter::object_manager_traits<U>::get_pytype();
    }
@@ -46,7 +46,7 @@ template <>
 struct object_manager_get_pytype<true>
 {
    template <class U>
-   static PyTypeObject const* get( U const& (*p)() =0)
+   static PyTypeObject const* get( U const& (*)() =0)
    {
       return converter::object_manager_traits<U>::get_pytype();
    }
diff --git a/test/Jamfile.v2 b/test/Jamfile.v2
@@ -5,7 +5,10 @@
 import python ;
 
 use-project /boost/python : ../build ;
-project /boost/python/test ;
+project /boost/python/test 
+  : requirements
+    <toolset>gcc:<cxxflags>-Wextra
+  ;
 
 local PY = ;
 if [ python.configured ]
diff --git a/test/borrowed.cpp b/test/borrowed.cpp
@@ -9,13 +9,13 @@
 using namespace boost::python;
 
 template <class T>
-void assert_borrowed_ptr(T const& x)
+void assert_borrowed_ptr(T const&)
 {
     BOOST_STATIC_ASSERT(boost::python::detail::is_borrowed_ptr<T>::value);
 }
     
 template <class T>
-void assert_not_borrowed_ptr(T const& x)
+void assert_not_borrowed_ptr(T const&)
 {
     BOOST_STATIC_ASSERT(!boost::python::detail::is_borrowed_ptr<T>::value);
 }
diff --git a/test/operators_wrapper.cpp b/test/operators_wrapper.cpp
@@ -5,10 +5,10 @@ struct vector
 {
     virtual ~vector() {}
     
-    vector operator+( const vector& x ) const
+    vector operator+( const vector& ) const
     { return vector(); }
 
-    vector& operator+=( const vector& x )
+    vector& operator+=( const vector& )
     { return *this; }
     
     vector operator-() const

Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ result(X const&, short = 0) { return 0; }`
`95`	`95`	`# define N BOOST_PP_ITERATION()`
`96`	`96`
`97`	`97`	`template <class R BOOST_PP_ENUM_TRAILING_PARAMS_Z(1, N, class A)>`
`98`		`-boost::type<R>* result(R (*pf)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)), int = 0)`
	`98`	`+boost::type<R>* result(R (*)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)), int = 0)`
`99`	`99`	`{`
`100`	`100`	`return 0;`
`101`	`101`	`}`
`@@ -120,7 +120,7 @@ boost::type<R>* result(R (*pf)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)), int = 0)`
`120`	`120`	`# define Q BOOST_PYTHON_CV_QUALIFIER(BOOST_PP_RELATIVE_ITERATION(1))`
`121`	`121`
`122`	`122`	`template <class R, class T BOOST_PP_ENUM_TRAILING_PARAMS_Z(1, N, class A)>`
`123`		`-boost::type<R>* result(R (T::*pmf)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)) Q, int = 0)`
	`123`	`+boost::type<R>* result(R (T::*)(BOOST_PP_ENUM_PARAMS_Z(1, N, A)) Q, int = 0)`
`124`	`124`	`{`
`125`	`125`	`return 0;`
`126`	`126`	`}`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ template <bool is_const_ref>`
`36`	`36`	`struct object_manager_get_pytype`
`37`	`37`	`{`
`38`	`38`	`template <class U>`
`39`		`- static PyTypeObject const* get( U& (*p)() =0)`
	`39`	`+ static PyTypeObject const* get( U& (*)() =0)`
`40`	`40`	`{`
`41`	`41`	`return converter::object_manager_traits<U>::get_pytype();`
`42`	`42`	`}`
`@@ -46,7 +46,7 @@ template <>`
`46`	`46`	`struct object_manager_get_pytype<true>`
`47`	`47`	`{`
`48`	`48`	`template <class U>`
`49`		`- static PyTypeObject const* get( U const& (*p)() =0)`
	`49`	`+ static PyTypeObject const* get( U const& (*)() =0)`
`50`	`50`	`{`
`51`	`51`	`return converter::object_manager_traits<U>::get_pytype();`
`52`	`52`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,13 +9,13 @@`
`9`	`9`	`using namespace boost::python;`
`10`	`10`
`11`	`11`	`template <class T>`
`12`		`-void assert_borrowed_ptr(T const& x)`
	`12`	`+void assert_borrowed_ptr(T const&)`
`13`	`13`	`{`
`14`	`14`	`BOOST_STATIC_ASSERT(boost::python::detail::is_borrowed_ptr<T>::value);`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`template <class T>`
`18`		`-void assert_not_borrowed_ptr(T const& x)`
	`18`	`+void assert_not_borrowed_ptr(T const&)`
`19`	`19`	`{`
`20`	`20`	`BOOST_STATIC_ASSERT(!boost::python::detail::is_borrowed_ptr<T>::value);`
`21`	`21`	`}`