From a60da308bfcc003fd07d2b7d848ccb8d166e472a Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Mon, 11 Apr 2016 14:38:00 +0200
Subject: Redo config inheritance logic

---
 build2/config/utility | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'build2/config/utility')

diff --git a/build2/config/utility b/build2/config/utility
index 451df30..8a93c09 100644
--- a/build2/config/utility
+++ b/build2/config/utility
@@ -21,12 +21,13 @@ namespace build2
     //
     // If override is true and the variable doesn't come from this root scope
     // or from the command line (i.e., it is inherited from the amalgamtion),
-    // then its value is "overridden" for this root scope.
+    // then its value is "overridden" to the default value on this root scope.
     //
     // Return the reference to the value as well as the indication of whether
-    // the value is "new", that is, it was either set (including override) or
-    // it came from the command line and was not inherited. This is usually
-    // used to test the new value.
+    // the value is "new", that is, it was set to the default value (inherited
+    // or not, including overrides). We also treat command line overrides
+    // (inherited or not) as new. This flag is usually used to test that the
+    // new value is valid, etc.
     //
     template <typename T>
     pair<reference_wrapper<const value>, bool>
-- 
cgit v1.1