Re: [boost] [assert] Proposed patch

From: Reece Dunn (msclrhd_at_[hidden])
Date: 2006-07-05 10:26:41

Gennaro Prota wrote:
> On Tue, 04 Jul 2006 15:12:43 +0200, Gennaro Prota
> <gennaro_prota_at_[hidden]> wrote:
> 
> > [something...]
> 
> I always wonder how a lack of answers should be taken: (a) nobody
> disagrees (b) nobody agrees (c) nobody cares.

How about:
(d) they are busy, so haven't had chance to look at it yet;
(e) they haven't noticed it because, since last they checked, there
they had 50+ new e-mail from Boost only!

> Hint: (c) is impossible, as *I* care, and for that reason I'll take
> (a) by default ;)

As for the original question, I would personally prefer:

c) # define BOOST_ASSERT(expr) assert("BOOST_LIBRARIES:" && (expr))

but with "BOOST_LIBRARIES:" something more meaningful/readible, like
"Boost assertion failure: ".

- Reece
_________________________________________________________________
Be one of the first to try Windows Live Mail.
http://ideas.live.com/programpage.aspx?versionId=5d21c51a-b161-4314-9b0e-4911fb2b2e6d