boost_range_array unused in several array_size() functions

[sparent@…]

The boost_range_array parameter is not used in several overloads of array_size() in range/details/implementation_help.hpp - this generates warnings and halts our builds.

[Thorsten Ottosen]

Please submit a patch

Thanks

-Thorsten

[Thorsten Ottosen]

I don't really no how to silence this warning, so I'd really appreciate a patch.

Thanks

-Thorsten

[Thorsten Ottosen]

I have now added a test that run for normal compilers. Neither vc8 nor gcc gives me any warnings. What compiler are you getting warnings with?

-Thorsten

[sparent@…]

gcc with -Wunused or -Wall (you can ask Mat Marcus for the bjam file if you would like) - our teams compile with all warnings on and warnings as errors. Here is the patch.

Sean

---

p4 diff 'boost_libraries/boost/range/detail/implementation_help.hpp#3'

boost_libraries/boost/range/detail/implementation_help.hpp#3 - /Users/sparent/development/projects/boost_libraries/boost/range/detail/implementation_help.hpp

112a113,120

/*

ADOBE (sparent@…) 1464 : Using unused parameter to silence warnings.

*/ template <typename T> inline void adobe_boost_range_size_unused(const T&) { } /*

END ADOBE

*/

116a125,131

/*

ADOBE (sparent@…) 1464 : Using unused parameter to silence warnings.

*/

adobe_boost_range_size_unused(boost_range_array);

/*

END ADOBE

*/

122a138,144

/*

ADOBE (sparent@…) 1464 : Using unused parameter to silence warnings.

*/

adobe_boost_range_size_unused(boost_range_array);

/*

END ADOBE

*/

128a151,157

/*

ADOBE (sparent@…) 1464 : Using unused parameter to silence warnings.

*/

adobe_boost_range_size_unused(boost_range_array);

/*

END ADOBE

*/

134a164,170

/*

ADOBE (sparent@…) 1464 : Using unused parameter to silence warnings.

*/

adobe_boost_range_size_unused(boost_range_array);

/*

END ADOBE

*/

[sparent@…]

Sorry - wiki ate that...

p4 diff '//boost_libraries/boost/range/detail/implementation_help.hpp#3'
==== //boost_libraries/boost/range/detail/implementation_help.hpp#3 - /Users/sparent/development/projects/boost_libraries/boost/range/detail/implementation_help.hpp ====
112a113,120
>         /*
>         	ADOBE (sparent@adobe.com) 1464 : Using unused parameter to silence warnings.
>         */
>         template <typename T>
>         inline void adobe_boost_range_size_unused(const T&) { }
>         /*
>         	END ADOBE
>         */
116a125,131
>         /*
>         	ADOBE (sparent@adobe.com) 1464 : Using unused parameter to silence warnings.
>         */
>         	adobe_boost_range_size_unused(boost_range_array);
>         /*
>         	END ADOBE
>         */
122a138,144
>         /*
>         	ADOBE (sparent@adobe.com) 1464 : Using unused parameter to silence warnings.
>         */
>         	adobe_boost_range_size_unused(boost_range_array);
>         /*
>         	END ADOBE
>         */
128a151,157
>         /*
>         	ADOBE (sparent@adobe.com) 1464 : Using unused parameter to silence warnings.
>         */
>         	adobe_boost_range_size_unused(boost_range_array);
>         /*
>         	END ADOBE
>         */
134a164,170
>         /*
>         	ADOBE (sparent@adobe.com) 1464 : Using unused parameter to silence warnings.
>         */
>         	adobe_boost_range_size_unused(boost_range_array);
>         /*
>         	END ADOBE
>         */

[Thorsten Ottosen]

Fix in trunk.