[Range] bugs in legacy compiler support

In range/detail/begin.hpp: template<> struct range_begin<array_> { template< typename T, std::size_t sz > static T* fun( T BOOST_ARRAY_REF[sz] ) BOOST_ARRAY_REF is not defined anywhere I can see. This should probably be: template< typename T, std::size_t sz > static T* fun( T BOOST_RANGE_ARRAY_REF()[sz] ) ---------------------------------- In range/detail/end.hpp: template<> struct range_end<char_array_> { template< typename T, std::size_t sz > static std::size_t fun( T BOOST_RANGE_ARRAY_REF()[sz] ) { return boost::range_detail::array_end( array ); } }; template<> struct range_end<wchar_t_array_> { template< typename T, std::size_t sz > static std::size_t fun( T BOOST_RANGE_ARRAY_REF()[sz] ) { return boost::range_detail::array_end( array ); } }; The return types of these two functions are wrong. They should be T* instead of std::size_t. -- Eric Niebler Boost Consulting www.boost-consulting.com

"Eric Niebler" <eric@boost-consulting.com> wrote in message news:41B72D89.2050906@boost-consulting.com... | | In range/detail/begin.hpp: | | template<> | struct range_begin<array_> | { | template< typename T, std::size_t sz > | static T* fun( T BOOST_ARRAY_REF[sz] ) | | BOOST_ARRAY_REF is not defined anywhere I can see. This should probably be: | | template< typename T, std::size_t sz > | static T* fun( T BOOST_RANGE_ARRAY_REF()[sz] ) yes, thanks! | ---------------------------------- | | In range/detail/end.hpp: | | template<> | struct range_end<char_array_> | { | template< typename T, std::size_t sz > | static std::size_t fun( T BOOST_RANGE_ARRAY_REF()[sz] ) | { | return boost::range_detail::array_end( array ); | } | }; | | template<> | struct range_end<wchar_t_array_> | { | template< typename T, std::size_t sz > | static std::size_t fun( T BOOST_RANGE_ARRAY_REF()[sz] ) | { | return boost::range_detail::array_end( array ); | } | }; | | The return types of these two functions are wrong. They should be T* | instead of std::size_t. yes. The cvs should be updated. Thanks Thorsten
participants (2)
-
Eric Niebler
-
Thorsten Ottosen