Boost C++ Libraries: Ticket #6610: customizing boost::pool/boost::object_pool via template parameter for ordered/non-ordered usage

owner, status changed

edupuis — Thu, 03 May 2012 16:50:24 GMT

owner changed from Chris Newbold to edupuis
status new → assigned

edupuis — Sun, 06 May 2012 13:48:20 GMT

Method boost::pool::purge_memory() does not require the pool to be ordered. Documentation has to be amended.

Current implementation of boost::pool::release_memory() fails if the pool is not ordered. It could be implemented by adding a pool.order() function which could be called by the user to make an unordered pool ordered. The pool.order() function could also be called automatically if the pool maintains a boolean describing whether it is ordered or not.

We could provide a gateway to release_memory() and to purge_memory(), the latter calling destructors of objects allocated, exactly what is already implemented in the destructor of object_pool.

This is related to ticket #3789. In the current implementation, boost::object_pool can not be used to allocate arrays of objects.

If ticket #3789 is resolved, this is probably not as needed. However, it could let a user choose between a slow 'object_pool::free()' or a slow 'object_pool::~object_pool()' as described in ticket #3789, although I fail to see relevant usages where one would choose a slow free. The suggested template parameter seems useless for boost::pool as the caller chooses between the ordered or unordered malloc/free functions.

Any comments ?

anonymous — Mon, 07 May 2012 11:35:35 GMT

So, I agree with you. The behaviour as described in your comment in ticket #3789 is what I expected.

status changed; resolution set

edupuis — Sun, 13 May 2012 16:41:29 GMT

status assigned → closed
resolution → fixed

(In [78454]) Added methods release_memory() and purge_memory() to object_pool. This complete and fixes #6610, as the remaining part of this ticket was addressed by #3789.

status changed; resolution deleted

edupuis — Mon, 21 May 2012 05:59:56 GMT

status closed → reopened
resolution fixed

owner, status changed

edupuis — Mon, 16 Jul 2012 20:01:58 GMT

owner changed from edupuis to John Maddock
status reopened → new

https://svn.boost.org/svn/boost/sandbox/pool at revision 79460 contains a solution for tickets #3789, #5902, #6561, #6610, #6701, #6718, #6865 and #6867. Related test cases are also present.

https://svn.boost.org/svn/boost/sandbox/pool at revision 79460 does not contain any other new features or modifications other than those related to the above tickets.

Boost.Pool currently has no maintainer and is thus orphaned.