Subject: Re: [boost] [Review] Formal Review: Boost.Move
From: Steven Watanabe (watanabesj_at_[hidden])
Date: 2010-05-22 12:48:10
I've read straight though the documentation and code.
It looks solid. I only saw a few trivial issues:
In the implementation of move, you fully qualify
::boost::disable_if, but not enable_if
Shouldn't the optimized version of forward use identity,
so it won't try to deduce T?
Regarding the use of BOOST_MOVE_OPTIMIZED, I
favor not using this. I would prefer to avoid unexpected
side-effects even at the cost of some performance, especially
since we can deal with the performance problems by
Finally, I vote to accept Boost.Move into Boost.