On Apr 2, 2005, at 7:49 AM, Peter Dimov wrote:
JOAQUIN LOPEZ MU?Z wrote:
It was my proposal to put it under <boost/functional/hash.hpp>, on the following rationales: * TR1 hash is located on <functional> * When John Maddock's Boost.TR1 inititative includes Boost.Hash, the naming conventions set up by John suggest that the wrapping header will be <boost/tr1/functional>, which is more consistent with <boost/functional/hash.hpp> than with <boost/hash.hpp> (IMHO).
I understand the logic, but with my user hat on, I expect boost::hash in <boost/functional.hpp> (since std::tr1::hash is in <functional>) or in <boost/hash.hpp> (because it is defined in namespace boost, not in namespace boost::functional).
<boost/functional/hash.hpp> is neither here nor there, IMO. The rest of the headers are fine, and placing hash under libs/functional is fine, too.
IMHO, just putting it in <boost/functional.hpp> is the right thing to do. It matches TR1 and avoids another top-level header. Doug