Use C++ streams for error messages

coin-or · Jul 29, 2024 · 345a254 · 345a254
1 parent 6fa5d84
commit 345a254
Show file tree

Hide file tree

Showing 14 changed files with 63 additions and 66 deletions.
diff --git a/src/CoinAdjacencyVector.cpp b/src/CoinAdjacencyVector.cpp
@@ -22,6 +22,7 @@
 #include <vector>
 #include <cassert>
 #include <algorithm>
+#include <iostream>
 #include <limits>
 #include "CoinAdjacencyVector.hpp"
 
@@ -116,7 +117,7 @@ static void *xmalloc( const size_t size )
    void *result = malloc( size );
    if (!result)
    {
-      fprintf(stderr, "No more memory available. Trying to allocate %zu bytes.", size);
+      std::cerr << "No more memory available. Trying to allocate " << size << " bytes.";
       abort();
    }
 
@@ -128,7 +129,7 @@ static void *xrealloc( void *ptr, const size_t size )
     void * res = realloc( ptr, size );
     if (!res)
     {
-       fprintf(stderr, "No more memory available. Trying to allocate %zu bytes.", size);
+       std::cerr << "No more memory available. Trying to allocate " << size << " bytes.";
        abort();
     }
 

diff --git a/src/CoinBronKerbosch.cpp b/src/CoinBronKerbosch.cpp
@@ -24,6 +24,7 @@
 #include <cstdio>
 #include <cstdlib>
 #include <ctime>
+#include <iostream>
 #include "CoinBronKerbosch.hpp"
 #include "CoinConflictGraph.hpp"
 #include "CoinCliqueList.hpp"
@@ -417,15 +418,15 @@ void CoinBronKerbosch::computeFitness(const double *weights) {
             break;
         }
         default:
-            fprintf(stderr, "Invalid option %lu for pivoting strategy!\n", pivotingStrategy_);
+            std::cerr << "Invalid option " << pivotingStrategy_ << " for pivoting strategy!" << std::endl;
             abort();
     }
 }
 
 static void *xmalloc( const size_t size ) {
     void *result = malloc( size );
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to allocate %zu bytes.", size);
+        std::cerr << "No more memory available. Trying to allocate " << size << "bytes.";
         abort();
     }
 
