Minor cleanups

1 files changed, 104 insertions, 124 deletions

diff --git a/libbuild2/context.hxx b/libbuild2/context.hxx
index f662fdd..86867a1 100644
--- a/libbuild2/context.hxx
+++ b/libbuild2/context.hxx
@@ -80,7 +80,7 @@ namespace build2
   // false. Note that in this case we still switch to the desired phase. See
   // the phase_{lock,switch,unlock} implementations for details.
   //
-  class LIBBUILD2_SYMEXPORT phase_mutex
+  class LIBBUILD2_SYMEXPORT run_phase_mutex
   {
   public:
     // Acquire a phase lock potentially blocking (unless already in the
@@ -101,20 +101,18 @@ namespace build2
     bool
     relock (run_phase unlock, run_phase lock);
 
-  private:
-    friend struct phase_lock;
-    friend struct phase_unlock;
-    friend struct phase_switch;
-
-    phase_mutex ()
+  public:
+    run_phase_mutex ()
         : fail_ (false), lc_ (0), mc_ (0), ec_ (0)
     {
       phase = run_phase::load;
     }
 
-    static phase_mutex instance;
-
   private:
+    friend struct phase_lock;
+    friend struct phase_unlock;
+    friend struct phase_switch;
+
     // We have a counter for each phase which represents the number of threads
     // in or waiting for this phase.
     //
@@ -140,6 +138,8 @@ namespace build2
     mutex lm_;
   };
 
+  extern run_phase_mutex phase_mutex;
+
   // Grab a new phase lock releasing it on destruction. The lock can be
   // "owning" or "referencing" (recursive).
   //
@@ -260,87 +260,6 @@ namespace build2
     bool phase;
   };
 
-  // Cached variables.
-  //
-  // Note: consider printing in info meta-operation if adding anything here.
-  //
-  LIBBUILD2_SYMEXPORT extern const variable* var_src_root;
-  LIBBUILD2_SYMEXPORT extern const variable* var_out_root;
-  LIBBUILD2_SYMEXPORT extern const variable* var_src_base;
-  LIBBUILD2_SYMEXPORT extern const variable* var_out_base;
-  LIBBUILD2_SYMEXPORT extern const variable* var_forwarded;
-
-  LIBBUILD2_SYMEXPORT extern const variable* var_project;
-  LIBBUILD2_SYMEXPORT extern const variable* var_amalgamation;
-  LIBBUILD2_SYMEXPORT extern const variable* var_subprojects;
-  LIBBUILD2_SYMEXPORT extern const variable* var_version;
-
-  // project.url
-  //
-  LIBBUILD2_SYMEXPORT extern const variable* var_project_url;
-
-  // project.summary
-  //
-  LIBBUILD2_SYMEXPORT extern const variable* var_project_summary;
-
-  // import.target
-  //
-  LIBBUILD2_SYMEXPORT extern const variable* var_import_target;
-
-  // [bool] target visibility
-  //
-  LIBBUILD2_SYMEXPORT extern const variable* var_clean;
-
-  // Forwarded configuration backlink mode. Valid values are:
-  //
-  // false     - no link.
-  // true      - make a link using appropriate mechanism.
-  // symbolic  - make a symbolic link.
-  // hard      - make a hard link.
-  // copy      - make a copy.
-  // overwrite - copy over but don't remove on clean (committed gen code).
-  //
-  // Note that it can be set by a matching rule as a rule-specific variable.
-  //
-  // [string] target visibility
-  //
-  LIBBUILD2_SYMEXPORT extern const variable* var_backlink;
-
-  // Prerequisite inclusion/exclusion. Valid values are:
-  //
-  // false  - exclude.
-  // true   - include.
-  // adhoc  - include but treat as an ad hoc input.
-  //
-  // If a rule uses prerequisites as inputs (as opposed to just matching them
-  // with the "pass-through" semantics), then the adhoc value signals that a
-  // prerequisite is an ad hoc input. A rule should match and execute such a
-  // prerequisite (whether its target type is recognized as suitable input or
-  // not) and assume that the rest will be handled by the user (e.g., it will
-  // be passed via a command line argument or some such). Note that this
-  // mechanism can be used to both treat unknown prerequisite types as inputs
-  // (for example, linker scripts) as well as prevent treatment of known
-  // prerequisite types as such while still matching and executing them (for
-  // example, plugin libraries).
-  //
-  // A rule with the "pass-through" semantics should treat the adhoc value
-  // the same as true.
-  //
-  // To query this value in rule implementations use the include() helpers
-  // from <libbuild2/prerequisites.hxx>.
-  //
-  // [string] prereq visibility
-  //
-  LIBBUILD2_SYMEXPORT extern const variable* var_include;
-
-  LIBBUILD2_SYMEXPORT extern const char var_extension[10]; // "extension"
-
-  // The build.* namespace.
-  //
-  // .meta_operation
-  //
-  LIBBUILD2_SYMEXPORT extern const variable* var_build_meta_operation;
-
   // Current action (meta/operation).
   //
   // The names unlike info are available during boot but may not yet be
@@ -383,37 +302,13 @@ namespace build2
   LIBBUILD2_SYMEXPORT extern atomic_count target_count;
   LIBBUILD2_SYMEXPORT extern atomic_count skip_count;
 
