Re: [boost] [assert] Proposed patch

From: Peter Dimov (pdimov_at_[hidden])
Date: 2006-07-06 07:27:57

Gennaro Prota wrote:
> On Thu, 6 Jul 2006 13:29:58 +0300, "Peter Dimov" <pdimov_at_[hidden]>
> wrote:
>
>> Gennaro Prota wrote:
>>
>>>  #define BOOST_ASSERT(expr)                         \
>>>    (                                                \
>>>      (void)sizeof boost::detail::assert_            \
>>>                    ::valid_assert_expression(expr)  \
>>>      , assert(expr)                                 \
>>>    )                                             /**/
>
> Sorry, I thought you had read my earlier post about this. I found some
> invocations of BOOST_ASSERT which don't use an expression with int
> type. When BOOST_ASSERT falls back to the standard assert macro it may
> work or not, depending on what the underlying C library does.

I know. Is this a real problem? Are you sure that making BOOST_ASSERT fail 
when assert works a good idea? It will just encourage people to not use it. 
Not that they need much encouragement. Almost everyone ignores BOOST_ASSERT 
(and the users' requests to please not do so).