[Dune] [#655] Alberta GridFactory leaks memory

[Oliver Sander]

Hi Martin!
My question is: what is a DuneBoundaryProjection and how does it relate to
BoundarySegment?

Yours,
Oliver

Martin Nolte schrieb:
> Hi Oli,
>
> I somehow do not understand your problem: AlbertaGrid does implement 
> insertBoundarySegment. And the missing deallocation is a bug -- this 
> is work in progress.
>
> Yours,
>
> Martin
>
> Oliver Sander wrote:
>> Hi Martin, hi all!
>> I have seen on the commit mails that you guys have recently added 
>> support for
>> parametrized boundaries in AlbertaGrid.  Your code does not use any 
>> of the infrastructure
>> for this already present in Dune (namely, the BoundarySegment class 
>> and the
>> corresponding method insertBoundarySegment() in the GridFactory).  
>> Could you
>> explain a bit what you did and why you couldn't use the existing 
>> infrastructure?
>> Thank you!
>>
>> Incidentally, in the documentation of 
>> GridFactory::insertBoundarySegment() it says
>> "The grid object takes control of this object and deallocates it when 
>> destructing itself. "
>> Maybe we should hand over an std::auto_ptr instead of a plain pointer 
>> to make this
>> even clearer.
>>
>> Yours,
>> Oliver
>>
>> Dune schrieb:
>>> THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
>>>
>>> The following task has a new comment added:
>>>
>>> FS#655 - Alberta GridFactory leaks memory
>>> User who did this - Martin Nolte (nolte)
>>>
>>> ----------
>>> This is a known Dune bug. It is still unclear how to determine 
>>> ownership of a boundary projection, because it is unclear whether 
>>> all of them are passed to the grid (and how often). Currently, the 
>>> grid does not free them either do avoid deleting the same pointer 
>>> multiple times.
>>> ----------
>>>
>>> More information can be found at the following URL:
>>> http://www.dune-project.org/flyspray/index.php?do=details&task_id=655#comment1325 
>>>
>>>
>>> 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.
>>
>>
>


-- 
************************************************************************
* Oliver Sander                ** email: sander at mi.fu-berlin.de        *
* Freie Universität Berlin     ** phone: + 49 (30) 838 75348           *
* Institut für Mathematik      ** URL  : page.mi.fu-berlin.de/~sander  *
* Arnimallee 6                 ** -------------------------------------*
* 14195 Berlin, Germany        ** Member of MATHEON (www.matheon.de)   *
************************************************************************