On 3/17/07, Ion Gaztañaga <igaztanaga@gmail.com> 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/re...
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