options: fix misindentation

Ensure that valobj.yielding is cleared for options in the toplevel project.

Signed-off-by: Paolo Bonzini <[email protected]>

Author: bonzini

mesonbuild/options.py

@@ -937,7 +937,7 @@ def add_project_option(self, key: T.Union[OptionKey, str], valobj: AnyOptionType
                 # Subproject is set to yield, but top level
                 # project does not have an option of the same
                 pass
-            valobj.yielding = bool(valobj.parent)
+        valobj.yielding = bool(valobj.parent)
 
         self.options[key] = valobj
         self.project_options.add(key)

unittests/optiontests.py

@@ -121,6 +121,14 @@ def test_project_nonyielding(self):
         self.assertEqual(optstore.get_value_for(name, 'sub'), sub_value)
         self.assertEqual(num_options(optstore), 2)
 
+    def test_toplevel_project_yielding(self):
+        optstore = OptionStore(False)
+        name = 'someoption'
+        top_value = 'top'
+        vo = UserStringOption(name, 'A top level option', top_value, True)
+        optstore.add_project_option(OptionKey(name, ''), vo)
+        self.assertEqual(optstore.get_value_for(name, ''), top_value)
+
     def test_project_yielding(self):
         optstore = OptionStore(False)
         name = 'someoption'