Remove unncessary <iostream> includes

jwnimmer-tri · jwnimmer-tri · commit df623ee370ea · 2024-12-31T08:17:44.000-08:00
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.

Signed-off-by: Jeremy Nimmer &lt;jeremy.nimmer@tri.global&gt;
diff --git a/include/sdf/Error.hh b/include/sdf/Error.hh
@@ -17,9 +17,9 @@
 #ifndef SDF_ERROR_HH_
 #define SDF_ERROR_HH_
 
-#include <iostream>
-#include <string>
 #include <optional>
+#include <ostream>
+#include <string>
 #include <gz/utils/ImplPtr.hh>
 #include <sdf/sdf_config.h>
 #include "sdf/Console.hh"
diff --git a/include/sdf/Exception.hh b/include/sdf/Exception.hh
@@ -19,8 +19,8 @@
 #define SDF_EXCEPTION_HH_
 
 #include <cstdint>
-#include <iostream>
 #include <memory>
+#include <ostream>
 #include <sstream>
 #include <string>
 
diff --git a/src/Console.cc b/src/Console.cc
@@ -16,6 +16,7 @@
  */
 
 #include <cstdlib>
+#include <iostream>
 #include <memory>
 #include <mutex>
 #include <sstream>
diff --git a/src/Element.cc b/src/Element.cc
@@ -16,6 +16,7 @@
  */
 
 #include <algorithm>
+#include <iostream>
 #include <sstream>
 #include <string>
 
diff --git a/src/FrameSemantics.cc b/src/FrameSemantics.cc
@@ -15,6 +15,7 @@
  *
 */
 #include <algorithm>
+#include <iostream>
 #include <string>
 #include <set>
 #include <utility>
diff --git a/src/SDF.cc b/src/SDF.cc
@@ -17,10 +17,11 @@
 
 #include <cstdlib>
 #include <fstream>
-#include <sstream>
 #include <functional>
+#include <iostream>
 #include <list>
 #include <map>
+#include <sstream>
 #include <string>
 #include <vector>
 

Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,7 @@`
`15`	`15`	`*`
`16`	`16`	`*/`
`17`	`17`	`#include <algorithm>`
	`18`	`+#include <iostream>`
`18`	`19`	`#include <string>`
`19`	`20`	`#include <set>`
`20`	`21`	`#include <utility>`