Re: [boost] intrusive containers review

From: Ion Gaztañaga (igaztanaga_at_[hidden])
Date: 2007-03-17 08:54:26

Cory Nelson wrote:
>> What is your evaluation of the design?
> 
> Very good.  It was a pleasure to use.  I have long wanted an easy way
> to apply C techniques in a true to C++ style, and this nails it.
>> What is your evaluation of the implementation?
> 
> I did not review the r/b tree, but everything else seemed well written.

Thanks.

>> What is your evaluation of the documentation?
> 
> I found documentation lacking.  I would like to see full class
> references: it took me a bit of hunting to find a commonly used
> function container::current() which was very frustrating.

What do you mean with full class references? Current reference

http://ice.prohosting.com/newfunk/boost/libs/intrusive/doc/html/intrusive/reference.html

lists all the operations of every public class. What do you think is 
missing?

>> And finally, every review should answer this question:
>> Do you think the library should be accepted as a Boost library?
> 
> Absolutely, I vote to accept.

Thanks for you vote!

Regards

Ion