Boost logo

Boost :

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
explicitly moving.

Finally, I vote to accept Boost.Move into Boost.

In Christ,
Steven Watanabe

Boost list run by bdawes at, gregod at, cpdaniel at, john at