tidy 05

Author: francoisbonneau

include/geode/stochastic/models/energy_terms/energy_term_builder.hpp

@@ -108,9 +108,9 @@ namespace geode
 
     template < typename ObjectType, typename NewEnergyTypeConfig >
     std::unique_ptr< EnergyTerm< ObjectType > > build_energy_term_impl(
-        const NewEnergyTypeConfig&,
-        const ObjectSets< ObjectType >&,
-        const SpatialDomain< ObjectType::dim >& )
+        const NewEnergyTypeConfig& /*unused*/,
+        const ObjectSets< ObjectType >& /*unused*/,
+        const SpatialDomain< ObjectType::dim >& /*unused*/ )
     {
         static_assert( sizeof( NewEnergyTypeConfig ) == 0,
             "Unsupported EnergyTermConfig type" );
@@ -119,9 +119,9 @@ namespace geode
 
     template < typename ObjectType >
     std::unique_ptr< EnergyTerm< ObjectType > > build_energy_term_impl(
-        const std::monostate&,
-        const ObjectSets< ObjectType >&,
-        const SpatialDomain< ObjectType::dim >& )
+        const std::monostate& /*unused*/,
+        const ObjectSets< ObjectType >& /*unused*/,
+        const SpatialDomain< ObjectType::dim >& /*unused*/ )
     {
         throw OpenGeodeStochasticStochasticException{ nullptr,
             OpenGeodeException::TYPE::data,

include/geode/stochastic/models/energy_terms/pairwise_term.hpp

@@ -106,7 +106,7 @@ namespace geode
         {
             double sum = 0.0;
             this->for_each_object_in_sets( state, this->impacted_set_ids(),
-                [&]( const ObjectId& cur_obj_id ) {
+                [&state]( const ObjectId& cur_obj_id ) {
                     sum += accumulate_interactions_with_neighbors(
                         cur_obj_id, state );
                 } );

include/geode/stochastic/spatial/pairwise_interactions/pairwise_interactions_builder.hpp

@@ -21,6 +21,7 @@
  *
  */
 
+#include <geode/basic/assert.hpp>
 #include <geode/stochastic/spatial/pairwise_interactions/pairwise_interactions_config.hpp>
 #include <variant>
 
@@ -47,7 +48,8 @@ namespace geode
 
     template < typename ObjectType, typename NewInteractionConfig >
     std::unique_ptr< PairwiseInteraction< ObjectType > >
-        build_pairwise_interaction_impl( const NewInteractionConfig& )
+        build_pairwise_interaction_impl(
+            const NewInteractionConfig& /*unused*/ )
     {
         static_assert( sizeof( NewInteractionConfig ) == 0,
             "Unsupported PairwiseInteractionConfig type" );
@@ -56,7 +58,7 @@ namespace geode
 
     template < typename ObjectType >
     std::unique_ptr< PairwiseInteraction< ObjectType > >
-        build_pairwise_interaction_impl( const std::monostate& )
+        build_pairwise_interaction_impl( const std::monostate& /*unused*/ )
     {
         throw OpenGeodeStochasticStochasticException{ nullptr,
             OpenGeodeException::TYPE::data,

include/geode/stochastic/spatial/single_object_features/single_object_feature_builder.hpp

@@ -24,6 +24,8 @@
 
 #include <variant>
 
+#include <geode/basic/assert.hpp>
+
 #include <geode/stochastic/spatial/single_object_features/segment_length_feature.hpp>
 #include <geode/stochastic/spatial/single_object_features/single_object_feature_config.hpp>
 
@@ -52,22 +54,25 @@ namespace geode
         {
             throw OpenGeodeStochasticStochasticException{ nullptr,
                 OpenGeodeException::TYPE::data,
-                "SegmentLengthInsideBoxFeature not valid for this ObjectType" };
+                "[SingleObjectFeatureBuilder] SegmentLengthInsideBoxFeature "
+                "not valid for this ObjectType" };
         }
     }
 
     template < typename ObjectType, typename NewObjectFeatureConfig >
     std::unique_ptr< SingleObjectFeature< ObjectType > >
-        build_single_object_feature_impl( const NewObjectFeatureConfig& )
+        build_single_object_feature_impl(
+            const NewObjectFeatureConfig& /*unused*/ )
     {
         static_assert( sizeof( NewObjectFeatureConfig ) == 0,
-            "Unsupported SingleObjectFeatureConfig type" );
+            "[SingleObjectFeatureBuilder] Unsupported "
+            "SingleObjectFeatureConfig type" );
         return nullptr;
     }
 
     template < typename ObjectType >
     std::unique_ptr< SingleObjectFeature< ObjectType > >
-        build_single_object_feature_impl( const std::monostate& )
+        build_single_object_feature_impl( const std::monostate& /*unused*/ )
     {
         throw OpenGeodeStochasticStochasticException{ nullptr,
             OpenGeodeException::TYPE::data,