Subject: Re: [boost] [Boost-users] [Review] Boost.Endian mini-review
From: Bjorn Reese (breese_at_[hidden])
Date: 2015-02-01 08:33:11
On 01/23/2015 05:17 PM, Joel FALCOU wrote:
> Is the library ready to be added to Boost releases?
The documentation is well-written, the code looks clean (except for the
multitude of #ifdefs), and the API is superior to N3783.
Having said that, I do have a few comments:
The endian_reverse() functions assume that there are only two types of
endianness (big and little) and that they are the reverse of each other.
In practice, this is a feasible assumption. However, once in a while, an
odd endianness does show up in this millennia. For example, ARM11
processors uses mix-endian  for double. Byte reversal will not
handle such cases correctly. The could become a problem during C++
Why are there no *_int8_t types?
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk