cleaning up includes and a couple comments

bench/bench_window.hpp

@@ -1,8 +1,10 @@
 #ifndef LIB94_BENCH_BENCH_WINDOW_HPP
 #define LIB94_BENCH_BENCH_WINDOW_HPP
 
+#include <lib94/lib94.hpp>
 #include <gtkmm.h>
 #include <thread>
+#include <set>
 
 #include "core_widget.hpp"
 

bench/core_widget.hpp

@@ -3,6 +3,8 @@
 
 #include <lib94/lib94.hpp>
 #include <gtkmm.h>
+#include <mutex>
+#include <set>
 
 class core_widget : public Gtk::Widget {
 public:

include/lib94/lib94.hpp

@@ -1,11 +1,9 @@
 #ifndef LIB94_LIB94_HPP
 #define LIB94_LIB94_HPP
 
-#include <filesystem>
 #include <optional>
-#include <variant>
+#include <cstdint>
 #include <string>
-#include <random>
 #include <vector>
 #include <deque>
 #include <set>

lib94/core.cpp

@@ -1,9 +1,8 @@
 #include <lib94/lib94.hpp>
 #include <functional>
+#include <algorithm>
 #include <cassert>
 #include <random>
-#include <deque>
-#include <set>
 
 namespace lib94 {
 
@@ -70,7 +69,7 @@ namespace lib94 {
     new_place_at:
       number_t place_at = i == 0 ? 0 : number(prng);
 
-      for (std::pair<number_t, number_t> &other : placements)
+      for (const std::pair<number_t, number_t> &other : placements)
         //there has to be a better way
         for (number_t i = 0; i < (number_t)w->instructions.size(); ++i)
           if (((place_at + i) % LIB94_CORE_SIZE >= other.first && (place_at + i) % LIB94_CORE_SIZE < other.first + other.second) ||

lib94/executors.cpp

@@ -2,8 +2,6 @@
 #include <functional>
 #include <cassert>
 
-#include <cstdio>
-
 namespace lib94 {
 
   void enqueue_process(number_t pc);

lib94/warrior.cpp

@@ -3,7 +3,6 @@
 #include <cassert>
 #include <fstream>
 #include <cctype>
-#include <cstdio>
 #include <memory>
 #include <map>
 
@@ -78,7 +77,7 @@ namespace lib94 {
     unsigned source_line_number;
   };
 
-  //internal stage used by stage 1. also records the name and author outputs.
+  //internal state used by stage 1. also records some information passed to later stages.
   struct preprocessor_state {
     std::optional<std::string> name;
     std::optional<std::string> author;
@@ -515,6 +514,7 @@ namespace lib94 {
 
   }
 
+  //some information collected in stage 2
   struct parser_state {
     label_offset_set label_offsets;
     std::unique_ptr<expr> org_expr;

tabulator-mpi/head.cpp

@@ -1,5 +1,4 @@
 #include <lib94/lib94.hpp>
-#include <csignal>
 #include <cstdio>
 #include <mpi.h>
 

tabulator-mpi/worker.cpp

@@ -1,4 +1,5 @@
 #include <lib94/lib94.hpp>
+#include <ctime>
 #include <mpi.h>
 
 #include "constants.hpp"