[functional] [variant] Hash patch for variant
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 ? Best regards, Antony Polukhin
On 20 October 2011 17:22, Antony Polukhin <antoshkka@gmail.com> 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>.
2011/10/20 Daniel James <dnljms@gmail.com>:
On 20 October 2011 17:22, Antony Polukhin <antoshkka@gmail.com> 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
As there were no objections and patch is trivial, I created ticket #6163 and committed changes to trunk (r75606). Best regards, Antony Polukhin
participants (2)
-
Antony Polukhin -
Daniel James