diff --git a/NEWS.md b/NEWS.md index 3253822fda..017db53cd0 100644 --- a/NEWS.md +++ b/NEWS.md @@ -19,6 +19,16 @@ Please execute the steps in this order to ensure that your issue has the correct No! If your code is only a couple of lines long AND has very little impact on the algorithms of t8code (a single line of changed code can have a big impact) we encourage you to directly open a PR. If no issues are referenced using the Closes-keyword, an issue is automatically created and moved into "Needs Review". That way we shouldn't miss the opening of your PR. +# User Updates for the upcoming t8code release (February 2026 - version format unclear) + +## Using structs instead of classes + +We decided to use `struct` instead of `class` throughout the entire codebase. +We have some `C` code that is referring to our `C++` classes via pointers. This is only possible when these classes are declared as `struct`, since `C` does not know about `class`. +In order to prevent mixing styles, we decided to use `struct` everywhere. +Please keep in mind to use the `private` `protected` `public` keywords within your struct appropriately. + + # User Updates for the upcoming t8code v4.0.0 Among many minor changes, we have several major updates in t8code v4.0.0. diff --git a/src/t8_cad/t8_cad.hxx b/src/t8_cad/t8_cad.hxx index 82ddf19593..363a026c3e 100644 --- a/src/t8_cad/t8_cad.hxx +++ b/src/t8_cad/t8_cad.hxx @@ -36,7 +36,8 @@ /** * This class manages OpenCASCADE shapes and implements helper functions for working with the shapes. */ -class t8_cad { +struct t8_cad +{ public: /** * Constructor of the cad shape. diff --git a/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_conn_tree_to_vertex.hxx b/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_conn_tree_to_vertex.hxx index cda71ec35f..2b553c2f4d 100644 --- a/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_conn_tree_to_vertex.hxx +++ b/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_conn_tree_to_vertex.hxx @@ -47,12 +47,13 @@ #include /** forward declaration of the class needed since the two class headers include each other. */ -class t8_cmesh_vertex_conn_vertex_to_tree; +struct t8_cmesh_vertex_conn_vertex_to_tree; /** * A class to hold the tree to vertex connectivity of a cmesh. */ -class t8_cmesh_vertex_conn_tree_to_vertex { +struct t8_cmesh_vertex_conn_tree_to_vertex +{ public: /** Standard constructor. Does nothing. */ t8_cmesh_vertex_conn_tree_to_vertex (): current_state (state::EMPTY) @@ -169,7 +170,7 @@ class t8_cmesh_vertex_conn_tree_to_vertex { friend struct t8_cmesh_vertex_connectivity; private: - enum class state { + enum struct state { EMPTY, /*< Is initialized but empty. */ FILLED /*< Is filled with at least one entry. */ }; diff --git a/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_conn_vertex_to_tree.hxx b/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_conn_vertex_to_tree.hxx index 892d29423c..7ae880100d 100644 --- a/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_conn_vertex_to_tree.hxx +++ b/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_conn_vertex_to_tree.hxx @@ -35,7 +35,7 @@ #include /** forward declaration of ttv class needed since the two class headers include each other. */ -class t8_cmesh_vertex_conn_tree_to_vertex; +struct t8_cmesh_vertex_conn_tree_to_vertex; /** This class stores the vertex to tree lookup for * global vertex indices for a cmesh. @@ -67,7 +67,8 @@ class t8_cmesh_vertex_conn_tree_to_vertex; * Table global_id -> TV_LIST: std::unordered_map * */ -class t8_cmesh_vertex_conn_vertex_to_tree { +struct t8_cmesh_vertex_conn_vertex_to_tree +{ public: /** Standard constructor. * Initializes the class and allows setting vertex entries diff --git a/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_connectivity.hxx b/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_connectivity.hxx index c4c19c4884..af53065c58 100644 --- a/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_connectivity.hxx +++ b/src/t8_cmesh/t8_cmesh_vertex_connectivity/t8_cmesh_vertex_connectivity.hxx @@ -53,7 +53,7 @@ struct t8_cmesh_vertex_connectivity ~t8_cmesh_vertex_connectivity () {}; /** The state this connectivity can be in. */ - enum class state { + enum struct state { INITIALIZED, /**< Initialized but not filled */ TREE_TO_VERTEX_VALID, /**< Ready to use, but only tree_to_vertex functionality. */ VALID /**< Ready to use for full vertex connectivity. Cannot be altered anymore. */ diff --git a/src/t8_data/t8_data_handler.hxx b/src/t8_data/t8_data_handler.hxx index a152144f82..934dc5c969 100644 --- a/src/t8_data/t8_data_handler.hxx +++ b/src/t8_data/t8_data_handler.hxx @@ -56,7 +56,8 @@ is_internal_data (const t8_data_handler_type type) * \tparam TType The type of data to be handled. */ template -class t8_data_handler { +struct t8_data_handler +{ public: /** * Construct a t8_data_handler. diff --git a/src/t8_data/t8_element_array_iterator.hxx b/src/t8_data/t8_element_array_iterator.hxx index 96a1f3ddcc..069bfaa47e 100644 --- a/src/t8_data/t8_element_array_iterator.hxx +++ b/src/t8_data/t8_element_array_iterator.hxx @@ -44,7 +44,8 @@ * and let the dereference operator return it as a value_type. Therefore, read-only operations on the * \a t8_element_array_t are possible. */ -class t8_element_array_iterator { +struct t8_element_array_iterator +{ private: const t8_scheme* scheme; /**< The scheme of the elements residing within the array. */ diff --git a/src/t8_data/t8_vector_handler.hxx b/src/t8_data/t8_vector_handler.hxx index e41dce66a2..6f61b27f47 100644 --- a/src/t8_data/t8_vector_handler.hxx +++ b/src/t8_data/t8_vector_handler.hxx @@ -40,7 +40,8 @@ /** * A base class for vector handlers. */ -class t8_abstract_vector_handler { +struct t8_abstract_vector_handler +{ public: /** * Pure virtual function to determine the buffer size. @@ -140,7 +141,8 @@ class t8_abstract_vector_handler { * \tparam TType The type of data to be handled. */ template -class t8_vector_handler: public t8_abstract_vector_handler { +struct t8_vector_handler: public t8_abstract_vector_handler +{ public: /** * Construct a t8_vector_handler. diff --git a/src/t8_forest/t8_forest_pfc_message.hxx b/src/t8_forest/t8_forest_pfc_message.hxx index 477dbcea41..46ab106521 100644 --- a/src/t8_forest/t8_forest_pfc_message.hxx +++ b/src/t8_forest/t8_forest_pfc_message.hxx @@ -42,7 +42,8 @@ * between the processes that will be required to decide on whether and where * families are split at process boundaries. */ -class t8_forest_pfc_message { +struct t8_forest_pfc_message +{ public: /** * Pack the data to prepare sending. diff --git a/src/t8_forest/t8_forest_search/t8_forest_search.hxx b/src/t8_forest/t8_forest_search/t8_forest_search.hxx index 3c0aca08ce..2a5c985906 100644 --- a/src/t8_forest/t8_forest_search/t8_forest_search.hxx +++ b/src/t8_forest/t8_forest_search/t8_forest_search.hxx @@ -175,7 +175,8 @@ using t8_partition_search_batched_queries_callback = std::function -class t8_search: public t8_search_base { +struct t8_search: public t8_search_base +{ public: /** * Constructor for the t8_search class. @@ -451,7 +453,8 @@ class t8_search: public t8_search_base { * \tparam Udata The type of the user data, defaults to void. */ template -class t8_search_with_queries: public t8_search { +struct t8_search_with_queries: public t8_search +{ public: /** * Constructor for the t8_search_with_queries class. @@ -564,7 +567,8 @@ class t8_search_with_queries: public t8_search { * \tparam Udata The type of the user data, defaults to void. */ template -class t8_search_with_batched_queries: public t8_search { +struct t8_search_with_batched_queries: public t8_search +{ public: /** * Constructor for the t8_search_with_batched_queries class. @@ -668,7 +672,8 @@ class t8_search_with_batched_queries: public t8_search { /** * A class that performs a search in the partition of a forest. */ -class t8_partition_search_base { +struct t8_partition_search_base +{ public: /** Constructor for the t8_partition_search_base class. * @@ -792,7 +797,8 @@ class t8_partition_search_base { * \tparam Udata */ template -class t8_partition_search: public t8_partition_search_base { +struct t8_partition_search: public t8_partition_search_base +{ public: /** * Constructor for the t8_partition_search class. @@ -907,7 +913,8 @@ class t8_partition_search: public t8_partition_search_base { * \tparam Udata The type of the user data, defaults to void. */ template -class t8_partition_search_with_queries: public t8_partition_search { +struct t8_partition_search_with_queries: public t8_partition_search +{ public: /** * Constructor for the t8_partition_search_with_queries class. @@ -1027,7 +1034,8 @@ class t8_partition_search_with_queries: public t8_partition_search { * \tparam Udata The type of the user data, defaults to void. */ template -class t8_partition_search_with_batched_queries: public t8_partition_search { +struct t8_partition_search_with_batched_queries: public t8_partition_search +{ public: /** * Constructor for the t8_partition_search_with_batched_queries class. diff --git a/src/t8_geometry/t8_geometry_implementations/t8_geometry_lagrange.hxx b/src/t8_geometry/t8_geometry_implementations/t8_geometry_lagrange.hxx index 9c625511a9..b2e87fffd1 100644 --- a/src/t8_geometry/t8_geometry_implementations/t8_geometry_lagrange.hxx +++ b/src/t8_geometry/t8_geometry_implementations/t8_geometry_lagrange.hxx @@ -338,7 +338,8 @@ flatten (const std::vector> &vec) * some specific to the Lagrange geometry, some valid for all the * geometries in t8code. */ -class t8_lagrange_element { +struct t8_lagrange_element +{ public: /** * Construct a new t8_lagrange_element object. diff --git a/src/t8_schemes/t8_default/t8_default_common/t8_default_common.hxx b/src/t8_schemes/t8_default/t8_default_common/t8_default_common.hxx index 50459885c5..f1f8146ff2 100644 --- a/src/t8_schemes/t8_default/t8_default_common/t8_default_common.hxx +++ b/src/t8_schemes/t8_default/t8_default_common/t8_default_common.hxx @@ -89,7 +89,8 @@ count_leaves_from_level (const int element_level, const int refinement_level, co * \tparam TUnderlyingEclassScheme The default scheme class of the element shape. */ template -class t8_default_scheme_common: public t8_scheme_helpers { +struct t8_default_scheme_common: public t8_scheme_helpers +{ private: friend TUnderlyingEclassScheme; /** Private constructor which can only be used by derived schemes. diff --git a/src/t8_schemes/t8_default/t8_default_hex/t8_default_hex.hxx b/src/t8_schemes/t8_default/t8_default_hex/t8_default_hex.hxx index f86b5d2d7e..9c6052012c 100644 --- a/src/t8_schemes/t8_default/t8_default_hex/t8_default_hex.hxx +++ b/src/t8_schemes/t8_default/t8_default_hex/t8_default_hex.hxx @@ -34,7 +34,7 @@ #include /* Forward declaration of the scheme so we can use it as an argument in the eclass schemes function. */ -class t8_scheme; +struct t8_scheme; /** The class holding a hexahedral element in the default scheme. * We make this definition public for interoperability of element classes. @@ -43,7 +43,8 @@ class t8_scheme; typedef p8est_quadrant_t t8_phex_t; /** Default implementation of the scheme for the hex element class. */ -class t8_default_scheme_hex: public t8_default_scheme_common { +struct t8_default_scheme_hex: public t8_default_scheme_common +{ public: /** Constructor which calls the specialized constructor for the base. */ t8_default_scheme_hex () noexcept: t8_default_scheme_common (sizeof (t8_phex_t)) {}; diff --git a/src/t8_schemes/t8_default/t8_default_line/t8_default_line.hxx b/src/t8_schemes/t8_default/t8_default_line/t8_default_line.hxx index 68485b16d4..8732d7e542 100644 --- a/src/t8_schemes/t8_default/t8_default_line/t8_default_line.hxx +++ b/src/t8_schemes/t8_default/t8_default_line/t8_default_line.hxx @@ -34,13 +34,14 @@ #include /* Forward declaration of the scheme so we can use it as an argument in the eclass schemes function. */ -class t8_scheme; +struct t8_scheme; /** Provide an implementation for the line element class. * It is written as a self-contained library in the t8_dline_* files. */ -class t8_default_scheme_line: public t8_default_scheme_common { +struct t8_default_scheme_line: public t8_default_scheme_common +{ public: /** Constructor which calls the specialized constructor for the base. */ t8_default_scheme_line () noexcept: t8_default_scheme_common (sizeof (t8_dline_t)) {}; diff --git a/src/t8_schemes/t8_default/t8_default_prism/t8_default_prism.hxx b/src/t8_schemes/t8_default/t8_default_prism/t8_default_prism.hxx index a541fc8b6c..f07417b7b8 100644 --- a/src/t8_schemes/t8_default/t8_default_prism/t8_default_prism.hxx +++ b/src/t8_schemes/t8_default/t8_default_prism/t8_default_prism.hxx @@ -36,13 +36,14 @@ #include /* Forward declaration of the scheme so we can use it as an argument in the eclass schemes function. */ -class t8_scheme; +struct t8_scheme; /** Provide an implementation for the prism element class. * It is written as a self-contained library in the t8_dprism_* files. */ -class t8_default_scheme_prism: public t8_default_scheme_common { +struct t8_default_scheme_prism: public t8_default_scheme_common +{ public: /** Constructor which calls the specialized constructor for the base. */ t8_default_scheme_prism () noexcept: t8_default_scheme_common (sizeof (t8_dprism_t)) {}; diff --git a/src/t8_schemes/t8_default/t8_default_pyramid/t8_default_pyramid.hxx b/src/t8_schemes/t8_default/t8_default_pyramid/t8_default_pyramid.hxx index 62de778894..b60b9015ec 100644 --- a/src/t8_schemes/t8_default/t8_default_pyramid/t8_default_pyramid.hxx +++ b/src/t8_schemes/t8_default/t8_default_pyramid/t8_default_pyramid.hxx @@ -34,13 +34,14 @@ #include /* Forward declaration of the scheme so we can use it as an argument in the eclass schemes function. */ -class t8_scheme; +struct t8_scheme; /** Provide an implementation for the pyramid element class. It is written as a self-contained library in the * t8_dpyramid_* files. */ -class t8_default_scheme_pyramid: public t8_default_scheme_common { +struct t8_default_scheme_pyramid: public t8_default_scheme_common +{ public: /** Constructor which calls the specialized constructor for the base. */ t8_default_scheme_pyramid () noexcept: t8_default_scheme_common (sizeof (t8_dpyramid_t)) {}; diff --git a/src/t8_schemes/t8_default/t8_default_quad/t8_default_quad.hxx b/src/t8_schemes/t8_default/t8_default_quad/t8_default_quad.hxx index 16e1804626..a9f199ee9f 100644 --- a/src/t8_schemes/t8_default/t8_default_quad/t8_default_quad.hxx +++ b/src/t8_schemes/t8_default/t8_default_quad/t8_default_quad.hxx @@ -37,7 +37,7 @@ #include /* Forward declaration of the scheme so we can use it as an argument in the eclass schemes function. */ -class t8_scheme; +struct t8_scheme; /** The structure holding a quadrilateral element in the default scheme. * We make this definition public for interoperability of element classes. @@ -79,7 +79,8 @@ typedef p4est_quadrant_t t8_pquad_t; } while (0) /** Default implementation of the scheme for the quad element class. */ -class t8_default_scheme_quad: public t8_default_scheme_common { +struct t8_default_scheme_quad: public t8_default_scheme_common +{ public: /** Constructor which calls the specialized constructor for the base. */ t8_default_scheme_quad () noexcept: t8_default_scheme_common (sizeof (t8_pquad_t)) {}; diff --git a/src/t8_schemes/t8_default/t8_default_tet/t8_default_tet.hxx b/src/t8_schemes/t8_default/t8_default_tet/t8_default_tet.hxx index da816aa765..e5223e7261 100644 --- a/src/t8_schemes/t8_default/t8_default_tet/t8_default_tet.hxx +++ b/src/t8_schemes/t8_default/t8_default_tet/t8_default_tet.hxx @@ -35,10 +35,11 @@ #include /* Forward declaration of the scheme so we can use it as an argument in the eclass schemes function. */ -class t8_scheme; +struct t8_scheme; /** Default implementation of the scheme for the tet element class. */ -class t8_default_scheme_tet: public t8_default_scheme_common { +struct t8_default_scheme_tet: public t8_default_scheme_common +{ public: /** Constructor which calls the specialized constructor for the base. */ t8_default_scheme_tet () noexcept: t8_default_scheme_common (sizeof (t8_dtet_t)) {}; diff --git a/src/t8_schemes/t8_default/t8_default_tri/t8_default_tri.hxx b/src/t8_schemes/t8_default/t8_default_tri/t8_default_tri.hxx index d2ed1c4af1..cb346c612f 100644 --- a/src/t8_schemes/t8_default/t8_default_tri/t8_default_tri.hxx +++ b/src/t8_schemes/t8_default/t8_default_tri/t8_default_tri.hxx @@ -35,10 +35,11 @@ #include /* Forward declaration of the scheme so we can use it as an argument in the eclass schemes function. */ -class t8_scheme; +struct t8_scheme; /** Default implementation of the scheme for the triangular element class. */ -class t8_default_scheme_tri: public t8_default_scheme_common { +struct t8_default_scheme_tri: public t8_default_scheme_common +{ public: /** Constructor which calls the specialized constructor for the base. */ t8_default_scheme_tri () noexcept: t8_default_scheme_common (sizeof (t8_dtri_t)) {}; diff --git a/src/t8_schemes/t8_default/t8_default_vertex/t8_default_vertex.hxx b/src/t8_schemes/t8_default/t8_default_vertex/t8_default_vertex.hxx index cfe3fa0b62..987d993b40 100644 --- a/src/t8_schemes/t8_default/t8_default_vertex/t8_default_vertex.hxx +++ b/src/t8_schemes/t8_default/t8_default_vertex/t8_default_vertex.hxx @@ -32,10 +32,11 @@ #include /* Forward declaration of the scheme so we can use it as an argument in the eclass schemes function. */ -class t8_scheme; +struct t8_scheme; /** Default implementation of the scheme for the vertex element class. */ -class t8_default_scheme_vertex: public t8_default_scheme_common { +struct t8_default_scheme_vertex: public t8_default_scheme_common +{ public: /** Constructor which calls the specialized constructor for the base. */ t8_default_scheme_vertex () noexcept: t8_default_scheme_common (sizeof (t8_dvertex_t)) {}; diff --git a/src/t8_schemes/t8_scheme.hxx b/src/t8_schemes/t8_scheme.hxx index 7e5f7aabb5..be042b397a 100644 --- a/src/t8_schemes/t8_scheme.hxx +++ b/src/t8_schemes/t8_scheme.hxx @@ -64,8 +64,9 @@ t8_debug_print_type () /** This class holds one or more element schemes. * It also relays the function calls to the specific schemes. */ -class t8_scheme { - friend class t8_scheme_builder; +struct t8_scheme +{ + friend struct t8_scheme_builder; public: t8_scheme () diff --git a/src/t8_schemes/t8_scheme_builder.hxx b/src/t8_schemes/t8_scheme_builder.hxx index 269f1f10e6..2976626d61 100644 --- a/src/t8_schemes/t8_scheme_builder.hxx +++ b/src/t8_schemes/t8_scheme_builder.hxx @@ -33,7 +33,8 @@ /** The scheme builder adds eclass schemes to a scheme container and returns it. * TODO: Make return value a reference. */ -class t8_scheme_builder { +struct t8_scheme_builder +{ public: t8_scheme_builder (): scheme (new t8_scheme) {}; ~t8_scheme_builder () {}; diff --git a/src/t8_schemes/t8_scheme_helpers.hxx b/src/t8_schemes/t8_scheme_helpers.hxx index 318cc68eab..c557897dcd 100644 --- a/src/t8_schemes/t8_scheme_helpers.hxx +++ b/src/t8_schemes/t8_scheme_helpers.hxx @@ -37,7 +37,8 @@ * \tparam TUnderlyingEclassScheme The scheme this helper class is adding functionality to. */ template -class t8_scheme_helpers: public t8_crtp_basic { +struct t8_scheme_helpers: public t8_crtp_basic +{ protected: /** * Default constructor which is only accessible by derived classes. diff --git a/src/t8_types/t8_type.hxx b/src/t8_types/t8_type.hxx index 802289ceee..3a54bb55c7 100644 --- a/src/t8_types/t8_type.hxx +++ b/src/t8_types/t8_type.hxx @@ -44,7 +44,8 @@ * \tparam competence Variadic template parameter for the competences. */ template class... competence> -class T8Type: public competence>... { +struct T8Type: public competence>... +{ public: /** The type of the value stored in this strong type. */ using value_type = T; diff --git a/src/t8_vtk/t8_vtk_writer.hxx b/src/t8_vtk/t8_vtk_writer.hxx index a396fef9d6..cb1961792f 100644 --- a/src/t8_vtk/t8_vtk_writer.hxx +++ b/src/t8_vtk/t8_vtk_writer.hxx @@ -62,7 +62,8 @@ * \tparam grid_t can be a forest or a cmesh. */ template -class vtk_writer { +struct vtk_writer +{ public: /** * Construct a new vtk writer object. All parameters are set to false by default. By default no data is used and diff --git a/test/t8_IO/t8_gtest_vtk_reader.cxx b/test/t8_IO/t8_gtest_vtk_reader.cxx index 2ced67cd0c..f0c067d46a 100644 --- a/test/t8_IO/t8_gtest_vtk_reader.cxx +++ b/test/t8_IO/t8_gtest_vtk_reader.cxx @@ -36,7 +36,8 @@ const vtk_file_type_t gtest_vtk_filetypes[VTK_NUM_TYPES] = { VTK_FILE_ERROR, VTK_UNSTRUCTURED_FILE, VTK_POLYDATA_FILE, VTK_PARALLEL_UNSTRUCTURED_FILE, VTK_PARALLEL_POLYDATA_FILE }; -class vtk_reader: public testing::TestWithParam> { +struct vtk_reader: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_IO/t8_gtest_vtk_writer.cxx b/test/t8_IO/t8_gtest_vtk_writer.cxx index 3983d47f7a..d6ebcef0b2 100644 --- a/test/t8_IO/t8_gtest_vtk_writer.cxx +++ b/test/t8_IO/t8_gtest_vtk_writer.cxx @@ -140,7 +140,8 @@ vtk_writer_test_fill_data (const t8_locidx_t cells_to_write_count, std::vector -class vtk_writer_test: public testing::Test { +struct vtk_writer_test: public testing::Test +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_attribute_gloidx_array.cxx b/test/t8_cmesh/t8_gtest_attribute_gloidx_array.cxx index 0883463f3e..95ef05b134 100644 --- a/test/t8_cmesh/t8_gtest_attribute_gloidx_array.cxx +++ b/test/t8_cmesh/t8_gtest_attribute_gloidx_array.cxx @@ -35,7 +35,8 @@ #define T8_ATTRIBUTE_TEST_MAX_NUM_ENTRIES 1000 -class cmesh_attribute_gloidx_array: public testing::TestWithParam> { +struct cmesh_attribute_gloidx_array: public testing::TestWithParam> +{ protected: /* in Setup we build a two tree cmesh, fill an array with entries * and set the array as attribute for both trees with different data_persists settings. */ diff --git a/test/t8_cmesh/t8_gtest_bcast.cxx b/test/t8_cmesh/t8_gtest_bcast.cxx index 69f9b6417c..9883e7814b 100644 --- a/test/t8_cmesh/t8_gtest_bcast.cxx +++ b/test/t8_cmesh/t8_gtest_bcast.cxx @@ -31,7 +31,8 @@ #include #include -class cmesh_hypercube: public testing::TestWithParam { +struct cmesh_hypercube: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_add_attributes_when_derive.cxx b/test/t8_cmesh/t8_gtest_cmesh_add_attributes_when_derive.cxx index 20852ccacd..56beb7f586 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_add_attributes_when_derive.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_add_attributes_when_derive.cxx @@ -30,7 +30,8 @@ * cmesh is working. * This test is currently disabled, see https://github.com/DLR-AMR/t8code/issues/923 */ -class DISABLED_t8_cmesh_add_attributes: public testing::TestWithParam { +struct DISABLED_t8_cmesh_add_attributes: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_bounding_box.cxx b/test/t8_cmesh/t8_gtest_cmesh_bounding_box.cxx index 4cc50b89b8..0a1b8b0fd6 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_bounding_box.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_bounding_box.cxx @@ -35,7 +35,8 @@ * \brief Test fixture for testing the bounding box of a t8_cmesh. Computes a cmesh inside * the unit cube, computes the bounding box and checks that it is correct. */ -class t8_cmesh_bounding_box: public testing::TestWithParam { +struct t8_cmesh_bounding_box: public testing::TestWithParam +{ protected: void SetUp () override @@ -77,7 +78,8 @@ TEST_P (t8_cmesh_bounding_box, test_box) compute_and_check_bounds (cmesh, eclass); } -class t8_cmesh_bounding_box_multi_trees: public testing::TestWithParam> { +struct t8_cmesh_bounding_box_multi_trees: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_copy.cxx b/test/t8_cmesh/t8_gtest_cmesh_copy.cxx index bc8cc596ec..138ac70679 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_copy.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_copy.cxx @@ -34,7 +34,8 @@ * We test whether the new and original cmesh are equal. */ -class t8_cmesh_copy: public testing::TestWithParam { +struct t8_cmesh_copy: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_face_is_boundary.cxx b/test/t8_cmesh/t8_gtest_cmesh_face_is_boundary.cxx index 2de38b3dc3..dfa8285def 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_face_is_boundary.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_face_is_boundary.cxx @@ -27,7 +27,8 @@ #include /* Test for one tree */ -class cmesh_face_boundary_one_tree: public testing::TestWithParam { +struct cmesh_face_boundary_one_tree: public testing::TestWithParam +{ protected: void SetUp () override @@ -112,7 +113,8 @@ t8_test_compute_parallel_bounds (sc_MPI_Comm comm, t8_gloidx_t *first_tree, t8_g * Creates coarse meshes with two trees for each eclass, * one for each face of the first tree as the connecting face. * This, only the remaining trees should register as boundary trees. */ -class cmesh_face_boundary_two_trees: public testing::TestWithParam> { +struct cmesh_face_boundary_two_trees: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_partition.cxx b/test/t8_cmesh/t8_gtest_cmesh_partition.cxx index 938c3dcea5..78519f3744 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_partition.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_partition.cxx @@ -35,7 +35,8 @@ * passed. */ -class t8_cmesh_partition_class: public testing::TestWithParam> { +struct t8_cmesh_partition_class: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_set_join_by_vertices.cxx b/test/t8_cmesh/t8_gtest_cmesh_set_join_by_vertices.cxx index 1407bd2731..f5b5be0a1b 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_set_join_by_vertices.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_set_join_by_vertices.cxx @@ -295,7 +295,8 @@ TEST (t8_cmesh_set_join_by_vertices, test_cmesh_set_join_by_vertices) } } -class t8_cmesh_set_join_by_vertices_class: public testing::TestWithParam { +struct t8_cmesh_set_join_by_vertices_class: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_set_partition_offsets.cxx b/test/t8_cmesh/t8_gtest_cmesh_set_partition_offsets.cxx index 74699e3427..5e2e4a0a8e 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_set_partition_offsets.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_set_partition_offsets.cxx @@ -45,7 +45,8 @@ /* The tests that do not commit the cmesh iterate over the number of trees, * hence we have a TestWithParam with one int. */ -class cmesh_set_partition_offsets_nocommit: public testing::TestWithParam { +struct cmesh_set_partition_offsets_nocommit: public testing::TestWithParam +{ protected: void SetUp () override @@ -70,7 +71,8 @@ class cmesh_set_partition_offsets_nocommit: public testing::TestWithParam> { +struct cmesh_set_partition_offsets_commit: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_vertex_conn.cxx b/test/t8_cmesh/t8_gtest_cmesh_vertex_conn.cxx index 00b662346b..34a88570b8 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_vertex_conn.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_vertex_conn.cxx @@ -57,7 +57,8 @@ * Note that the test itself then has to be set to parallel in the CMake file. */ -class t8_test_cmesh_vertex_conn: public testing::Test { +struct t8_test_cmesh_vertex_conn: public testing::Test +{ protected: void SetUp () override @@ -213,7 +214,8 @@ TEST_F (t8_test_cmesh_vertex_conn, check_global_vertex_number) /* Parallel test suite (to be extended) is currently disabled since * the cmesh vertex connecticity does not support partitioned cmeshes. */ -class t8_test_cmesh_vertex_conn_partitioned: public testing::Test { +struct t8_test_cmesh_vertex_conn_partitioned: public testing::Test +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_vertex_conn_tree_to_vertex.cxx b/test/t8_cmesh/t8_gtest_cmesh_vertex_conn_tree_to_vertex.cxx index 94982b3119..2c103fe77c 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_vertex_conn_tree_to_vertex.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_vertex_conn_tree_to_vertex.cxx @@ -71,7 +71,8 @@ Note to future developers: the _temp tests should be replaced with the non-temp test suites.. */ -class cmesh_vertex_conn_ttv_with_core_classes: public testing::TestWithParam { +struct cmesh_vertex_conn_ttv_with_core_classes: public testing::TestWithParam +{ protected: void SetUp () override @@ -153,8 +154,8 @@ TEST_P (cmesh_vertex_conn_ttv_with_core_classes, DISABLED_get_global) #define VTT_TEST_MAX_NUM_TREES 100 #endif -class cmesh_vertex_conn_ttv_with_core_classes_temp: - public testing::TestWithParam> { +struct cmesh_vertex_conn_ttv_with_core_classes_temp: public testing::TestWithParam> +{ protected: void SetUp () override @@ -268,7 +269,8 @@ INSTANTIATE_TEST_SUITE_P (t8_gtest_cmesh_vertex_tree_to_vertex, cmesh_vertex_con INSTANTIATE_TEST_SUITE_P (t8_gtest_cmesh_vertex_tree_to_vertex, cmesh_vertex_conn_ttv_with_core_classes_temp, testing::Combine (testing::Values (1, VTT_TEST_MAX_NUM_TREES + 1), AllEclasses)); -class cmesh_vertex_conn_ttv_with_cmesh_functions: public testing::TestWithParam { +struct cmesh_vertex_conn_ttv_with_cmesh_functions: public testing::TestWithParam +{ protected: void SetUp () override @@ -342,8 +344,9 @@ TEST_P (cmesh_vertex_conn_ttv_with_cmesh_functions, DISABLED_get_global) * as soon as we can enable the tests cmesh_vertex_conn_ttv. * That is as soon as we can add attributes to cmeshes while deriving. */ -class cmesh_vertex_conn_ttv_with_cmesh_functions_temp: - public testing::TestWithParam> { +struct cmesh_vertex_conn_ttv_with_cmesh_functions_temp: + public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_cmesh_vertex_conn_vertex_to_tree.cxx b/test/t8_cmesh/t8_gtest_cmesh_vertex_conn_vertex_to_tree.cxx index f6e8cf6d11..9bba5b760d 100644 --- a/test/t8_cmesh/t8_gtest_cmesh_vertex_conn_vertex_to_tree.cxx +++ b/test/t8_cmesh/t8_gtest_cmesh_vertex_conn_vertex_to_tree.cxx @@ -57,7 +57,8 @@ t8_compute_global_vertex_hash (t8_locidx_t itree, t8_locidx_t ivertex, t8_locidx return (itree * ivertex) % (num_local_trees + 1); } -class t8_test_cmesh_vertex_conn_vtt: public testing::TestWithParam { +struct t8_test_cmesh_vertex_conn_vtt: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_compute_first_element.cxx b/test/t8_cmesh/t8_gtest_compute_first_element.cxx index 9631b60d30..9ba0ca1f98 100644 --- a/test/t8_cmesh/t8_gtest_compute_first_element.cxx +++ b/test/t8_cmesh/t8_gtest_compute_first_element.cxx @@ -39,7 +39,8 @@ #include #include -class t8_gtest_rank_times_global_num_elems_over_size: public testing::TestWithParam> { +struct t8_gtest_rank_times_global_num_elems_over_size: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_cmesh/t8_gtest_hypercube.cxx b/test/t8_cmesh/t8_gtest_hypercube.cxx index 3af7fc253b..8e489c3928 100644 --- a/test/t8_cmesh/t8_gtest_hypercube.cxx +++ b/test/t8_cmesh/t8_gtest_hypercube.cxx @@ -30,7 +30,8 @@ #include /* Create class for parameterized Test with multiple test parameters */ -class cmesh_hypercube_trees: public testing::TestWithParam> { +struct cmesh_hypercube_trees: public testing::TestWithParam> +{ protected: /* SetUp the test parameters (eclass, bcast and partition) and define the test value cmesh. */ void diff --git a/test/t8_cmesh/t8_gtest_multiple_attributes.cxx b/test/t8_cmesh/t8_gtest_multiple_attributes.cxx index 4776353074..629bdfd66b 100644 --- a/test/t8_cmesh/t8_gtest_multiple_attributes.cxx +++ b/test/t8_cmesh/t8_gtest_multiple_attributes.cxx @@ -42,7 +42,8 @@ t8_cmesh_partition_cmesh (t8_cmesh_t cmesh, const t8_scheme *scheme, sc_MPI_Comm return cmesh_partition; } -class cmesh_multiple_attributes: public testing::TestWithParam> { +struct cmesh_multiple_attributes: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_cmesh_generator/t8_gtest_cmesh_cartestian_product.hxx b/test/t8_cmesh_generator/t8_gtest_cmesh_cartestian_product.hxx index 0c9215f938..a301a43df5 100644 --- a/test/t8_cmesh_generator/t8_gtest_cmesh_cartestian_product.hxx +++ b/test/t8_cmesh_generator/t8_gtest_cmesh_cartestian_product.hxx @@ -38,7 +38,8 @@ * The * */ -class cmesh_example_base { +struct cmesh_example_base +{ public: /** * Construct a new base example. An example must have at least have a name. @@ -74,7 +75,8 @@ class cmesh_example_base { * @tparam Args */ template -class cmesh_example_with_parameter: cmesh_example_base { +struct cmesh_example_with_parameter: cmesh_example_base +{ public: cmesh_example_with_parameter (std::function function, std::tuple parameter, std::function parameter_to_string, std::string name) @@ -102,7 +104,8 @@ class cmesh_example_with_parameter: cmesh_example_base { * A base class to hold sets of examples that can be created in various ways. * */ -class example_set { +struct example_set +{ public: /** * Generate a cmesh according to a function @@ -212,7 +215,8 @@ cartesian_product (OutputIterator out, std::function -class cmesh_cartesian_product_params: example_set { +struct cmesh_cartesian_product_params: example_set +{ public: cmesh_cartesian_product_params () {}; @@ -261,7 +265,8 @@ class cmesh_cartesian_product_params: example_set { * @tparam Iter */ template -class cmesh_cartesian_product_with_rules: example_set { +struct cmesh_cartesian_product_with_rules: example_set +{ public: cmesh_cartesian_product_with_rules () {}; diff --git a/test/t8_cmesh_generator/t8_gtest_cmesh_generator_test.cxx b/test/t8_cmesh_generator/t8_gtest_cmesh_generator_test.cxx index 391b1b42b0..5395194bf7 100644 --- a/test/t8_cmesh_generator/t8_gtest_cmesh_generator_test.cxx +++ b/test/t8_cmesh_generator/t8_gtest_cmesh_generator_test.cxx @@ -26,7 +26,8 @@ #include "test/t8_cmesh_generator/t8_cmesh_parameterized_examples/t8_cmesh_new_bigmesh_param.hxx" #include "test/t8_cmesh_generator/t8_gtest_cmesh_cartestian_product.hxx" -class t8_cmesh_iter: public testing::TestWithParam { +struct t8_cmesh_iter: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_cmesh_generator/t8_gtest_cmesh_sum_of_sets.hxx b/test/t8_cmesh_generator/t8_gtest_cmesh_sum_of_sets.hxx index ad0bd9e2db..5530836c69 100644 --- a/test/t8_cmesh_generator/t8_gtest_cmesh_sum_of_sets.hxx +++ b/test/t8_cmesh_generator/t8_gtest_cmesh_sum_of_sets.hxx @@ -31,7 +31,8 @@ T8_EXTERN_C_BEGIN (); * A class that holds multiple ways to create a cmesh. * */ -class cmesh_sum_of_sets { +struct cmesh_sum_of_sets +{ public: cmesh_sum_of_sets () {}; /** diff --git a/test/t8_data/t8_data_handler_specs.hxx b/test/t8_data/t8_data_handler_specs.hxx index 56a4b15af5..6b07d340e7 100644 --- a/test/t8_data/t8_data_handler_specs.hxx +++ b/test/t8_data/t8_data_handler_specs.hxx @@ -43,7 +43,8 @@ along with t8code; if not, write to the Free Software Foundation, Inc., * */ template -class enlarged_data { +struct enlarged_data +{ public: /** * Default constructor. @@ -73,7 +74,8 @@ class enlarged_data { * \tparam TType The type of the data to be stored. */ template -class data_creator { +struct data_creator +{ public: data_creator () { diff --git a/test/t8_data/t8_enlarged_stdtypes.hxx b/test/t8_data/t8_enlarged_stdtypes.hxx index 579efd0886..2d4f56157f 100644 --- a/test/t8_data/t8_enlarged_stdtypes.hxx +++ b/test/t8_data/t8_enlarged_stdtypes.hxx @@ -47,7 +47,8 @@ enum pseudo_types { T8_ENLARGED_INT = 0, T8_ENLARGED_DOUBLE = 1 }; * */ template <> -class t8_data_handler> { +struct t8_data_handler> +{ public: /** * Returns the size of an enlarged_int. @@ -126,7 +127,8 @@ class t8_data_handler> { * */ template <> -class t8_data_handler> { +struct t8_data_handler> +{ public: /** * Returns the size of an enlarged_double. diff --git a/test/t8_data/t8_gtest_data_handler.cxx b/test/t8_data/t8_gtest_data_handler.cxx index d6c4ce9b1e..effbd45ff2 100644 --- a/test/t8_data/t8_gtest_data_handler.cxx +++ b/test/t8_data/t8_gtest_data_handler.cxx @@ -42,7 +42,8 @@ along with t8code; if not, write to the Free Software Foundation, Inc., * \tparam T the type of data */ template -class data_handler_test: public testing::Test { +struct data_handler_test: public testing::Test +{ protected: void SetUp () override diff --git a/test/t8_data/t8_gtest_shmem.cxx b/test/t8_data/t8_gtest_shmem.cxx index d2594bd668..871f47c292 100644 --- a/test/t8_data/t8_gtest_shmem.cxx +++ b/test/t8_data/t8_gtest_shmem.cxx @@ -29,7 +29,8 @@ #define T8_TEST_SHMEM_NUM_COMMS 2 -class shmem: public testing::TestWithParam> { +struct shmem: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_data/t8_pseudo_trees.hxx b/test/t8_data/t8_pseudo_trees.hxx index 14bdb8d028..29cb2e3b1d 100644 --- a/test/t8_data/t8_pseudo_trees.hxx +++ b/test/t8_data/t8_pseudo_trees.hxx @@ -44,14 +44,16 @@ along with t8code; if not, write to the Free Software Foundation, Inc., * - `topo_data`: A vector of integers representing topological data. * - `tree_data`: A vector of pointers to t8_abstract_data_handler objects, representing tree data. */ -class pseudo_tree { +struct pseudo_tree +{ public: std::vector topo_data; std::vector> tree_data; }; template <> -class t8_data_handler { +struct t8_data_handler +{ public: /** * Returns the size of a pseudo_tree. diff --git a/test/t8_forest/t8_gtest_balance.cxx b/test/t8_forest/t8_gtest_balance.cxx index 4589c46581..464ef97ff9 100644 --- a/test/t8_forest/t8_gtest_balance.cxx +++ b/test/t8_forest/t8_gtest_balance.cxx @@ -41,7 +41,8 @@ #include #include -class gtest_balance: public testing::TestWithParam, int, int>> { +struct gtest_balance: public testing::TestWithParam, int, int>> +{ public: static const int kNumTrees = 4; diff --git a/test/t8_forest/t8_gtest_element_is_leaf.cxx b/test/t8_forest/t8_gtest_element_is_leaf.cxx index 835b899421..f19af2371c 100644 --- a/test/t8_forest/t8_gtest_element_is_leaf.cxx +++ b/test/t8_forest/t8_gtest_element_is_leaf.cxx @@ -69,7 +69,8 @@ t8_test_adapt_first_child (t8_forest_t forest, [[maybe_unused]] t8_forest_t fore return 0; } -class element_is_leaf: public testing::TestWithParam, int>> { +struct element_is_leaf: public testing::TestWithParam, int>> +{ protected: void SetUp () override @@ -105,7 +106,8 @@ class element_is_leaf: public testing::TestWithParam { +struct element_is_leaf_hybrid: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_element_volume.cxx b/test/t8_forest/t8_gtest_element_volume.cxx index dce3586827..43efbf5b01 100644 --- a/test/t8_forest/t8_gtest_element_volume.cxx +++ b/test/t8_forest/t8_gtest_element_volume.cxx @@ -38,7 +38,8 @@ /* Construct a forest of a hypercube with volume 1. If the element are refined uniformly * all elements have volume 1/global_num_elements. */ -class t8_forest_volume: public testing::TestWithParam, int>> { +struct t8_forest_volume: public testing::TestWithParam, int>> +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_find_owner.cxx b/test/t8_forest/t8_gtest_find_owner.cxx index 323c929fc4..c2a3acc84f 100644 --- a/test/t8_forest/t8_gtest_find_owner.cxx +++ b/test/t8_forest/t8_gtest_find_owner.cxx @@ -34,7 +34,8 @@ #include #include -class forest_find_owner: public testing::TestWithParam> { +struct forest_find_owner: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_forest_commit.cxx b/test/t8_forest/t8_gtest_forest_commit.cxx index b944fb68f1..7234ecc6db 100644 --- a/test/t8_forest/t8_gtest_forest_commit.cxx +++ b/test/t8_forest/t8_gtest_forest_commit.cxx @@ -39,7 +39,8 @@ * After these two forests are created, we check for equality. */ -class forest_commit: public testing::TestWithParam> { +struct forest_commit: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_forest_face_normal.cxx b/test/t8_forest/t8_gtest_forest_face_normal.cxx index 4fc8a3e481..13f6d1a4a8 100644 --- a/test/t8_forest/t8_gtest_forest_face_normal.cxx +++ b/test/t8_forest/t8_gtest_forest_face_normal.cxx @@ -34,7 +34,8 @@ * This file tests the face normal computation of elements. */ -class class_forest_face_normal: public testing::TestWithParam, int>> { +struct class_forest_face_normal: public testing::TestWithParam, int>> +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_ghost_and_owner.cxx b/test/t8_forest/t8_gtest_ghost_and_owner.cxx index 7bf26eeb7e..69dd9eae61 100644 --- a/test/t8_forest/t8_gtest_ghost_and_owner.cxx +++ b/test/t8_forest/t8_gtest_ghost_and_owner.cxx @@ -36,8 +36,8 @@ * parse through all ghost elements and test whether the owner of an * element is in face the owner that is stored in the ghost layer. */ - -class forest_ghost_owner: public testing::TestWithParam> { +struct forest_ghost_owner: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_ghost_delete.cxx b/test/t8_forest/t8_gtest_ghost_delete.cxx index 3efb39e071..ea7e37ce71 100644 --- a/test/t8_forest/t8_gtest_ghost_delete.cxx +++ b/test/t8_forest/t8_gtest_ghost_delete.cxx @@ -56,7 +56,8 @@ test_adapt_holes ([[maybe_unused]] t8_forest_t forest, t8_forest_t forest_from, return 0; } -class DISABLED_forest_ghost_exchange_holes: public testing::TestWithParam { +struct DISABLED_forest_ghost_exchange_holes: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_ghost_exchange.cxx b/test/t8_forest/t8_gtest_ghost_exchange.cxx index ad077f6fc9..4c05fe6db8 100644 --- a/test/t8_forest/t8_gtest_ghost_exchange.cxx +++ b/test/t8_forest/t8_gtest_ghost_exchange.cxx @@ -42,7 +42,8 @@ * in a second test, we store the element's linear id in the data array. */ -class forest_ghost_exchange: public testing::TestWithParam> { +struct forest_ghost_exchange: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_half_neighbors.cxx b/test/t8_forest/t8_gtest_half_neighbors.cxx index 9873e04ef0..835c0229f3 100644 --- a/test/t8_forest/t8_gtest_half_neighbors.cxx +++ b/test/t8_forest/t8_gtest_half_neighbors.cxx @@ -35,7 +35,8 @@ #include #include -class forest_half_neighbors: public testing::TestWithParam, int>> { +struct forest_half_neighbors: public testing::TestWithParam, int>> +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_partition_data.cxx b/test/t8_forest/t8_gtest_partition_data.cxx index 59fa047de5..d688af287e 100644 --- a/test/t8_forest/t8_gtest_partition_data.cxx +++ b/test/t8_forest/t8_gtest_partition_data.cxx @@ -46,7 +46,8 @@ * It provides some operator overloads in order to be handled equal to arithmetic data types within the * function \see TestPartitionData. */ -class t8_test_partition_data_t { +struct t8_test_partition_data_t +{ public: t8_test_partition_data_t () = default; @@ -218,7 +219,8 @@ t8_test_partition_data_adapt ([[maybe_unused]] t8_forest_t forest, t8_forest_t f } } -class t8_test_partition_data_test: public testing::TestWithParam> { +struct t8_test_partition_data_test: public testing::TestWithParam> +{ protected: void diff --git a/test/t8_forest/t8_gtest_partition_for_coarsening.cxx b/test/t8_forest/t8_gtest_partition_for_coarsening.cxx index 58ef50de21..bda11ed815 100644 --- a/test/t8_forest/t8_gtest_partition_for_coarsening.cxx +++ b/test/t8_forest/t8_gtest_partition_for_coarsening.cxx @@ -133,7 +133,8 @@ coarsen_all_callback ([[maybe_unused]] t8_forest_t forest, [[maybe_unused]] t8_f /** * Class to test the partition-for-coarsening functionality. */ -class t8_test_partition_for_coarsening_test: public testing::TestWithParam> { +struct t8_test_partition_for_coarsening_test: public testing::TestWithParam> +{ protected: /** During SetUp, set the scheme and the eclass based on the current testing parameters.*/ diff --git a/test/t8_forest/t8_gtest_search.cxx b/test/t8_forest/t8_gtest_search.cxx index b15697069e..214021dec6 100644 --- a/test/t8_forest/t8_gtest_search.cxx +++ b/test/t8_forest/t8_gtest_search.cxx @@ -31,7 +31,8 @@ #include #include -class forest_search: public testing::TestWithParam, int>> { +struct forest_search: public testing::TestWithParam, int>> +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_set_partition_offset.cxx b/test/t8_forest/t8_gtest_set_partition_offset.cxx index d705248971..cc2bfc4e57 100644 --- a/test/t8_forest/t8_gtest_set_partition_offset.cxx +++ b/test/t8_forest/t8_gtest_set_partition_offset.cxx @@ -65,7 +65,8 @@ /** * Test class for validating \ref t8_forest_set_partition_offset and \ref t8_forest_new_gather. */ -class t8_test_set_partition_offset_test: public testing::TestWithParam> { +struct t8_test_set_partition_offset_test: public testing::TestWithParam> +{ protected: /** diff --git a/test/t8_forest/t8_gtest_transform.cxx b/test/t8_forest/t8_gtest_transform.cxx index eede68d82c..66593080a7 100644 --- a/test/t8_forest/t8_gtest_transform.cxx +++ b/test/t8_forest/t8_gtest_transform.cxx @@ -39,7 +39,8 @@ #include #include -class forest_transform: public testing::TestWithParam, int>> { +struct forest_transform: public testing::TestWithParam, int>> +{ protected: void SetUp () override diff --git a/test/t8_forest/t8_gtest_user_data.cxx b/test/t8_forest/t8_gtest_user_data.cxx index 5ea310625a..edf39f8176 100644 --- a/test/t8_forest/t8_gtest_user_data.cxx +++ b/test/t8_forest/t8_gtest_user_data.cxx @@ -35,7 +35,8 @@ * We build a forest and set user data for it. * We then retrieve the data and check whether it is the same. */ -class forest_user_data: public testing::TestWithParam { +struct forest_user_data: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_forest_incomplete/t8_gtest_empty_global_tree.cxx b/test/t8_forest_incomplete/t8_gtest_empty_global_tree.cxx index f7dd31cfff..5014a199ce 100644 --- a/test/t8_forest_incomplete/t8_gtest_empty_global_tree.cxx +++ b/test/t8_forest_incomplete/t8_gtest_empty_global_tree.cxx @@ -38,7 +38,8 @@ * */ /* Remove `DISABLED_` from the name of the Test(suite) or use `--gtest_also_run_disabled_tests` when you start working on the issue. */ -class DISABLED_global_tree: public testing::TestWithParam> { +struct DISABLED_global_tree: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_forest_incomplete/t8_gtest_empty_local_tree.cxx b/test/t8_forest_incomplete/t8_gtest_empty_local_tree.cxx index d88d41845e..d802ef6866 100644 --- a/test/t8_forest_incomplete/t8_gtest_empty_local_tree.cxx +++ b/test/t8_forest_incomplete/t8_gtest_empty_local_tree.cxx @@ -56,7 +56,8 @@ /** This test covers the functionality described in Issue: https://github.com/DLR-AMR/t8code/issues/1137 * Remove `DISABLED_` from the name of the Test(suite) or use `--gtest_also_run_disabled_tests` when you start working on the issue. */ -class DISABLED_local_tree: public testing::TestWithParam { +struct DISABLED_local_tree: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_forest_incomplete/t8_gtest_iterate_replace.cxx b/test/t8_forest_incomplete/t8_gtest_iterate_replace.cxx index c690353dfe..ca69023e74 100644 --- a/test/t8_forest_incomplete/t8_gtest_iterate_replace.cxx +++ b/test/t8_forest_incomplete/t8_gtest_iterate_replace.cxx @@ -35,7 +35,8 @@ * t8_forest_iterate_replace if it is passed the correct values. */ -class forest_iterate: public testing::TestWithParam { +struct forest_iterate: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_forest_incomplete/t8_gtest_permute_hole.cxx b/test/t8_forest_incomplete/t8_gtest_permute_hole.cxx index 6347a7fdb4..43fcc42ba3 100644 --- a/test/t8_forest_incomplete/t8_gtest_permute_hole.cxx +++ b/test/t8_forest_incomplete/t8_gtest_permute_hole.cxx @@ -59,7 +59,8 @@ * Note, this test runs only on one rank. */ -class forest_permute: public testing::TestWithParam { +struct forest_permute: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_forest_incomplete/t8_gtest_recursive.cxx b/test/t8_forest_incomplete/t8_gtest_recursive.cxx index d0b2a5ad71..bb41f0f8f0 100644 --- a/test/t8_forest_incomplete/t8_gtest_recursive.cxx +++ b/test/t8_forest_incomplete/t8_gtest_recursive.cxx @@ -37,7 +37,8 @@ * Note, that each rank has its own local/global tree. No trees are shared. */ -class recursive_tree: public testing::TestWithParam> { +struct recursive_tree: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_cad.cxx b/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_cad.cxx index 0091d6c610..c3f8584e57 100644 --- a/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_cad.cxx +++ b/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_cad.cxx @@ -544,7 +544,8 @@ TEST (t8_gtest_geometry_cad, jacobian) /* The test checks if the mapping algorithms for curved 2d elements do not shift values on an edge which is not curved. * In that case, the cad geometry should output the same out_coords as the linear geometry function. */ -class class_2d_element_cad_curve: public testing::TestWithParam> { +struct class_2d_element_cad_curve: public testing::TestWithParam> +{ protected: void SetUp () override @@ -688,7 +689,8 @@ INSTANTIATE_TEST_SUITE_P (t8_gtest_check_2d_element_cad_curve, class_2d_element_ /* The test checks if the mapping algorithms for curved 2d elements do not shift values on a surface which is not curved. * In that case, the cad geometry should output the same out_coords as the linear geometry function. */ -class class_2d_element_linear_cad_surface: public testing::TestWithParam { +struct class_2d_element_linear_cad_surface: public testing::TestWithParam +{ protected: void SetUp () override @@ -786,7 +788,8 @@ INSTANTIATE_TEST_SUITE_P (t8_gtest_check_2d_element_linear_cad_surface, class_2d AllEclasses2D, print_eclass); /* The test checks if the mapping algorithms for curved 2d elements shift values on a curved surface correctly. */ -class class_2d_element_curved_cad_surface: public testing::TestWithParam { +struct class_2d_element_curved_cad_surface: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_lagrange.cxx b/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_lagrange.cxx index c327f309d1..778df8c415 100644 --- a/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_lagrange.cxx +++ b/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_lagrange.cxx @@ -199,7 +199,7 @@ create_sample_element (t8_eclass_t eclass, int degree) * Common resources for all the tests. * */ -class LagrangeCmesh: public testing::TestWithParam> { +struct LagrangeCmesh: public testing::TestWithParam> { protected: void SetUp () override diff --git a/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_linear.cxx b/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_linear.cxx index 1a76bef7c5..a0546cfaaf 100644 --- a/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_linear.cxx +++ b/test/t8_geometry/t8_geometry_implementations/t8_gtest_geometry_linear.cxx @@ -36,7 +36,8 @@ #include #include -class geometry_test: public testing::TestWithParam> { +struct geometry_test: public testing::TestWithParam> +{ public: static void SetUpTestSuite () diff --git a/test/t8_geometry/t8_gtest_geometry_negative_volume.cxx b/test/t8_geometry/t8_gtest_geometry_negative_volume.cxx index 5771dc02a6..1b218bff12 100644 --- a/test/t8_geometry/t8_gtest_geometry_negative_volume.cxx +++ b/test/t8_geometry/t8_gtest_geometry_negative_volume.cxx @@ -32,7 +32,8 @@ #include #include -class check_negative_volume: public testing::TestWithParam { +struct check_negative_volume: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_geometry/t8_gtest_point_inside.cxx b/test/t8_geometry/t8_gtest_point_inside.cxx index 71eddde1db..d7f80d64b3 100644 --- a/test/t8_geometry/t8_gtest_point_inside.cxx +++ b/test/t8_geometry/t8_gtest_point_inside.cxx @@ -117,7 +117,8 @@ TEST (t8_point_inside, test_point_inside_specific_quad) t8_forest_unref (&forest); } -class geometry_point_inside: public testing::TestWithParam> { +struct geometry_point_inside: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_gtest_eclass.cxx b/test/t8_gtest_eclass.cxx index 7040571e61..31f112cf55 100644 --- a/test/t8_gtest_eclass.cxx +++ b/test/t8_gtest_eclass.cxx @@ -27,7 +27,8 @@ #include #include -class gtest_eclass: public testing::TestWithParam { +struct gtest_eclass: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_helper_functions/t8_gtest_vector_split.cxx b/test/t8_helper_functions/t8_gtest_vector_split.cxx index c4d70759f9..b5fdde812c 100644 --- a/test/t8_helper_functions/t8_gtest_vector_split.cxx +++ b/test/t8_helper_functions/t8_gtest_vector_split.cxx @@ -34,7 +34,8 @@ split (const int value, const int div) return value / div; } -class test_vector_split: public testing::TestWithParam { +struct test_vector_split: public testing::TestWithParam +{ public: void SetUp () override diff --git a/test/t8_schemes/t8_gtest_ancestor.cxx b/test/t8_schemes/t8_gtest_ancestor.cxx index 3c3bf587ae..5b48eb596d 100644 --- a/test/t8_schemes/t8_gtest_ancestor.cxx +++ b/test/t8_schemes/t8_gtest_ancestor.cxx @@ -32,7 +32,8 @@ #include #include -class ancestor: public testing::TestWithParam { +struct ancestor: public testing::TestWithParam +{ protected: void SetUp () override diff --git a/test/t8_schemes/t8_gtest_ancestor_id.cxx b/test/t8_schemes/t8_gtest_ancestor_id.cxx index 320719e2c3..7a0c01a92a 100644 --- a/test/t8_schemes/t8_gtest_ancestor_id.cxx +++ b/test/t8_schemes/t8_gtest_ancestor_id.cxx @@ -33,7 +33,9 @@ #include "t8_gtest_dfs_base.hxx" #include -class class_ancestor_id: public TestDFS { +struct class_ancestor_id: public TestDFS +{ + private: void check_element () override { diff --git a/test/t8_schemes/t8_gtest_bfs_base.hxx b/test/t8_schemes/t8_gtest_bfs_base.hxx index a1f37283ff..49de65b599 100644 --- a/test/t8_schemes/t8_gtest_bfs_base.hxx +++ b/test/t8_schemes/t8_gtest_bfs_base.hxx @@ -33,7 +33,8 @@ #include #include -class TestBFS: public testing::TestWithParam> { +struct TestBFS: public testing::TestWithParam> +{ public: /** recursive tests check something for all descendants of a starting element (currently only root) upto maxlevel */ diff --git a/test/t8_schemes/t8_gtest_boundary_extrude.cxx b/test/t8_schemes/t8_gtest_boundary_extrude.cxx index f4574f1601..eacd6bff2e 100644 --- a/test/t8_schemes/t8_gtest_boundary_extrude.cxx +++ b/test/t8_schemes/t8_gtest_boundary_extrude.cxx @@ -28,7 +28,9 @@ #include "t8_gtest_dfs_base.hxx" #include -class class_test_boundary_extrude: public TestDFS { +struct class_test_boundary_extrude: public TestDFS +{ + private: /* For elements that are on the face of the root element, check that creating the boundary element * and extruding it results in the original element */ diff --git a/test/t8_schemes/t8_gtest_child_parent_face.cxx b/test/t8_schemes/t8_gtest_child_parent_face.cxx index d0cc9415db..70a79d518b 100644 --- a/test/t8_schemes/t8_gtest_child_parent_face.cxx +++ b/test/t8_schemes/t8_gtest_child_parent_face.cxx @@ -27,7 +27,9 @@ #include "t8_gtest_dfs_base.hxx" #include -class class_child_parent_face: public TestDFS { +struct class_child_parent_face: public TestDFS +{ + private: void check_element () override { diff --git a/test/t8_schemes/t8_gtest_descendant.cxx b/test/t8_schemes/t8_gtest_descendant.cxx index 4d61553c78..8fb0604c77 100644 --- a/test/t8_schemes/t8_gtest_descendant.cxx +++ b/test/t8_schemes/t8_gtest_descendant.cxx @@ -28,7 +28,8 @@ /* This program tests the descendant function of an element. */ -class class_schemes_descendant: public testing::TestWithParam> { +struct class_schemes_descendant: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_schemes/t8_gtest_dfs_base.hxx b/test/t8_schemes/t8_gtest_dfs_base.hxx index 261a28d541..156ddfb358 100644 --- a/test/t8_schemes/t8_gtest_dfs_base.hxx +++ b/test/t8_schemes/t8_gtest_dfs_base.hxx @@ -27,7 +27,8 @@ #include #include -class TestDFS: public testing::TestWithParam> { +struct TestDFS: public testing::TestWithParam> +{ public: /** recursive tests check something for all descendants of a starting element (currently only root) upto maxlevel */ diff --git a/test/t8_schemes/t8_gtest_element_count_leaves.cxx b/test/t8_schemes/t8_gtest_element_count_leaves.cxx index 9e74fea9bb..20be78302d 100644 --- a/test/t8_schemes/t8_gtest_element_count_leaves.cxx +++ b/test/t8_schemes/t8_gtest_element_count_leaves.cxx @@ -34,7 +34,8 @@ /* Tests whether the leaf count for one additional level matches the number of children */ -class class_element_leaves: public testing::TestWithParam> { +struct class_element_leaves: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_schemes/t8_gtest_element_ref_coords.cxx b/test/t8_schemes/t8_gtest_element_ref_coords.cxx index 6fba562446..97ee0f62d4 100644 --- a/test/t8_schemes/t8_gtest_element_ref_coords.cxx +++ b/test/t8_schemes/t8_gtest_element_ref_coords.cxx @@ -223,7 +223,8 @@ t8_test_coords (const t8_forest_t forest, const t8_locidx_t ltree_id, const t8_e << t8_generate_additional_info_centroid (shape, centroid_by_vertices, centroid_by_element_ref_coords); } -class class_ref_coords: public testing::TestWithParam> { +struct class_ref_coords: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_schemes/t8_gtest_elements_are_family.cxx b/test/t8_schemes/t8_gtest_elements_are_family.cxx index 0e4fb3b83f..365b645edd 100644 --- a/test/t8_schemes/t8_gtest_elements_are_family.cxx +++ b/test/t8_schemes/t8_gtest_elements_are_family.cxx @@ -26,8 +26,9 @@ #include #include "t8_gtest_dfs_base.hxx" -class are_family: public TestDFS { - +struct are_family: public TestDFS +{ + private: void check_element () override { diff --git a/test/t8_schemes/t8_gtest_equal.cxx b/test/t8_schemes/t8_gtest_equal.cxx index a191b4528d..fdf98b39ba 100644 --- a/test/t8_schemes/t8_gtest_equal.cxx +++ b/test/t8_schemes/t8_gtest_equal.cxx @@ -27,7 +27,9 @@ #include #include "t8_gtest_dfs_base.hxx" -class class_test_equal: public TestDFS { +struct class_test_equal: public TestDFS +{ + private: void check_element () override { diff --git a/test/t8_schemes/t8_gtest_face_corner.cxx b/test/t8_schemes/t8_gtest_face_corner.cxx index 9696d94982..3d17d6ba01 100644 --- a/test/t8_schemes/t8_gtest_face_corner.cxx +++ b/test/t8_schemes/t8_gtest_face_corner.cxx @@ -31,8 +31,9 @@ * The first test gets the corners of all faces of an element and checks if the reverse function can get the correct face of the corner. * The second test does the same but in reverse order. */ -class class_face_corner_test: public TestDFS { - +struct class_face_corner_test: public TestDFS +{ + private: void check_element () override { diff --git a/test/t8_schemes/t8_gtest_face_descendant.cxx b/test/t8_schemes/t8_gtest_face_descendant.cxx index 1b294db06b..32c3db6f20 100644 --- a/test/t8_schemes/t8_gtest_face_descendant.cxx +++ b/test/t8_schemes/t8_gtest_face_descendant.cxx @@ -58,7 +58,9 @@ t8_test_manual_first_last_face_descendant (const t8_scheme *scheme, const t8_ele T8_TESTSUITE_FREE (child_indices); } -class class_descendant: public TestDFS { +struct class_descendant: public TestDFS +{ + private: void check_element () override { diff --git a/test/t8_schemes/t8_gtest_face_neigh.cxx b/test/t8_schemes/t8_gtest_face_neigh.cxx index 104ef2ac28..875b9e536a 100644 --- a/test/t8_schemes/t8_gtest_face_neigh.cxx +++ b/test/t8_schemes/t8_gtest_face_neigh.cxx @@ -28,7 +28,8 @@ #include -class face_neigh: public testing::TestWithParam> { +struct face_neigh: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_schemes/t8_gtest_find_parent.cxx b/test/t8_schemes/t8_gtest_find_parent.cxx index 6f78d35bce..6f58b165e5 100644 --- a/test/t8_schemes/t8_gtest_find_parent.cxx +++ b/test/t8_schemes/t8_gtest_find_parent.cxx @@ -27,7 +27,9 @@ #include "t8_gtest_dfs_base.hxx" #include -class class_find_parent: public TestDFS { +struct class_find_parent: public TestDFS +{ + private: void check_element () override { diff --git a/test/t8_schemes/t8_gtest_get_linear_id.cxx b/test/t8_schemes/t8_gtest_get_linear_id.cxx index 1ba46f6575..62764c3924 100644 --- a/test/t8_schemes/t8_gtest_get_linear_id.cxx +++ b/test/t8_schemes/t8_gtest_get_linear_id.cxx @@ -28,7 +28,8 @@ #include #include -class get_linear_id: public testing::TestWithParam> { +struct get_linear_id: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_schemes/t8_gtest_input_equal_output.cxx b/test/t8_schemes/t8_gtest_input_equal_output.cxx index e93d202500..7738482073 100644 --- a/test/t8_schemes/t8_gtest_input_equal_output.cxx +++ b/test/t8_schemes/t8_gtest_input_equal_output.cxx @@ -31,7 +31,9 @@ /* The mainly tested function in this test is element_get_children_at_face. The function allows that the input elem is equal children[0] (output). Therefore the test checks if this is true and the input is not overridden in the loop or is overridden in the last iteration. This is tested by comparing the output of two cases. In the first case the condition is not true in the second case elem is equal children[0]. */ -class class_test_equal: public TestDFS { +struct class_test_equal: public TestDFS +{ + private: void check_element () override { diff --git a/test/t8_schemes/t8_gtest_nca.cxx b/test/t8_schemes/t8_gtest_nca.cxx index 779f07b199..252911581a 100644 --- a/test/t8_schemes/t8_gtest_nca.cxx +++ b/test/t8_schemes/t8_gtest_nca.cxx @@ -31,7 +31,8 @@ #include #include -class nca: public testing::TestWithParam> { +struct nca: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_schemes/t8_gtest_pack_unpack.cxx b/test/t8_schemes/t8_gtest_pack_unpack.cxx index 51f71d07da..0a70d5b383 100644 --- a/test/t8_schemes/t8_gtest_pack_unpack.cxx +++ b/test/t8_schemes/t8_gtest_pack_unpack.cxx @@ -30,7 +30,9 @@ /** Use DFS to check for all elements, if packing them, sending them to ourself and unpacking them results in the same element * Here, each element is sent individually. */ -class class_test_pack: public TestDFS { +struct class_test_pack: public TestDFS +{ + private: /* pack the element and its children, send to ourself, unpack and check if it is the same element */ void check_element () override diff --git a/test/t8_schemes/t8_gtest_root.cxx b/test/t8_schemes/t8_gtest_root.cxx index 77cbb5a1e2..41711c24bd 100644 --- a/test/t8_schemes/t8_gtest_root.cxx +++ b/test/t8_schemes/t8_gtest_root.cxx @@ -31,7 +31,8 @@ #include #include -class root: public testing::TestWithParam> { +struct root: public testing::TestWithParam> +{ protected: void SetUp () override diff --git a/test/t8_schemes/t8_gtest_set_linear_id.cxx b/test/t8_schemes/t8_gtest_set_linear_id.cxx index f0322524f1..38fb7bd583 100644 --- a/test/t8_schemes/t8_gtest_set_linear_id.cxx +++ b/test/t8_schemes/t8_gtest_set_linear_id.cxx @@ -32,7 +32,9 @@ * The id_counter is then increased to match the id of the next leaf. After we have reached the last element on a level, * we increase the level and reset the id_counter to 0. */ -class class_test_set_linear_id: public TestBFS { +struct class_test_set_linear_id: public TestBFS +{ + private: void check_element () override { diff --git a/test/t8_schemes/t8_gtest_successor.cxx b/test/t8_schemes/t8_gtest_successor.cxx index ae49d5ecb1..34a6e7eb33 100644 --- a/test/t8_schemes/t8_gtest_successor.cxx +++ b/test/t8_schemes/t8_gtest_successor.cxx @@ -26,7 +26,8 @@ #include #include -class class_successor: public testing::TestWithParam> { +struct class_successor: public testing::TestWithParam> +{ protected: void SetUp () override