On 03/15/2016 11:29 PM, Glen Fernandes wrote:
3. Surely:
p1 = node::allocate(node
::allocator_type(x, y), p, q)
can become simpler still. i.e. the target is still something as clean as:
p1 = allocate_shared<U>(Allocator(x, y), p, q);
I have simplified it and now it cannot be any simpler than the following
(using template template argument):
template
class Allocator
{
[...]
int main()
{
int n1 = 0, m1 = 0;
int n2 = 0, m2 = 0;
{
boost::root_ptr<U> p1, p2, p3;
p1 =
boost::allocate_node<U>(boost::make_node_allocator(n1,
m1), 1, 'a');
p2 =
boost::allocate_node<U>(boost::make_node_allocator(n2, m2), 2, 'b');
p3 =
boost::allocate_node<U>(boost::make_node_allocator(n2, m2), 3, 'c');
if (n1 != 1 || m1 != 1 || n2 != 2 || m2 != 2) {
throw 3;
}
}
if (n1 != 0 || m1 != 0 || n2 != 0 || m2 != 0) {
throw 4;
}
}
Or:
https://github.com/philippeb8/root_ptr/blob/master/example/allocator.cpp#L82
4. No 'const_cast'-ing away 'const'. You need to use the allocators correctly.
I took the const_cast<>s away.