|
Boost : |
From: Beman Dawes (bdawes_at_[hidden])
Date: 2006-06-04 22:06:10
On 6/4/06, Christopher Kohlhoff <chris_at_[hidden]> wrote:
> Hi Beman,
>
> Beman Dawes <bdawes_at_[hidden]> wrote:
> > A refresh of the .zip file for the Endian library, based on
> > comments received so far, is available at
> > http://mysite.verizon.net/~beman/endian-0.2.zip
>
> The interface looks pretty good to me, except that I'm a little
> concerned that the endianness enum names are introduced
> directly into the boost namespace. In particular 'big', 'little'
> and 'native' seem like common names. Perhaps it could be:
>
> namespace endianness {
> enum type { big, ... };
> }
Good point. I'm not sure about your solution yet (not for any
technical concerns, but just cause it's past my bedtime and my mind is
already half asleep).
Thanks,
--Beman
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk