Yitzhak Sapir <[EMAIL PROTECTED]> writes:

> On Tue, 10 Dec 2002, David Abrahams wrote:
>
>> And, I suggest
>>
>>     BOOST_WORKAROUND(__BORLANDC__, |0x569)
>
> Since I began wondering whether it was a typo that you used | instead of
> &, since the | obviously always evaluates true, leading me to browse this
> thread, I think something more descriptive (not necessarily as long as the
> following) might be helpful:
>
> #define WORKAROUND_LAST_CHECKED_AT_VERSION(version) |1 // documentation macro only
> ...
> BOOST_WORKAROUND(__BORLANDC__, WORKAROUND_LAST_CHECKED_AT_VERSION(0x569))

That's awfully verbose, though. And it would need a "BOOST_" prefix,
making it more verbose still.

Given that the meaning of BOOST_WORKAROUND is already non-transparent,
you need to read the documentation to understand it. I think it'd be
best to simply document the "|" convention there.

-- 
                       David Abrahams
   [EMAIL PROTECTED] * http://www.boost-consulting.com
Boost support, enhancements, training, and commercial distribution

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

Reply via email to