Skip to content

Commit

Permalink
Sort out the always-included libmesh headers by reason for inclusion
Browse files Browse the repository at this point in the history
Try to remove mesh_tools.h for now
  • Loading branch information
GiudGiud committed Dec 5, 2024
1 parent f09f22a commit e14e914
Showing 1 changed file with 10 additions and 6 deletions.
16 changes: 10 additions & 6 deletions framework/include/base/libMeshReducedNamespace.h
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,20 @@

// This is the minimal set of headers to make all the types below that cannot be
// forward-declared as classes available.
// If we want to reduce this list, we would have to forward-declare functions as well

// These headers add constants that we want to add to the namespace.
#include "libmesh/libmesh.h"
#include "libmesh/remote_elem.h"

// This header declares functions that we want to add to the namespace. If we chose not to include
// this header, we will need to repeat the function declaration here
#include "libmesh/int_range.h" // for make_range + several typedefs

// These headers introduce typedefs that we are adding to the local namespace. If we chose not
// to include these headers, we will need to repeat the typedef here
#include "libmesh/id_types.h"
#include "libmesh/int_range.h"
#include "libmesh/vector_value.h"
#include "libmesh/tensor_value.h"
#include "libmesh/mesh_tools.h"

// For the remote_elem singleton
#include "libmesh/remote_elem.h"

// These are required for the range typedefs they contain, which cannot be forward-declared
// without re-defining the typedef. This allows us to add the ranges in the local namespace
Expand Down

0 comments on commit e14e914

Please sign in to comment.