Boost logo

Boost :

Subject: Re: [boost] [Containers Library Review] The review of the Containers library by Ion Gaztanaga starts today
From: Ion Gaztañaga (igaztanaga_at_[hidden])
Date: 2011-08-08 04:03:10


El 08/08/2011 6:58, Kohei Takahashi escribió:
> Hi,
>
> My review is here.
>
>> * What is your evaluation of the design?
> I think it is important, emulating move semantics and emplace inserter.
>
>> * What is your evaluation of the implementation?
> Some requirements added in C++0x (see 23.2.1 in n3290). Does
> Boost.Container satisfy, partial or not? (just question) I anxious
> about 23.2.1/8 especially.
> And associated to the requirements, do you have any plans to provide
> allocator_traits?

It implements 23.2.1/8 without allocator_traits. I plan to add
allocator_traits and C++11 features soon as C++11 features are now stable.

>> * What is your evaluation of the documentation?
> 1) There are no documentation about free functions. (operators,
> swap, ...

Ok, I'll fix this, although I think they are self-describing.

> 2) The documentation of deque::push_[back|front] are missing.

Thanks for spotting this.

> 3) Some member functions are not documented about Returns/Effects,
> Complexity or Throws. (ex: after No.23 in basic_string) Please check it.

Ok. I'd need to do a full review of all descriptions.

>> 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.
> I think should be accepted it with improved documentations.

Thanks.

Best,

Ion


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk