Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GridGenerator: avoid a warning in release mode #14452

Merged
merged 2 commits into from
Nov 23, 2022
Merged

GridGenerator: avoid a warning in release mode #14452

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
184371e
GridGenerator: avoid a warning in release mode
tamiko Nov 23, 2022
d30557f
GridGenerators: modernize a cell loop and use range-based for
tamiko Nov 23, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
24 changes: 10 additions & 14 deletions source/grid/grid_generator.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1414,25 +1414,21 @@ namespace GridGenerator
}
else if (tria.n_cells() == 96)
{
// the 96-cell hypershell is
// based on a once refined
// 12-cell mesh. consequently,
// since the outer faces all
// are face_no==5 above, so
// they are here (unless they
// are in the interior). Use
// this to assign boundary
// indicators, but also make
// sure that we encounter
// exactly 48 such faces
// the 96-cell hypershell is based on a once refined 12-cell
// mesh. consequently, since the outer faces all are face_no==5
// above, so they are here (unless they are in the interior). Use
// this to assign boundary indicators, but also make sure that we
// encounter exactly 48 such faces
# ifdef DEBUG
unsigned int count = 0;
for (Triangulation<3>::cell_iterator cell = tria.begin();
cell != tria.end();
++cell)
# endif
for (const auto &cell : tria.cell_iterators())
if (cell->face(5)->at_boundary())
{
cell->face(5)->set_all_boundary_ids(1);
# ifdef DEBUG
++count;
# endif
}
Assert(count == 48, ExcInternalError());
}
Expand Down