[boost] Re: [Range] bugs in legacy compiler support

From: Thorsten Ottosen (nesotto_at_[hidden])
Date: 2004-12-08 13:00:46

"Eric Niebler" <eric_at_[hidden]> wrote in message 
news:41B72D89.2050906_at_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