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 | |
| 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.
| -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)) |