[boost] 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; }