Re: [boost] intrusive containers review

From: Cory Nelson (phrosty_at_[hidden])
Date: 2007-03-17 12:12:57

On 3/17/07, Ion Gaztañaga <igaztanaga_at_[hidden]> wrote:
> 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?

Never mind, I apologize for that.  I was taking a some notes while
checking the library out and had written that down before finding the
reference area, and it got mixed up while writing the final review.

Who knew writing a review at 2am before going to bed would turn out to
be a bad thing? :)

> >> 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
> _______________________________________________
> Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
>

-- 
Cory Nelson
http://www.int64.org