search

ForestClaw / forestclaw

Quadtree/octree adaptive PDE solver based based on p4est.
http://www.forestclaw.org
BSD 2-Clause "Simplified" License
58 stars 21 forks source link

Fixes memleak #180

Closed donnaaboise closed 2 years ago

donnaaboise commented 2 years ago

This fixes the memory leak found in #178 . Fortran was allocating memory for an options array of variable length, but never deallocating it. I thought memory for allocatable arrays was automatically deallocated, but I guess valgrind doesn't think so. So I just allocated memory at compile time, with a check to make sure mwaves never exceeds 20 (size of static array).

cburstedde commented 2 years ago

Cool! That filament regression is a but annoying in the long run. Nothing to do with this one. ;)