Skip to content

Commit 93e83b3

Browse files
bonzinieli-schwartz
authored andcommitted
options: fix types for initialize_from_subproject_call
No code changes, just making mypy annotations truthful. Signed-off-by: Paolo Bonzini <[email protected]> (cherry picked from commit d55364f)
1 parent 4bd7d63 commit 93e83b3

File tree

1 file changed

+6
-9
lines changed

1 file changed

+6
-9
lines changed

mesonbuild/options.py

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1400,21 +1400,19 @@ def validate_cmd_line_options(self, cmd_line_options: OptionStringLikeDict) -> N
14001400
keys = ', '.join(unknown_options)
14011401
raise MesonException(f'Unknown options: {keys}')
14021402

1403-
def hacky_mchackface_back_to_list(self, optdict: T.Dict[str, str]) -> T.List[str]:
1403+
def hacky_mchackface_back_to_list(self, optdict: T.Union[T.List[str], OptionStringLikeDict]) -> T.List[str]:
14041404
if isinstance(optdict, dict):
14051405
return [f'{k}={v}' for k, v in optdict.items()]
14061406
return optdict
14071407

14081408
def initialize_from_subproject_call(self,
14091409
subproject: str,
1410-
spcall_default_options: T.Union[T.List[str], OptionStringLikeDict],
1411-
project_default_options: T.Union[T.List[str], OptionStringLikeDict],
1412-
cmd_line_options: T.Union[T.List[str], OptionStringLikeDict]) -> None:
1410+
spcall_default_options_in: T.Union[T.List[str], OptionStringLikeDict],
1411+
project_default_options_in: T.Union[T.List[str], OptionStringLikeDict],
1412+
cmd_line_options: OptionStringLikeDict) -> None:
14131413
is_first_invocation = True
1414-
spcall_default_options = self.hacky_mchackface_back_to_list(spcall_default_options) # type: ignore [arg-type]
1415-
project_default_options = self.hacky_mchackface_back_to_list(project_default_options) # type: ignore [arg-type]
1416-
if isinstance(spcall_default_options, str):
1417-
spcall_default_options = [spcall_default_options]
1414+
spcall_default_options = self.hacky_mchackface_back_to_list(spcall_default_options_in)
1415+
project_default_options = self.hacky_mchackface_back_to_list(project_default_options_in)
14181416
for o in itertools.chain(project_default_options, spcall_default_options):
14191417
keystr, valstr = o.split('=', 1)
14201418
key = OptionKey.from_string(keystr)
@@ -1432,7 +1430,6 @@ def initialize_from_subproject_call(self,
14321430
aug_str = str(key)
14331431
self.augments[aug_str] = valstr
14341432
# Check for pending options
1435-
assert isinstance(cmd_line_options, dict)
14361433
for key, valstr in cmd_line_options.items(): # type: ignore [assignment]
14371434
if not isinstance(key, OptionKey):
14381435
key = OptionKey.from_string(key)

0 commit comments

Comments
 (0)