[review] Boost.Functional/Factory library result : accepted

John Torjo <[email protected]>
Newsgroups gmane.comp.lib.boost.announce
Message-ID <4777C486.7030206__18578.7054397681$1199032304$gmane$org@torjo.com>
Hi all,

The Boost.Functional/Factory library, submitted by Tobias Schwinger has 
been *accepted*. Thanks to all reviewers and to all who contributed 
before the review!

We've received 3 reviews, all positive: Larry Evans, Stejan Rajko, and 
Detlef Wilkening*.

*Suggested improvements
*
*Implementation
*- Allocator should be inherited

*Docs
*- should include some hint about how to overcome the Lvalue constraint 
on the arguments that's mentioned in the docs:
   a0...aN - argument LValues to a constructor of T
- separate the two templates (factory and value_factory) a little bit 
more in the introduction so that it reinforces that they are different 
in usage
- better motivation
- more detailed examples




* note: I consider Detlef's review positive, even though he wanted 3 
extensions (in light of Tobias's answer):
- Better name for factory: factory_ptr or factory_pointer.
- Better motivation for beginners in the docu.
- Extension with a third template parameter for e.g. 2 phase construcion.


Best,
John

-- 
http://John.Torjo.com -- C++ expert
... call me only if you want things done right

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost-announce
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.