core: replace iostream by runtimeErrorMsg()

espressomd · May 1, 2020 · 09294e7 · 09294e7
1 parent d82b49d
commit 09294e7
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 25 deletions.
diff --git a/src/core/actor/DipolarBarnesHut.hpp b/src/core/actor/DipolarBarnesHut.hpp
@@ -30,9 +30,6 @@
 #include "errorhandling.hpp"
 #include "grid.hpp"
 
-#include <cstdio>
-#include <iostream>
-
 #ifndef ACTOR_DIPOLARBARNESHUT_HPP
 #define ACTOR_DIPOLARBARNESHUT_HPP
 
@@ -47,16 +44,13 @@ class DipolarBarnesHut : public Actor {
     m_itolsq = itolsq;
     setBHPrecision(&m_epssq, &m_itolsq);
     if (!s.requestFGpu())
-      std::cerr << "DipolarBarnesHut needs access to forces on GPU!"
-                << std::endl;
+      runtimeErrorMsg() << "DipolarBarnesHut needs access to forces on GPU!";
 
     if (!s.requestRGpu())
-      std::cerr << "DipolarBarnesHut needs access to positions on GPU!"
-                << std::endl;
+      runtimeErrorMsg() << "DipolarBarnesHut needs access to positions on GPU!";
 
     if (!s.requestDipGpu())
-      std::cerr << "DipolarBarnesHut needs access to dipoles on GPU!"
-                << std::endl;
+      runtimeErrorMsg() << "DipolarBarnesHut needs access to dipoles on GPU!";
   };
 
   void computeForces(SystemInterface &s) override {
@@ -69,10 +63,7 @@ class DipolarBarnesHut : public Actor {
     summarizeBH(m_bh_data.blocks);
     sortBH(m_bh_data.blocks);
     if (forceBH(&m_bh_data, k, s.fGpuBegin(), s.torqueGpuBegin())) {
-      fprintf(
-          stderr,
-          "forceBH: some of kernels encounter the algorithm functional error");
-      errexit();
+      runtimeErrorMsg() << "kernels encountered a functional error";
     }
   };
   void computeEnergy(SystemInterface &s) override {
@@ -85,10 +76,7 @@ class DipolarBarnesHut : public Actor {
     summarizeBH(m_bh_data.blocks);
     sortBH(m_bh_data.blocks);
     if (energyBH(&m_bh_data, k, (&(((CUDA_energy *)s.eGpu())->dipolar)))) {
-      fprintf(
-          stderr,
-          "energyBH: some of kernels encounter the algorithm functional error");
-      errexit();
+      runtimeErrorMsg() << "kernels encountered a functional error";
     }
   };
 

diff --git a/src/core/actor/DipolarDirectSum.hpp b/src/core/actor/DipolarDirectSum.hpp
@@ -25,9 +25,9 @@
 #include "SystemInterface.hpp"
 #include "cuda_interface.hpp"
 #include "electrostatics_magnetostatics/dipole.hpp"
+#include "errorhandling.hpp"
 #include "grid.hpp"
 
-#include <iostream>
 #include <memory>
 
 #ifndef ACTOR_DIPOLARDIRECTSUM_HPP
@@ -50,16 +50,13 @@ class DipolarDirectSum : public Actor {
     k = static_cast<float>(dipole.prefactor);
 
     if (!s.requestFGpu())
-      std::cerr << "DipolarDirectSum needs access to forces on GPU!"
-                << std::endl;
+      runtimeErrorMsg() << "DipolarDirectSum needs access to forces on GPU!";
 
     if (!s.requestRGpu())
-      std::cerr << "DipolarDirectSum needs access to positions on GPU!"
-                << std::endl;
+      runtimeErrorMsg() << "DipolarDirectSum needs access to positions on GPU!";
 
     if (!s.requestDipGpu())
-      std::cerr << "DipolarDirectSum needs access to dipoles on GPU!"
-                << std::endl;
+      runtimeErrorMsg() << "DipolarDirectSum needs access to dipoles on GPU!";
   };
   void computeForces(SystemInterface &s) override {
     dds_float box[3];

diff --git a/src/core/grid_based_algorithms/electrokinetics_cuda.cu b/src/core/grid_based_algorithms/electrokinetics_cuda.cu
@@ -2475,7 +2475,7 @@ int ek_init() {
     try {
       electrostatics = new FdElectrostatics(es_parameters, stream[0]);
     } catch (std::string e) {
-      std::cout << "Error in initialization of electrokinetics electrostatics "
+      std::cerr << "Error in initialization of electrokinetics electrostatics "
                    "solver: "
                 << e << std::endl;
       return 1;