From: Larry Evans (cppljevans_at_[hidden])
Date: 2007-12-17 09:20:04
On 12/17/07 04:22, John Torjo wrote:
> Hi all,
> Today starts the formal Fast-Track review of the
> Boost.Functional/Factory library.
> What to include in Review Comments
> * What is your evaluation of the documentation?
Good, however, it should include some hint about how to overcome
the Lvalue constraint on the arguments that's mentioned in the
argument LValues to a constructor of T
I'd think something like the following:
< class Source
: boost::forward_adapter<boost::factory<Source> >
would nicely show how the forward lib can be used to overcome this
> * What is your evaluation of the potential usefulness of the
Very. I'm thinking it could be used as a replacement for Abraham's
new_ as shown in the attachment to:
I think it could also be used for similar factories which, for
example, want to decorate the source object with some hidden features,
e.g. a refcount. The containing smart_ptr would know about that and
somehow retrieve it and avoid the extra overhead of a detached
refcount as in, for example, shared_ptr.
> * Did you try to use the library? With what compiler?
> Did you have any problems?
Compiled factory.cpp and factory_fwd template with gcc-4.1 and had no
> * How much effort did you put into your evaluation?
> A glance? A quick reading? In-depth study?
> * Are you knowledgeable about the problem domain?
> And finally, every review should answer this question:
> * Do you think the library should be accepted as a Boost library?
> Be sure to say this explicitly so that your other comments
> don't obscure your overall opinion.
Boost list run by bdawes at acm.org, david.abrahams at rcn.com, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk