diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index e4fa3f029c9..a5b5ef17e2e 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,3 +1,12 @@ +2006-06-20 Paolo Carlini + + * include/tr1/random.tcc (struct _Private::_Mod_w<>, + _Private::__mod_w<>): Remove. + (struct _Private::_Shift<>): New. + (struct _Private::_Max_w<>): Rename to _Max, use the latter. + (mersenne_twister<>::seed(unsigned long), seed(_Gen&, false_type), + max()): Adjust. + 2006-06-20 Vladimir Prus * libsupc++/eh_arm.cc (__cxa_begin_cleanup): Always return 'true'. diff --git a/libstdc++-v3/include/tr1/random.tcc b/libstdc++-v3/include/tr1/random.tcc index 7fef4cb73ed..93bc498c67e 100644 --- a/libstdc++-v3/include/tr1/random.tcc +++ b/libstdc++-v3/include/tr1/random.tcc @@ -96,46 +96,19 @@ _GLIBCXX_BEGIN_NAMESPACE(tr1) __mod(_Tp __x) { return _Mod<_Tp, __a, __c, __m, __m == 0>::__calc(__x); } - // Like the above, for a == 1, c == 0, in terms of w. - template - struct _Mod_w - { - static _Tp - __calc(_Tp __x) - { return __x % (_Tp(1) << __w); } - }; + template::digits> + struct _Shift + { static const _UIntType __value = 0; }; - template - struct _Mod_w<_Tp, __w, true> - { - static _Tp - __calc(_Tp __x) - { return __x; } - }; + template + struct _Shift<_UIntType, __w, true> + { static const _UIntType __value = _UIntType(1) << __w; }; - template - inline _Tp - __mod_w(_Tp __x) - { return _Mod_w<_Tp, __w, - __w == std::numeric_limits<_Tp>::digits>::__calc(__x); } - - // Selector to return the maximum value possible that will fit in - // @p __w bits of @p _Tp. - template - struct _Max_w - { - static _Tp - __value() - { return (_Tp(1) << __w) - 1; } - }; - - template - struct _Max_w<_Tp, __w, true> - { - static _Tp - __value() - { return std::numeric_limits<_Tp>::max(); } - }; + // The maximum value that will fit in @p __w bits of @p _UIntType. + template + struct _Max + { static const _UIntType __value = _Shift<_UIntType, __w>::__value - 1; }; } // namespace _Private @@ -236,7 +209,8 @@ _GLIBCXX_BEGIN_NAMESPACE(tr1) __b, __t, __c, __l>:: seed(unsigned long __value) { - _M_x[0] = _Private::__mod_w<_UIntType, __w>(__value); + _M_x[0] = _Private::__mod<_UIntType, 1, 0, + _Private::_Shift<_UIntType, __w>::__value>(__value); for (int __i = 1; __i < state_size; ++__i) { @@ -244,7 +218,8 @@ _GLIBCXX_BEGIN_NAMESPACE(tr1) __x ^= __x >> (__w - 2); __x *= 1812433253ul; __x += __i; - _M_x[__i] = _Private::__mod_w<_UIntType, __w>(__x); + _M_x[__i] = _Private::__mod<_UIntType, 1, 0, + _Private::_Shift<_UIntType, __w>::__value>(__x); } _M_p = state_size; } @@ -259,7 +234,8 @@ _GLIBCXX_BEGIN_NAMESPACE(tr1) seed(_Gen& __gen, false_type) { for (int __i = 0; __i < state_size; ++__i) - _M_x[__i] = _Private::__mod_w<_UIntType, __w>(__gen()); + _M_x[__i] = _Private::__mod<_UIntType, 1, 0, + _Private::_Shift<_UIntType, __w>::__value>(__gen()); _M_p = state_size; } @@ -272,12 +248,7 @@ _GLIBCXX_BEGIN_NAMESPACE(tr1) mersenne_twister<_UIntType, __w, __n, __m, __r, __a, __u, __s, __b, __t, __c, __l>:: max() const - { - using _Private::_Max_w; - using std::numeric_limits; - return _Max_w<_UIntType, __w, - __w == numeric_limits<_UIntType>::digits>::__value(); - } + { return _Private::_Max<_UIntType, __w>::__value; } template