diff options
author | ROUVREAU Vincent <vincent.rouvreau@inria.fr> | 2020-07-31 15:23:40 +0200 |
---|---|---|
committer | ROUVREAU Vincent <vincent.rouvreau@inria.fr> | 2020-07-31 15:23:40 +0200 |
commit | 12c0266d7defec73a6e8da3990a7fc8d482cbde5 (patch) | |
tree | 6bd7c426f2e47f3abe3d49c85e7de78e53685bd1 /src/python/gudhi/simplex_tree.pyx | |
parent | cb79887df3f5155224ccc6c79d6a0e04853925ea (diff) |
code review: nogil and use ptr as suggested
Diffstat (limited to 'src/python/gudhi/simplex_tree.pyx')
-rw-r--r-- | src/python/gudhi/simplex_tree.pyx | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/python/gudhi/simplex_tree.pyx b/src/python/gudhi/simplex_tree.pyx index 236435a7..9eee5e14 100644 --- a/src/python/gudhi/simplex_tree.pyx +++ b/src/python/gudhi/simplex_tree.pyx @@ -593,8 +593,10 @@ cdef class SimplexTree: """ # Backup old pointer cdef Simplex_tree_interface_full_featured* ptr = self.get_ptr() - # New pointer is a new collapsed simplex tree - self.thisptr = <intptr_t>(self.get_ptr().collapse_edges(nb_iterations)) - # Delete old pointer - if ptr != NULL: - del ptr + cdef int nb_iter = nb_iterations + with nogil: + # New pointer is a new collapsed simplex tree + self.thisptr = <intptr_t>(ptr.collapse_edges(nb_iter)) + # Delete old pointer + if ptr != NULL: + del ptr |