[Dune] [#645] Review and apply ArrayList patch

Dune dune at dune-project.org
Thu Dec 3 17:28:02 CET 2009


THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

The following task has a new comment added:

FS#645 - Review and apply ArrayList patch
User who did this - Markus Blatt (mblatt)

----------
valgrind reports leaks now. And I do not know how to get rid of them fast enough. 
At least one of them seems to subject to the shared_ptr implementation of boost.
The other one is that the patch assumes that reset frees memory, which it does not. 

Furthermore it seems like SmartPointer is missing a reset member function that is used in Arraylist. (Did you test this without tr1?)

Anyway, the original idea (which I had forgotten about) seems to be violated, as the allocator is only used to allocate the shared_ptr and the actual object is allocated with new (with the original SmartPointer this was on the stack by the way). This can be fixed of course but we still end up with allocates using the allocator. That is probably the cost we have to take.
----------

More information can be found at the following URL:
http://www.dune-project.org/flyspray/index.php?do=details&task_id=645#comment1439

You are receiving this message because you have requested it from the Flyspray bugtracking system.  If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.




More information about the Dune mailing list