#2208: Fix small memory leak in Piraha ---------------------------------+-------------------- Reporter: Steven R. Brandt | Type: defect Status: new | Priority: minor Milestone: ET_2019_02 | Component: Cactus Version: development version | Keywords: ---------------------------------+-------------------- This pull request
https://bitbucket.org/cactuscode/cactus/pull-requests/54/piraha-mem/diff
Fixes a small memory leak caused by a cyclic reference.
#2208: Fix small memory leak in Piraha -------------------------------+--------------------------------- Reporter: Steven R. Brandt | Owner: (none) Type: defect | Status: new Priority: minor | Milestone: ET_2019_02 Component: Cactus | Version: development version Resolution: | Keywords: -------------------------------+---------------------------------
Comment (by Roland Haas):
Steve and Roland discussed this in person at SC18. Roland asks to have the prototype of the constructor changed to take a bare pointer rather than a smart-pointer to make it more obvious for callers of the Lookup objects that they store a bare pointer to the their grammar.
The current logic in piraha should be such that the grammar object exists for as long as the lookup objects exist so the pointer should not become dangling.
#2208: Fix small memory leak in Piraha -------------------------------+--------------------------------- Reporter: Steven R. Brandt | Owner: (none) Type: defect | Status: review Priority: minor | Milestone: ET_2019_02 Component: Cactus | Version: development version Resolution: | Keywords: -------------------------------+--------------------------------- Changes (by Steven R. Brandt):
* status: new => review
Comment:
The pull request has been updated to address Roland's concern.
#2208: Fix small memory leak in Piraha -------------------------------+--------------------------------- Reporter: Steven R. Brandt | Owner: (none) Type: defect | Status: reviewed_ok Priority: minor | Milestone: ET_2019_02 Component: Cactus | Version: development version Resolution: | Keywords: -------------------------------+--------------------------------- Changes (by Roland Haas):
* status: review => reviewed_ok
Comment:
Please apply.
#2208: Fix small memory leak in Piraha -------------------------------+--------------------------------- Reporter: Steven R. Brandt | Owner: (none) Type: defect | Status: closed Priority: minor | Milestone: ET_2019_02 Component: Cactus | Version: development version Resolution: fixed | Keywords: -------------------------------+--------------------------------- Changes (by Steven R. Brandt):
* status: reviewed_ok => closed * resolution: => fixed
Comment:
Fixed in 2b07304cd5089d411d8808ecf9e1e7a08af0892c of https://bitbucket.org/cactuscode/cactus.git
trac@lists.einsteintoolkit.org