On Tue, Apr 17, 2012, at 09:48 PM, Mathias Gaunard wrote:
On 17/04/12 12:52, Michael Kochetkov wrote:
Anyway the real problems are that 1. boost's implementation of uuid is at least weird in terms there are much better approaches that cost nothing in implementation but are ignored.
Comparing 16 bytes is hardly the important part in a UUID library. Boost.UUID is using the correct construct for this, it is sad some compilers cannot optimize it properly, but it's still the ideal solution from a C++ point of view.
The approach you suggest is not well-formed C++ and is specific to your particular configuration; it might fare worst on others. So it's not really "better" nor does it "cost nothing".
An option is to simply fix the problem in your local copy of Boost.
That being said, I'm sure the library author will consider fixing it if you file a bug for it.
Absolutely, I am always happy to review submitted bugs/patches. < snip > Regards, Andy Tompkins boost.uuid author