diff options
| author | Eli Schwartz <eschwartz@archlinux.org> | 2023-03-19 17:05:37 -0400 |
|---|---|---|
| committer | Eli Schwartz <eschwartz@archlinux.org> | 2023-05-24 19:05:06 -0400 |
| commit | 1ff996ea6753595d181e357a6b8821c1bd5c8724 (patch) | |
| tree | b74432cfe0444a9c5bc080b22f55fced0bde8578 /mesonbuild | |
| parent | d3804d057976b08d37b64ccd1eb8c02fc36e5f78 (diff) | |
| download | meson-1ff996ea6753595d181e357a6b8821c1bd5c8724.zip meson-1ff996ea6753595d181e357a6b8821c1bd5c8724.tar.gz meson-1ff996ea6753595d181e357a6b8821c1bd5c8724.tar.bz2 | |
remove inaccurate comment
This convertor was initially implemented doing all the things the TODO
says it doesn't yet do. The freestanding interpreter function is what
doesn't do this.
Diffstat (limited to 'mesonbuild')
| -rw-r--r-- | mesonbuild/interpreter/type_checking.py | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/mesonbuild/interpreter/type_checking.py b/mesonbuild/interpreter/type_checking.py index 9b2e46c..beae001 100644 --- a/mesonbuild/interpreter/type_checking.py +++ b/mesonbuild/interpreter/type_checking.py @@ -95,13 +95,8 @@ def _install_mode_validator(mode: T.List[T.Union[str, bool, int]]) -> T.Optional def _install_mode_convertor(mode: T.Optional[T.List[T.Union[str, bool, int]]]) -> FileMode: - """Convert the DSL form of the `install_mode` keyword argument to `FileMode` + """Convert the DSL form of the `install_mode` keyword argument to `FileMode`""" - This is not required, and if not required returns None - - TODO: It's not clear to me why this needs to be None and not just return an - empty FileMode. - """ # this has already been validated by the validator return FileMode(*(m if isinstance(m, str) else None for m in mode)) |