Merge pull request mlpack#413 from coatless-mlpack/bugfix/unused-vari…

…ables Bugfix/unused variables
rcurtin · Nov 30, 2024 · 92468ea · 92468ea
2 parents 9406243 + c456e8e
commit 92468ea
Show file tree

Hide file tree

Showing 9 changed files with 1 addition and 14 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -42,7 +42,7 @@ target_include_directories(ensmallen INTERFACE
 if(MSVC)
   target_compile_options(ensmallen INTERFACE $<BUILD_INTERFACE:/Wall>)
 else()
-  target_compile_options(ensmallen INTERFACE $<BUILD_INTERFACE:-Wall -Wpedantic -Wunused-parameter>)
+  target_compile_options(ensmallen INTERFACE $<BUILD_INTERFACE:-Wall -Wpedantic -Wunused-parameter -Wunused-variable -Wunused-private-field>)
 endif()
 
 # Find OpenMP and link it.

diff --git a/include/ensmallen_bits/agemoea/agemoea.hpp b/include/ensmallen_bits/agemoea/agemoea.hpp
@@ -452,12 +452,6 @@ class AGEMOEA
   //! Probability that crossover will occur.
   double crossoverProb;
 
-  //! Probability that mutation will occur.
-  double mutationProb;
-
-  //! Strength of the mutation.
-  double mutationStrength;
-
   //! The crowding degree of the mutation. Higher value produces a mutant
   //! resembling its parent.
   double distributionIndex;

diff --git a/include/ensmallen_bits/problems/dtlz/dtlz2_function.hpp b/include/ensmallen_bits/problems/dtlz/dtlz2_function.hpp
@@ -110,7 +110,6 @@ namespace test {
        */
       arma::Row<typename MatType::elem_type> g(const MatType& coords)
       {
-        size_t k = numVariables - numObjectives + 1;
 
         // Convenience typedef.
         typedef typename MatType::elem_type ElemType;

diff --git a/include/ensmallen_bits/problems/dtlz/dtlz4_function.hpp b/include/ensmallen_bits/problems/dtlz/dtlz4_function.hpp
@@ -113,7 +113,6 @@ namespace test {
        */
       arma::Row<typename MatType::elem_type> g(const MatType& coords)
       {
-        size_t k = numVariables - numObjectives + 1;
 
         // Convenience typedef.
         typedef typename MatType::elem_type ElemType;

diff --git a/include/ensmallen_bits/problems/dtlz/dtlz5_function.hpp b/include/ensmallen_bits/problems/dtlz/dtlz5_function.hpp
@@ -110,7 +110,6 @@ namespace test {
        */
       arma::Row<typename MatType::elem_type> g(const MatType& coords)
       {
-        size_t k = numVariables - numObjectives + 1;
 
         // Convenience typedef.
         typedef typename MatType::elem_type ElemType;

diff --git a/include/ensmallen_bits/problems/dtlz/dtlz6_function.hpp b/include/ensmallen_bits/problems/dtlz/dtlz6_function.hpp
@@ -110,7 +110,6 @@ namespace test {
        */
       arma::Row<typename MatType::elem_type> g(const MatType& coords)
       {
-        size_t k = numVariables - numObjectives + 1;
 
         // Convenience typedef.
         typedef typename MatType::elem_type ElemType;

diff --git a/include/ensmallen_bits/problems/dtlz/dtlz7_function.hpp b/include/ensmallen_bits/problems/dtlz/dtlz7_function.hpp
@@ -131,7 +131,6 @@ namespace test {
       arma::Row<typename MatType::elem_type> h(
           const MatType& coords, const arma::Row<typename MatType::elem_type>& G)
       {
-        size_t k = numVariables - numObjectives + 1;
 
         // Convenience typedef.
         typedef typename MatType::elem_type ElemType;

diff --git a/include/ensmallen_bits/problems/maf/maf5_function.hpp b/include/ensmallen_bits/problems/maf/maf5_function.hpp
@@ -131,7 +131,6 @@ namespace test {
        */
       arma::Row<typename MatType::elem_type> g(const MatType& coords)
       {
-        size_t k = numVariables - numObjectives + 1;
 
         // Convenience typedef.
         typedef typename MatType::elem_type ElemType;

diff --git a/include/ensmallen_bits/problems/maf/maf6_function.hpp b/include/ensmallen_bits/problems/maf/maf6_function.hpp
@@ -116,7 +116,6 @@ namespace test {
        */
       arma::Row<typename MatType::elem_type> g(const MatType& coords)
       {
-        size_t k = numVariables - numObjectives + 1;
 
         // Convenience typedef.
         typedef typename MatType::elem_type ElemType;