|
Boost : |
Subject: [boost] Move notes
From: David Abrahams (dave_at_[hidden])
Date: 2009-09-07 15:57:01
Hi Ion,
A few things I noticed looking through move.hpp in the sandbox
1. you static_cast in one direction but reinterpret_cast in the other.
Why?
2. I think move_iterator's operator-> return type is wrong; maybe make
the pointer type be the underlying iterator type instead?
3. I think its operator[] can be wrong for some iterators. See
http://www.boost.org/doc/libs/1_40_0/libs/iterator/doc/iterator_facade.html#operator
4. bool-valued traits should be derived from an MPL integral constant,
e.g.
template <class T>
struct is_whatever : mpl::bool_< (some compile-time expression ) > {};
Cheers,
-- Dave Abrahams BoostPro Computing http://www.boostpro.com
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk