Remove unncessary <iostream> includes (#582)

Including iostream means introducing the static (global) constructors and destructors for std::cin, std::cerr, and std::cout. That extra init and fini code is undesirable when those streams are not actually used.
stevengj · Dec 31, 2024 · 9128062 · 9128062
1 parent e414622
commit 9128062
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 5 deletions.
diff --git a/src/algs/ags/solver.cc b/src/algs/ags/solver.cc
@@ -9,7 +9,6 @@ Copyright (C) 2018 Sovrasov V. - All Rights Reserved
 
 #include <algorithm>
 #include <cmath>
-#include <iostream>
 
 using namespace ags;
 
@@ -300,7 +299,6 @@ void NLPSolver::CalculateNextPoints()
 
     if (mNextPoints[i].x >= mNextIntervals[i]->pr.x || mNextPoints[i].x <= mNextIntervals[i]->pl.x)
       mNeedStop = true;
-      //std::cout << "Warning: resolution of evolvent is not enough to continue the search";
 
     mEvolvent.GetImage(mNextPoints[i].x, mNextPoints[i].y);
   }

diff --git a/src/algs/stogo/linalg.cc b/src/algs/stogo/linalg.cc
@@ -3,7 +3,7 @@
    No attempt is made to check if the function arguments are valid
 */
 
-#include <iostream>
+#include <ostream>
 #include <cmath>         // for sqrt()
 
 #include "linalg.h"

diff --git a/src/algs/stogo/linalg.h b/src/algs/stogo/linalg.h
@@ -6,7 +6,7 @@
 #ifndef LINALG_H
 #define LINALG_H
 
-#include <iostream>
+#include <ostream>
 using namespace std;
 #include <cmath>         // for sqrt()
 #include <cfloat>

diff --git a/src/algs/stogo/tools.h b/src/algs/stogo/tools.h
@@ -5,7 +5,7 @@
 #define TOOLS_H
 
 #include <cfloat>
-#include <iostream>
+#include <ostream>
 
 #include <algorithm>
 #include <iterator>