From: Matthias Troyer (troyer_at_[hidden])
Date: 2007-05-11 23:20:56


On 11 May 2007, at 23:13, Neal Becker wrote:

> diff -r 5fcada374f41 boost/random/uniform_real.hpp
> --- a/boost/random/uniform_real.hpp Thu May 10 08:59:36 2007 -0400
> +++ b/boost/random/uniform_real.hpp Fri May 11 11:12:44 2007 -0400
> @@ -40,7 +40,7 @@ public:
> #ifndef BOOST_NO_LIMITS_COMPILE_TIME_CONSTANTS
> BOOST_STATIC_ASSERT(!std::numeric_limits<RealType>::is_integer);
> #endif
> - assert(min_arg < max_arg);
> + assert(min_arg <= max_arg);
> }
>
> // compiler-generated copy ctor and assignment operator are fine

This does not make sense to me. uniform_real should create numbers u
that are

min_arg <= u < maxarg

Thus choosing minarg == maxarg does not make sense since the set of
possible values would be empty.

Matthias