use #if 0 instead of && 0 to disable some if's

- avoids -Wconstant-logical-operand warning
coin-or · Mar 17, 2023 · f85d6cf · f85d6cf
1 parent 19526dc
commit f85d6cf
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/Clp/src/ClpCholeskyBase.cpp b/Clp/src/ClpCholeskyBase.cpp
@@ -2957,13 +2957,15 @@ int ClpCholeskyBase::factorize(const CoinWorkDouble *diagonal, int *rowsDropped)
         }
       }
       numberRowsDropped_ += newDropped;
-      if (numberRowsDropped_ && 0) {
+#if 0
+      if (numberRowsDropped_) {
         std::cout << "Rank " << numberRows_ - numberRowsDropped_ << " ( " << numberRowsDropped_ << " dropped)";
         if (newDropped) {
           std::cout << " ( " << newDropped << " dropped this time)";
         }
         std::cout << std::endl;
       }
+#endif
     }
   } else {
     //KKT

diff --git a/Clp/src/ClpCholeskyMumps.cpp b/Clp/src/ClpCholeskyMumps.cpp
@@ -437,13 +437,15 @@ int ClpCholeskyMumps::factorize(const double *diagonal, int *rowsDropped)
       }
     }
     numberRowsDropped_ += newDropped;
-    if (numberRowsDropped_ && 0) {
+#if 0
+    if (numberRowsDropped_) {
       std::cout << "Rank " << numberRows_ - numberRowsDropped_ << " ( " << numberRowsDropped_ << " dropped)";
       if (newDropped) {
         std::cout << " ( " << newDropped << " dropped this time)";
       }
       std::cout << std::endl;
     }
+#endif
   }
   status_ = 0;
   return newDropped;