Boost logo

Boost :

Subject: Re: [boost] [Review][Move] Boost.Move Review Results
From: Thomas Klimpel (Thomas.Klimpel_at_[hidden])
Date: 2011-02-01 20:09:36

Michael Caisse wrote:
> Boost.Move library ACCEPTED

Congratulation to Ion and a big thank you to Ion, Michael and everybody else involved.

> Six reviews were submitted:
> - 5 reviews ACCEPT

I just looked who dared to accept this library only conditionally, and found a out that it was me complaining about too many typos in the "Two Emulation Mode" documentation. Oh well...

> -- Other Issues --
> 1. The class rv should be public and documented.

The way Boost.Move is currently designed and implemented, it uses true rvalue references for compilers that implement them. I just hope that in case the class rv becomes public and documented, it will be done in a way that still preserves this property. On the other hand, if I understand the meaning of "Other Issues" correctly, Ion is not forced to follow every one of these suggestions.


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