-  inline void
-  set_current_mif (const meta_operation_info& mif)
-  {
-    if (current_mname != mif.name)
-    {
-      current_mname = mif.name;
-      global_scope->rw ().assign (var_build_meta_operation) = mif.name;
-    }
+  void
+  set_current_mif (const meta_operation_info&);
 
-    current_mif = &mif;
-    current_on = 0; // Reset.
-  }
-
-  inline void
-  set_current_oif (const operation_info& inner_oif,
-                   const operation_info* outer_oif = nullptr,
-                   bool diag_noise = true)
-  {
-    current_oname = (outer_oif == nullptr ? inner_oif : *outer_oif).name;
-    current_inner_oif = &inner_oif;
-    current_outer_oif = outer_oif;
-    current_on++;
-    current_mode = inner_oif.mode;
-    current_diag_noise = diag_noise;
-
-    // Reset counters (serial execution).
-    //
-    dependency_count.store (0, memory_order_relaxed);
-    target_count.store (0, memory_order_relaxed);
-    skip_count.store (0, memory_order_relaxed);
-  }
+  void
+  set_current_oif (const operation_info& inner,
+                   const operation_info* outer = nullptr,
+                   bool diag_noise = true);
 
   // Keep going flag.
   //
@@ -456,6 +351,12 @@ namespace build2
   //
   LIBBUILD2_SYMEXPORT extern bool dry_run;
 
+  // Reset the build state. In particular, this removes all the targets,
+  // scopes, and variables.
+  //
+  LIBBUILD2_SYMEXPORT variable_overrides
+  reset (const strings& cmd_vars);
+
   // Config module entry points.
   //
   LIBBUILD2_SYMEXPORT extern void (*config_save_variable) (
@@ -468,11 +369,90 @@ namespace build2
     bool lifted,
     const location&);
 
-  // Reset the build state. In particular, this removes all the targets,
-  // scopes, and variables.
+  // Cached variables.
+  //
+
+  // Note: consider printing in info meta-operation if adding anything here.
+  //
+  LIBBUILD2_SYMEXPORT extern const variable* var_src_root;
+  LIBBUILD2_SYMEXPORT extern const variable* var_out_root;
+  LIBBUILD2_SYMEXPORT extern const variable* var_src_base;
+  LIBBUILD2_SYMEXPORT extern const variable* var_out_base;
+  LIBBUILD2_SYMEXPORT extern const variable* var_forwarded;
+
+  LIBBUILD2_SYMEXPORT extern const variable* var_project;
+  LIBBUILD2_SYMEXPORT extern const variable* var_amalgamation;
+  LIBBUILD2_SYMEXPORT extern const variable* var_subprojects;
+  LIBBUILD2_SYMEXPORT extern const variable* var_version;
+
+  // project.url
+  //
+  LIBBUILD2_SYMEXPORT extern const variable* var_project_url;
+
+  // project.summary
+  //
+  LIBBUILD2_SYMEXPORT extern const variable* var_project_summary;
+
+  // import.target
+  //
+  LIBBUILD2_SYMEXPORT extern const variable* var_import_target;
+
+  // [bool] target visibility
+  //
+  LIBBUILD2_SYMEXPORT extern const variable* var_clean;
+
+  // Forwarded configuration backlink mode. Valid values are:
+  //
+  // false     - no link.
+  // true      - make a link using appropriate mechanism.
+  // symbolic  - make a symbolic link.
+  // hard      - make a hard link.
+  // copy      - make a copy.
+  // overwrite - copy over but don't remove on clean (committed gen code).
+  //
+  // Note that it can be set by a matching rule as a rule-specific variable.
+  //
+  // [string] target visibility
+  //
+  LIBBUILD2_SYMEXPORT extern const variable* var_backlink;
+
+  // Prerequisite inclusion/exclusion. Valid values are:
+  //
+  // false  - exclude.
+  // true   - include.
+  // adhoc  - include but treat as an ad hoc input.
+  //
+  // If a rule uses prerequisites as inputs (as opposed to just matching them
+  // with the "pass-through" semantics), then the adhoc value signals that a
+  // prerequisite is an ad hoc input. A rule should match and execute such a
+  // prerequisite (whether its target type is recognized as suitable input or
+  // not) and assume that the rest will be handled by the user (e.g., it will
+  // be passed via a command line argument or some such). Note that this
+  // mechanism can be used to both treat unknown prerequisite types as inputs
+  // (for example, linker scripts) as well as prevent treatment of known
+  // prerequisite types as such while still matching and executing them (for
+  // example, plugin libraries).
+  //
+  // A rule with the "pass-through" semantics should treat the adhoc value
+  // the same as true.
+  //
+  // To query this value in rule implementations use the include() helpers
+  // from <libbuild2/prerequisites.hxx>.
+  //
+  // [string] prereq visibility
+  //
+  LIBBUILD2_SYMEXPORT extern const variable* var_include;
+
+  LIBBUILD2_SYMEXPORT extern const char var_extension[10]; // "extension"
+
+  // The build.* namespace.
+  //
+  // .meta_operation
+  //
+  LIBBUILD2_SYMEXPORT extern const variable* var_build_meta_operation;
+
+  // Utility.
   //
-  LIBBUILD2_SYMEXPORT variable_overrides
-  reset (const strings& cmd_vars);
 
   // Return the project name or empty string if unnamed.
   //