Re: [boost] [assert][lightweight_test] Quick review of proposed changes

Subject: Re: [boost] [assert][lightweight_test] Quick review of proposed changes
From: Beman Dawes (bdawes_at_[hidden])
Date: 2011-01-24 15:10:44

On Sun, Jan 23, 2011 at 12:32 PM, Peter Dimov <pdimov_at_[hidden]> wrote:
> Beman Dawes wrote:
>
>> BOOST_ENABLE_ASSERT_MSG_HANDLER enables a separate error handler.
>
> I think that the existing BOOST_ENABLE_ASSERT_HANDLER should control the
> behavior of BOOST_ASSERT_MSG as well. It's very unlikely that users will
> want one of the macros to go to their own handler, and the other to keep
> calling std::abort, and in the current implementation, this is what will
> happen silently in existing projects that already define
> BOOST_ENABLE_ASSERT_HANDLER. I suspect that such users will prefer a
> link-time error, alerting them to the presence of BOOST_ASSERT_MSG macros in
> the program.

KISS, I guess.

Will change,

--Beman