Add another config::origin() overload

2 files changed, 15 insertions, 2 deletions

diff --git a/libbuild2/config/utility.cxx b/libbuild2/config/utility.cxx
index 243c03a..a64d4d5 100644
--- a/libbuild2/config/utility.cxx
+++ b/libbuild2/config/utility.cxx
@@ -177,14 +177,19 @@ namespace build2
     origin (const scope& rs, const variable& var)
     {
       // Make sure this is a config.* variable. This could matter since we
-      // reply on the semantics of value::extra. We could also detect
+      // rely on the semantics of value::extra. We could also detect
       // special variables like config.booted, some config.config.*, etc.,
       // (see config_save() for details) but that seems harmless.
       //
       if (var.name.compare (0, 7, "config.") != 0)
         throw invalid_argument ("config.* variable expected");
 
-      pair<lookup, size_t> org (rs.lookup_original (var));
+      return origin (rs, var, rs.lookup_original (var));
+    }
+
+    pair<variable_origin, lookup>
+    origin (const scope& rs, const variable& var, pair<lookup, size_t> org)
+    {
       pair<lookup, size_t> ovr (var.overrides == nullptr
                                 ? org
                                 : rs.lookup_override (var, org));
diff --git a/libbuild2/config/utility.hxx b/libbuild2/config/utility.hxx
index b998e3c..1e33568 100644
--- a/libbuild2/config/utility.hxx
+++ b/libbuild2/config/utility.hxx
@@ -514,6 +514,14 @@ namespace build2
 
     LIBBUILD2_SYMEXPORT pair<variable_origin, lookup>
     origin (const scope& rs, const variable&);
+
+    // As above but using the result of scope::lookup_original() or
+    // semantically equivalent (e.g., lookup_namespace()).
+    //
+    // Note that this version does not check that the variable is config.*.
+    //
+    LIBBUILD2_SYMEXPORT pair<variable_origin, lookup>
+    origin (const scope& rs, const variable&, pair<lookup, size_t> original);
   }
 }