Neal Becker | 20 May 20:01

constrained_value redundant check?

   constrained & operator = (const value_type & v)
    {
        if(constraint()(v))
            _value = v;
        else
            error_handler()(_value, v, _constraint());

        BOOST_ASSERT(_check_value());  << Isn't this redundant?
        return *this;
    }

Maybe this should be:

  constrained & operator = (const value_type & v)
    {
        if(constraint()(v))
            _value = v;
        else {
            error_handler()(_value, v, _constraint());

            BOOST_ASSERT(_check_value());
        }
        return *this;
    }

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost


Gmane