Re: [boost] variant - errors in mpl/less.hpp when bind.hpp isincludedbeforevariant.hpp with vc7.1

From: Reece Dunn (msclrhd_at_[hidden])
Date: 2004-02-06 17:36:32

Rene Rivera wrote:
>Brock Peabody wrote:
>>>-----Original Message-----
>>>From: boost-bounces_at_[hidden] 
>>>[mailto:boost-bounces_at_[hidden]]
>>>On Behalf Of Peter Dimov
>>
>>>Nice bug. I think that it's triggered by the "value<" sequence in the 
>>>code
>>>below:
>>>
>>>    enum
>>>    {
>>>        msvc71_wknd_ = ( BOOST_MPL_AUX_VALUE_WKND(T1)::value
>>>                        < BOOST_MPL_AUX_VALUE_WKND(T2)::value )
>>>    };
>>>
>>
>>>You can work around the problem by replacing the above with
>>>
>>>    enum
>>>    {
>>>        msvc71_wknd_ = ( BOOST_MPL_AUX_VALUE_WKND(T2)::value
>>>                        > BOOST_MPL_AUX_VALUE_WKND(T1)::value )
>>>    };
>>>
>>>I've commited the fix to the CVS HEAD as it seems fairly harmless (he 
>>>said
>>>before breaking all other compilers). ;-)
>
>Shouldn't that be ">=" ???

No, the above is correct. Consider:

   a < b == b > a

Read either size backwards, noting the direction of the slant in the 
comparison operator.

Regards,
Reece

_________________________________________________________________
Tired of 56k? Get a FREE BT Broadband connection 
http://www.msn.co.uk/specials/btbroadband