diff options
author | ROUVREAU Vincent <vincent.rouvreau@inria.fr> | 2020-06-04 22:13:46 +0200 |
---|---|---|
committer | ROUVREAU Vincent <vincent.rouvreau@inria.fr> | 2020-06-04 22:13:46 +0200 |
commit | a81c5adcf4426338ff80cfb2f1973f113a17b16d (patch) | |
tree | 23591c4275107fbc623e4b75b9683b5f8ddea7d7 /src/Collapse/include/gudhi | |
parent | 85dc5c1e96c66e1505599eab2ee7e61174fd169d (diff) |
code review: better use a for loop
Diffstat (limited to 'src/Collapse/include/gudhi')
-rw-r--r-- | src/Collapse/include/gudhi/Flag_complex_sparse_matrix.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/Collapse/include/gudhi/Flag_complex_sparse_matrix.h b/src/Collapse/include/gudhi/Flag_complex_sparse_matrix.h index 75df52bf..67399044 100644 --- a/src/Collapse/include/gudhi/Flag_complex_sparse_matrix.h +++ b/src/Collapse/include/gudhi/Flag_complex_sparse_matrix.h @@ -339,7 +339,6 @@ class Flag_complex_sparse_matrix { */ template<typename FilteredEdgeOutput> void filtered_edge_collapse(FilteredEdgeOutput filtered_edge_output) { - Row_index endIdx = 0; // Sort edges auto sort_by_filtration = [](const Filtered_edge& edge_a, const Filtered_edge& edge_b) -> bool { @@ -352,7 +351,7 @@ class Flag_complex_sparse_matrix { std::sort(f_edge_vector_.begin(), f_edge_vector_.end(), sort_by_filtration); #endif - while (endIdx < f_edge_vector_.size()) { + for (Row_index endIdx = 0; endIdx < f_edge_vector_.size(); endIdx++) { Filtered_edge fec = f_edge_vector_[endIdx]; Edge edge = std::get<0>(fec); Vertex_handle u = std::get<0>(edge); @@ -371,7 +370,6 @@ class Flag_complex_sparse_matrix { if (endIdx > 1) set_edge_critical(endIdx, filt, filtered_edge_output); } - endIdx++; } } |