From 4d1f51146cd04b6342bb6eb87033a05f0d6c089d Mon Sep 17 00:00:00 2001
From: Timo Heister <timo.heister@gmail.com>
Date: Tue, 13 Aug 2024 15:45:17 -0400
Subject: [PATCH] cleanup maybe-unused in fe.cc

followup to #17508
---
 source/fe/fe.cc | 30 ++++++++++--------------------
 1 file changed, 10 insertions(+), 20 deletions(-)

diff --git a/source/fe/fe.cc b/source/fe/fe.cc
index cabb98d0ca..f9b3978cc7 100644
--- a/source/fe/fe.cc
+++ b/source/fe/fe.cc
@@ -851,7 +851,6 @@ FiniteElement<dim, spacedim>::constraints(
   // same number of dofs
   AssertDimension(this->n_unique_faces(), 1);
   [[maybe_unused]] const unsigned int face_no = 0;
-  (void)subface_case;
 
   Assert(subface_case == internal::SubfaceCase<dim>::case_isotropic,
          ExcMessage("Constraints for this element are only implemented "
@@ -1178,8 +1177,6 @@ FiniteElement<dim, spacedim>::get_sub_fe(
   const unsigned int first_component,
   const unsigned int n_selected_components) const
 {
-  (void)first_component;
-  (void)n_selected_components;
   // No complicated logic is needed here, because it is overridden in
   // FESystem<dim,spacedim>. Just make sure that what the user chose is valid:
   Assert(first_component == 0 && n_selected_components == this->n_components(),
@@ -1328,29 +1325,23 @@ FiniteElement<dim, spacedim>::fill_fe_face_values(
 template <int dim, int spacedim>
 inline void
 FiniteElement<dim, spacedim>::fill_fe_face_values(
-  const typename Triangulation<dim, spacedim>::cell_iterator &cell,
-  const unsigned int                                          face_no,
-  const Quadrature<dim - 1>                                  &quadrature,
-  const Mapping<dim, spacedim>                               &mapping,
-  const typename Mapping<dim, spacedim>::InternalDataBase    &mapping_internal,
+  const typename Triangulation<dim, spacedim>::cell_iterator & /* cell */,
+  const unsigned int /* face_no */,
+  const Quadrature<dim - 1> & /* quadrature */,
+  const Mapping<dim, spacedim> & /* mapping */,
+  const typename Mapping<dim, spacedim>::InternalDataBase
+    & /* mapping_internal */,
   const internal::FEValuesImplementation::MappingRelatedData<dim, spacedim>
-                                                                &mapping_data,
-  const typename FiniteElement<dim, spacedim>::InternalDataBase &fe_internal,
+    & /* mapping_data */,
+  const typename FiniteElement<dim, spacedim>::InternalDataBase
+    & /* fe_internal */,
   dealii::internal::FEValuesImplementation::FiniteElementRelatedData<dim,
                                                                      spacedim>
-    &output_data) const
+    & /* output_data */) const
 {
   Assert(false,
          ExcMessage("Use of a deprecated interface, please implement "
                     "fill_fe_face_values taking a hp::QCollection argument"));
-  (void)face_no;
-  (void)cell;
-  (void)quadrature;
-  (void)mapping;
-  (void)mapping_internal;
-  (void)mapping_data;
-  (void)fe_internal;
-  (void)output_data;
 }
 #  endif
 
@@ -1379,7 +1370,6 @@ template <int dim, int spacedim>
 const FiniteElement<dim, spacedim> &
 FiniteElement<dim, spacedim>::base_element(const unsigned int index) const
 {
-  (void)index;
   AssertIndexRange(index, 1);
   // This function should not be
   // called for a system element
-- 
2.39.5