[Getfem-commits] [getfem-commits] branch master updated: fix a memory leak

[Yves Renard]

This is an automated email from the git hooks/post-receive script.

renard pushed a commit to branch master
in repository getfem.

The following commit(s) were added to refs/heads/master by this push:
     new 3ae01b8  fix a memory leak
     new 7f934b5  Merge branch 'master' of ssh://git.sv.gnu.org:/srv/git/getfem
3ae01b8 is described below

commit 3ae01b87cf08fd253b4a5658a2481df9ee9ce336
Author: Yves Renard <Yves.Renard@insa-lyon.fr>
AuthorDate: Mon Dec 28 20:09:39 2020 +0100

    fix a memory leak
---
 src/getfem_generic_assembly_tree.cc | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/getfem_generic_assembly_tree.cc 
b/src/getfem_generic_assembly_tree.cc
index 37fb76e..d8cf672 100644
--- a/src/getfem_generic_assembly_tree.cc
+++ b/src/getfem_generic_assembly_tree.cc
@@ -1416,9 +1416,7 @@ namespace getfem {
                            pnode_old->parent,pnode->children[0]);
             GMM_ASSERT1(pnode_old->children.empty(), "Internal error");
             delete pnode_old;
-
           } else { // Macro with parameters
-
             if (gam.nb_params()+1 != pnode->children.size())
               ga_throw_error(pnode->expr, pnode->pos,
                              "Bad number of parameters in the use of macro '"
@@ -1433,6 +1431,7 @@ namespace getfem {
             else
               tree.root = pnode;
             ga_replace_macro_params(tree, pnode, pnode_old->children);
+            tree.clear_node_rec(pnode_old);
           }
         }
       }