Boost logo

Boost Users :

Subject: Re: [Boost-users] [Review] UUID library (mini-)review starts today, November 23rd
From: Dave Jenkins (david_at_[hidden])
Date: 2008-11-25 17:27:42

> - What is your evaluation of the design?

I like the design, but wish it had time-based UUIDs.

> - What is your evaluation of the implementation?

I had no problem reading the code. But there's almost nothing in
"uuid.ipp". Could you merge it into "uuid.hpp" for simplicity?

> - What is your evaluation of the documentation?

I'd like to see the Rationale section come first, before you give the whole
Class Synopsis. That would make it easier for casual readers to understand
the purpose of the library before reading further.

A small example program near the beginning of the document would help too.

There is a spelling error in the Rationale section, s/indended/intended/.

> - What is your evaluation of the potential usefulness of the library?

It's a nice addition to Boost.

> - Did you try to use the library? With what compiler? Did you have any
> problems?

Yes, I tried running some test code and had no problems.

I vote Yes, to include the UUID library in Boost.

Dave Jenkins

Boost-users list run by williamkempf at, kalb at, bjorn.karlsson at, gregod at, wekempf at