From 6dd018765b2ba866258a7e1c841e0310d24a77a9 Mon Sep 17 00:00:00 2001 From: Marco Feder Date: Wed, 28 Aug 2024 22:04:41 +0200 Subject: [PATCH] Remove unused variables. Improve comments --- include/agglomeration_accessor.h | 2 +- source/agglomeration_handler.cc | 41 ++++++++++---------------------- 2 files changed, 14 insertions(+), 29 deletions(-) diff --git a/include/agglomeration_accessor.h b/include/agglomeration_accessor.h index 222d7924..4e6256b1 100644 --- a/include/agglomeration_accessor.h +++ b/include/agglomeration_accessor.h @@ -415,7 +415,7 @@ AgglomerationAccessor::neighbor_of_agglomerated_neighbor( unsigned int n_faces_agglomerated_neighbor; - // if it's not locally owned, retrieve the number of faces + // if it is locally owned, retrieve the number of faces if (neigh_polytope->is_locally_owned()) { n_faces_agglomerated_neighbor = neigh_polytope->n_faces(); diff --git a/source/agglomeration_handler.cc b/source/agglomeration_handler.cc index 6485b6af..16e9f84b 100644 --- a/source/agglomeration_handler.cc +++ b/source/agglomeration_handler.cc @@ -845,11 +845,6 @@ namespace dealii for (const auto &[deal_cell, local_face_idx] : common_face) { - // std::cout << "deal cell = " << deal_cell->active_cell_index() - // << std::endl; - // std::cout << "local face index = " << local_face_idx << - // std::endl; - handler.no_face_values->reinit(deal_cell, local_face_idx); auto q_points = handler.no_face_values->get_quadrature_points(); @@ -1038,8 +1033,8 @@ namespace dealii .visited_cell_and_faces)) { handler.polytope_cache - .interface[{ - current_polytope_id, neighbor_polytope_id}] + .interface[{current_polytope_id, + neighbor_polytope_id}] .emplace_back(cell, f); handler.polytope_cache.visited_cell_and_faces @@ -1053,8 +1048,8 @@ namespace dealii .visited_cell_and_faces)) { handler.polytope_cache - .interface[{ - neighbor_polytope_id, current_polytope_id}] + .interface[{neighbor_polytope_id, + current_polytope_id}] .emplace_back(neighboring_cell, nof); handler.polytope_cache.visited_cell_and_faces @@ -1105,8 +1100,8 @@ namespace dealii .visited_cell_and_faces)) { handler.polytope_cache - .interface[{ - current_polytope_id, neighbor_polytope_id}] + .interface[{current_polytope_id, + neighbor_polytope_id}] .emplace_back(cell, f); handler.polytope_cache.visited_cell_and_faces @@ -1119,8 +1114,8 @@ namespace dealii .visited_cell_and_faces)) { handler.polytope_cache - .interface[{ - neighbor_polytope_id, current_polytope_id}] + .interface[{neighbor_polytope_id, + current_polytope_id}] .emplace_back(neighboring_cell, nof); handler.polytope_cache.visited_cell_and_faces @@ -1198,8 +1193,8 @@ namespace dealii handler.polytope_cache.visited_cell_and_faces_id)) { handler.polytope_cache - .interface[{ - current_polytope_id, check_neigh_polytope_id}] + .interface[{current_polytope_id, + check_neigh_polytope_id}] .emplace_back(cell, f); // std::cout << "ADDED (" @@ -1220,8 +1215,8 @@ namespace dealii handler.polytope_cache.visited_cell_and_faces_id)) { handler.polytope_cache - .interface[{ - check_neigh_polytope_id, current_polytope_id}] + .interface[{check_neigh_polytope_id, + current_polytope_id}] .emplace_back(neighboring_cell, nof); handler.polytope_cache.visited_cell_and_faces_id @@ -1237,8 +1232,6 @@ namespace dealii handler.polygon_boundary[master_cell].push_back( cell->face(f)); - - if (visited_polygonal_neighbors.find( std::numeric_limits::max()) == std::end(visited_polygonal_neighbors)) @@ -1251,17 +1244,10 @@ namespace dealii [current_polytope_index]}] = {true, neighboring_cell}; - - const unsigned int n_face = handler.number_of_agglomerated_faces [current_polytope_index]; - std::pair p{current_polytope_id, - n_face}; - - std::pair bdary_info{true, CellId()}; - is_face_at_boundary[n_face] = true; ++handler.number_of_agglomerated_faces @@ -1278,8 +1264,7 @@ namespace dealii std::end(handler.polytope_cache.visited_cell_and_faces)) { handler.polytope_cache - .interface[{ - current_polytope_id, current_polytope_id}] + .interface[{current_polytope_id, current_polytope_id}] .emplace_back(cell, f); handler.polytope_cache.visited_cell_and_faces.insert(