headers
Doug Gregor | 5 Dec 2004 21:10
Picon
Picon
Favicon

[Review Results] Named Parameters Library

The Named Parameters Library has been accepted into Boost. 
Congratulations to the authors, David Abrahams and Daniel Wallin.

The vast majority of reviews were positive, although most reviewers 
requested additional documentation. Here is a list of the most 
important/commonly-requested changes or additions to the library:

   - More documentation! Specifically,
     + Automatic overload generation should go into the tutorial
     + Document operator,
     + Performance data (what's the cost?)
     + Reference section
     + "Best practices" section.

   - Add a metafunction for getting the type of a parameter, e.g.,
     param_type<Params, width_t>::type.

   - Add test cases for the automatic overload generation macros.

   - Fix the BOOST_NAMED_PARAMS_MAX_ARITY bug; Cromwell Enage provided
     a solution.

	Doug Gregor
	Review Manager

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost-announce
Permalink | Reply | 0 comments |
headers
Pavol Droba | 7 Dec 2004 18:13
Picon

[review-results] Smart Container Library

Hi,

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<T,clone_manager> becomes
   ptr_vector<T,clone_allocator = something, std::allocator<T*> = std::allocator<T*> >
   or perhaps
   ptr_vector<T,clone_allocator = something< std::allocator<T* > >
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.
(Continue reading)

Permalink | Reply | 0 comments |

Gmane