Boost logo

Boost :

From: Neil Mayhew (neil_mayhew_at_[hidden])
Date: 2008-05-29 23:39:01

On 2008-05-26 05:22, Beman Dawes wrote:
> There's an updated version available in the file vault. Changes include:
> * Use of namespaces to simulate C++0x scoped enums.
> * Addition of operator= for improved efficiency.
> * Updated docs, including new FAQ entries.
> * Jamfile.v2 added.
> See
> Does this look ready for formal review? Comments appreciated!

The 0.8 version looks good to me, except for two things:

1. As I commented earlier about the name:

> ... there is already an endian.hpp elsewhere in boost and I think it
> would be easy for people to miss this endian, and never discover its
> functionality.

When I was searching to see if boost had something like this, the great
majority of the hits were for the existing boost/detail/endian.hpp,
which wasn't what I was hoping for. It doesn't seem good to me to have
two headers in boost with the same file name, even if they are in
different directories. The only good solution I can see is to rename
detail/endian.hpp to detail/endianness.hpp, which is more descriptive,
IMHO, but would probably break a lot of existing client code. Although
maybe clients include detail/ headers at their own risk?

2. As mentioned in other posts, the implementation of operators using
<boost/operators.hpp> needs to be reconsidered.

Thanks, Beman, for your work in getting this moving again.


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