Re: [Boost-users] Re: [boost] [review-results] Smart Container Library

On Tue, Dec 07, 2004 at 08:10:27PM -0600, Aleksey Gurtovoy wrote:

Pavol Droba writes:

...

I hereby announce that the Smart Container library is ACCEPTED into Boost. Congratulations and thanks to Thorsten Ottosen for providing us with this library.

Most of the reviewers were in favor of acceptance, but there were several requests for improvements. I have consulted them with Thorsten and he proposed to do following changes. I consider them sufficient for acceptance.

1. add support for boost.range in insert functions, constructors etc 2. use void* implementation 3. improve docs as discussed during the review 4. add support for null values: - remove preconditions - add functions is_null( size_t idx ); is_null( iterator ); 5. add more debug checks after access to ptr_begin(), ptr_end() 6. reconfigure policies: ptr_vector becomes ptr_vector = std::allocator > or perhaps ptr_vector > 7. improve tests

I also suggest, that before the inclusion of the library into the boost distribution, the library should be brought in the discussion to verify, that all issues above have been addressed.

I raised this during the review, and I reiterate: I strongly opposite inclusion of the library into the distribution under the current name. It's way too general and doesn't reflect the library's essence. Please choose something more concrete and to the point.

Yes, you are right, I'm aware of this (as well as Thorsten), I just forgot to add it to the list of issues. Regards, Pavol