@@ -435,7 +436,7 @@ static void *xmalloc( const size_t size ) {
 static void *xcalloc( const size_t elements, const size_t size ) {
     void *result = calloc( elements, size );
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to callocate %zu bytes.", size * elements);
+        std::cerr << "No more memory available. Trying to callocate " << size * elements << " bytes.";
         abort();
     }
 
@@ -445,7 +446,7 @@ static void *xcalloc( const size_t elements, const size_t size ) {
 static void *xrealloc( void *ptr, const size_t size ) {
     void * res = realloc( ptr, size );
     if (!res) {
-        fprintf(stderr, "No more memory available. Trying to allocate %zu bytes in CoinCliqueList", size);
+        std::cerr << "No more memory available. Trying to allocate " << size << " bytes in CoinCliqueList";
         abort();
     }
 

diff --git a/src/CoinCliqueExtender.cpp b/src/CoinCliqueExtender.cpp
@@ -20,6 +20,7 @@
 #include <ctime>
 #include <cassert>
 #include <cmath>
+#include <iostream>
 #include <limits>
 #include <algorithm>
 #include "CoinCliqueExtender.hpp"
@@ -66,7 +67,7 @@ CoinCliqueExtender::CoinCliqueExtender(const CoinConflictGraph *cgraph, size_t e
     maxRC_ = maxRC;
 
     if ((extMethod_ == 4 || extMethod_ == 5) && !rc_) {
-        fprintf(stderr, "Warning: using random selection for extension since no costs were informed.\n");
+        std::cerr << "Warning: using random selection for extension since no costs were informed." << std::endl;
         extMethod_ = 1;
     }
 
@@ -133,7 +134,7 @@ CoinCliqueExtender::CoinCliqueExtender(const CoinConflictGraph *cgraph, size_t e
             break;
         }
         default:
-            fprintf(stderr, "Invalid option %lu\n", extMethod_);
+            std::cerr << "Invalid option " << extMethod_ << std::endl;
             abort();
     }
 }
@@ -273,7 +274,7 @@ bool CoinCliqueExtender::extendClique(const size_t *clqIdxs, const size_t clqSiz
             result = greedySelection(clqIdxs, clqSize, costs_);
             break;
         default:
-            fprintf(stderr, "Invalid option %lu\n", extMethod_);
+            std::cerr << "Invalid option " << extMethod_ << std::endl;
             abort();
     }
 
@@ -363,7 +364,7 @@ size_t CoinCliqueExtender::getCliqueSize(const size_t i) const {
 static void *xmalloc( const size_t size ) {
     void *result = malloc( size );
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to allocate %zu bytes.", size);
+        std::cerr << "No more memory available. Trying to allocate " << size << " bytes.";
         abort();
     }
 
@@ -373,7 +374,7 @@ static void *xmalloc( const size_t size ) {
 static void *xcalloc( const size_t elements, const size_t size ) {
     void *result = calloc( elements, size );
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to callocate %zu bytes.", size * elements);
+        std::cerr << "No more memory available. Trying to callocate " << size * elements << " bytes.";
         abort();
     }
 

diff --git a/src/CoinCliqueList.cpp b/src/CoinCliqueList.cpp
@@ -21,6 +21,7 @@
 #include <cstdio>
 #include <cstring>
 #include <cassert>
+#include <iostream>
 #include "CoinCliqueList.hpp"
 
 #ifdef DEBUGCG
@@ -91,7 +92,7 @@ static void *xmalloc( const size_t size )
   void *result = malloc( size );
   if (!result)
   {
-    fprintf(stderr, "No more memory available. Trying to allocate %zu bytes in CoinCliqueList.\n", size);
+    std::cerr << "No more memory available. Trying to allocate " << size << " bytes in CoinCliqueList." << std::endl;
     exit(1);
   }
 
@@ -113,7 +114,7 @@ void CoinCliqueList::computeNodeOccurrences(size_t nNodes)
   // couting number of occurrences for each node
   size_t *noc = (size_t *)calloc( nNodes, sizeof(size_t) );
   if (!noc) {
-    fprintf( stderr, "No more memory available.\n" );
+    std::cerr << "No more memory available." << std::endl;
     abort();
   }
 
@@ -160,7 +161,7 @@ static void *xrealloc( void *ptr, const size_t size )
   void * res = realloc( ptr, size );
   if (!res)
   {
-    fprintf(stderr, "No more memory available. Trying to allocate %zu bytes in CoinCliqueList", size);
+    std::cerr << "No more memory available. Trying to allocate " << size << " bytes in CoinCliqueList";
     abort();
   }
 
@@ -194,7 +195,7 @@ size_t CoinCliqueList::nNodeOccurrences(size_t idxNode) const
 #ifdef DEBUGCG
 void CoinCliqueList::validateClique(const CoinConflictGraph *cgraph, const size_t *idxs, const size_t size) {
     if (size == 0) {
-        fprintf(stderr, "Empty clique!\n");
+        std::cerr << "Empty clique!" << std::endl;
         abort();
     }
 
@@ -206,10 +207,10 @@ void CoinCliqueList::validateClique(const CoinConflictGraph *cgraph, const size_
     for (size_t i = 0; i < size - 1; i++) {
         for (size_t j = i + 1; j < size; j++) {
             if ((!cgraph->conflicting(idxs[i], idxs[j])) || (idxs[i] == idxs[j])) {
-                fprintf(stderr, "ERROR: Nodes %ld and %ld are not in conflict.\n", idxs[i], idxs[j]);
+                std::cerr << "ERROR: Nodes " << idxs[i] << " and " << idxs[j] << " are not in conflict." << std::endl;
                 abort();
             }
         }
     }
 }
-#endif
+#endif
diff --git a/src/CoinCliqueSet.cpp b/src/CoinCliqueSet.cpp
@@ -22,6 +22,7 @@
 #include <cstdio>
 #include <cstdlib>
 #include <cstring>
+#include <iostream>
 
 const size_t CoinCliqueSet::sequence_[] = {37, 31, 29, 17, 13, 11, 7, 1};
 const size_t CoinCliqueSet::nSequence_ = 8;
@@ -184,7 +185,7 @@ bool CoinCliqueSet::alreadyInserted(size_t size, const size_t els[], size_t hash
 static void *xmalloc( const size_t size ) {
     void *result = malloc( size );
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to allocate %zu bytes.", size);
+        std::cerr << "No more memory available. Trying to allocate " << size << " bytes.";
         abort();
     }
 
@@ -194,9 +195,9 @@ static void *xmalloc( const size_t size ) {
 static void *xrealloc( void *ptr, const size_t size ) {
     void * result = realloc( ptr, size );
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to allocate %zu bytes in CoinCliqueList", size);
+        std::cerr << "No more memory available. Trying to allocate " << size << " bytes in CoinCliqueList";
         abort();
     }
 
     return result;
-}
+}
diff --git a/src/CoinConflictGraph.cpp b/src/CoinConflictGraph.cpp
@@ -93,7 +93,7 @@ bool CoinConflictGraph::conflicting(size_t n1, size_t n2) const {
 static void *xmalloc(const size_t size) {
     void *result = malloc(size);
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to allocate %zu bytes.", size);
+        std::cerr << "No more memory available. Trying to allocate " << size << " bytes.";
         abort();
     }
 
@@ -103,7 +103,7 @@ static void *xmalloc(const size_t size) {
 static void *xcalloc(const size_t elements, const size_t size) {
     void *result = calloc(elements, size);
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to callocate %zu bytes.", size * elements);
+        std::cerr << "No more memory available. Trying to callocate " << size * elements << " bytes.";
         abort();
     }
 

diff --git a/src/CoinCutPool.cpp b/src/CoinCutPool.cpp
@@ -31,6 +31,7 @@
 #include <cmath>
 #include <cassert>
 #include <algorithm>
+#include <iostream>
 
 #define CUTPOOL_EPS 1e-8
 
@@ -408,7 +409,7 @@ void CoinCutPool::removeNullCuts() {
 static void *xmalloc( const size_t size ) {
     void *result = malloc( size );
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to allocate %zu bytes.", size);
+        std::cerr << "No more memory available. Trying to allocate " << size << " bytes.";
         abort();
     }
 
@@ -418,7 +419,7 @@ static void *xmalloc( const size_t size ) {
 static void *xrealloc( void *ptr, const size_t size ) {
     void * res = realloc( ptr, size );
     if (!res) {
-        fprintf(stderr, "No more memory available. Trying to allocate %zu bytes in CoinCliqueList", size);
+        std::cerr << "No more memory available. Trying to allocate " << size << " bytes in CoinCliqueList";
         abort();
     }
 
@@ -428,9 +429,9 @@ static void *xrealloc( void *ptr, const size_t size ) {
 static void *xcalloc( const size_t elements, const size_t size ) {
     void *result = calloc( elements, size );
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to callocate %zu bytes.", size * elements);
+        std::cerr << "No more memory available. Trying to callocate " << size * elements << " bytes.";
         abort();
     }
 
     return result;
-}
+}
diff --git a/src/CoinDynamicConflictGraph.cpp b/src/CoinDynamicConflictGraph.cpp
@@ -306,7 +306,7 @@ CoinDynamicConflictGraph::CoinDynamicConflictGraph (
 
     char *iv = (char *) calloc( size_, sizeof(char) );
     if (!iv) {
-      fprintf( stderr, "out of memory.\n" );
+      std::cerr << "out of memory." << std::endl;
       abort();
     }
 
@@ -546,7 +546,7 @@ static void *xrealloc( void *ptr, const size_t size )
     void * res = realloc( ptr, size );
     if (!res)
     {
-       fprintf(stderr, "No more memory available. Trying to allocate %zu bytes.", size);
+       std::cerr << "No more memory available. Trying to allocate " << size << " bytes.";
        abort();
     }
 
@@ -558,7 +558,7 @@ static void *xmalloc( const size_t size )
    void *result = malloc( size );
    if (!result)
    {
-      fprintf(stderr, "No more memory available. Trying to allocate %zu bytes.", size);
+      std::cerr << "No more memory available. Trying to allocate " << size << " bytes.";
       abort();
    }
 
@@ -568,7 +568,7 @@ static void *xmalloc( const size_t size )
 static void *xcalloc( const size_t elements, const size_t size ) {
     void *result = calloc( elements, size );
     if (!result) {
-        fprintf(stderr, "No more memory available. Trying to callocate %zu bytes.", size * elements);
+        std::cerr << "No more memory available. Trying to callocate " << size * elements << " bytes.";
         abort();
     }
 

diff --git a/src/CoinFactorization.hpp b/src/CoinFactorization.hpp
@@ -1004,7 +1004,7 @@ class COINUTILSLIB_EXPORT CoinFactorization {
     if (l + numberInPivotColumn > lengthAreaL_) {
       //need more memory
       if ((messageLevel_ & 4) != 0)
-        printf("more memory needed in middle of invert\n");
+        std::cout <<"more memory needed in middle of invert\n";
       return false;
     }
     //l+=currentAreaL_->elementByColumn-elementL;