Re: [boost] [bind][tr1] patch for more compliant placeholders

From: Peter Dimov (pdimov_at_[hidden])
Date: 2007-05-10 07:26:12

Daniel Walker wrote:
> On 5/9/07, Peter Dimov <pdimov_at_[hidden]> wrote:

>> How about just using
>>
>> #if defined(__BORLANDC__) || defined(__GNUC__) && (__GNUC__ * 100 +
>> __GNUC_MINOR__ <= 400)
>>
>> ?
>
> I just like the idea of being able to search for BOOST_WORKAROUND and
> find workaround specific code. I think it should be
>
> #if defined(__BORLANDC__) || (defined(BOOST_GCC) &&
> BOOST_WORKAROUND(BOOST_GCC, <= 0x0400))

I still prefer the above, sorry. It limits the patch to just 
placeholders.hpp and is consistent with the rest of its checks. Let's leave 
the task of defining BOOST_GCC properly to someone else. I see that there is 
already BOOST_CXX_GNUC in Boost.Config along with a bunch of other 
BOOST_CXX_ macros, but it seems always zero.