Demote OptionProcessor from globals
Some more cutting back on global mutable state that's hard to reason about the
validity of. This makes it more obvious what actually depends on what aspects
of the option processor and should make it easier to come even on the TODO to
make GlobalVariables.options const.
PiperOrigin-RevId: 251294543
diff --git a/src/main/cpp/global_variables.cc b/src/main/cpp/global_variables.cc
index 3001e43..c1a2569 100644
--- a/src/main/cpp/global_variables.cc
+++ b/src/main/cpp/global_variables.cc
@@ -16,9 +16,8 @@
namespace blaze {
-GlobalVariables::GlobalVariables(OptionProcessor* option_processor)
- : option_processor(option_processor),
- server_pid(-1),
+GlobalVariables::GlobalVariables()
+ : server_pid(-1),
options(NULL), /* Initialized after parsing with option_processor. */
startup_time(0),
extract_data_time(0),
