Re: [boost] [functional] [variant] Hash patch for variant

Subject: Re: [boost] [functional] [variant] Hash patch for variant
From: Antony Polukhin (antoshkka_at_[hidden])
Date: 2011-10-20 14:34:39

2011/10/20 Daniel James <dnljms_at_[hidden]>:
> On 20 October 2011 17:22, Antony Polukhin <antoshkka_at_[hidden]> wrote:
>> Header and tests in attachments.
>> Passes tests on GCC-4.4 and VC10.
>>
>> OK to commit to trunk and include header file from
>> boost/functional/hash/extensions.hpp ?
>
> It should be included in variant not hash. You might be able to just
> include <boost/functional/hash_fwd.hpp> rather than
> <boost/functional/hash.hpp>.

Thanks, you are right, <boost/functional/hash_fwd.hpp> is enough.
Header and tests are updated.

OK to commit boost/variant/detail/hash_variant.hpp and
boost/libs/variant/test/hash_variant_test.cpp and Jamfile to trunk?

Best regards,
Antony Polukhin