diff options
| author | Eli Schwartz <eschwartz@archlinux.org> | 2021-10-19 19:56:48 -0400 |
|---|---|---|
| committer | Eli Schwartz <eschwartz@archlinux.org> | 2021-10-26 20:53:43 -0400 |
| commit | b1f0eec38f0da1966aaedc7cb1ec2a737e2b7c64 (patch) | |
| tree | e4826a6c73189bc72a1eb0d46987d7c670e73bf0 /mesonbuild/mlog.py | |
| parent | 739de7b088614e811b8f68ab8357e088239aafa7 (diff) | |
| download | meson-b1f0eec38f0da1966aaedc7cb1ec2a737e2b7c64.tar.gz | |
fix lgtm.com "Use of the return value of a procedure"
we return _log even though this entire family of functions returns None,
because a side effect of returning is that the other version of the
function is not run.
We can do that more obviously, using an else clause that doesn't attach
meaning to return values.
Diffstat (limited to 'mesonbuild/mlog.py')
| -rw-r--r-- | mesonbuild/mlog.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/mesonbuild/mlog.py b/mesonbuild/mlog.py index 18cbc4818..8347f2d49 100644 --- a/mesonbuild/mlog.py +++ b/mesonbuild/mlog.py @@ -251,8 +251,9 @@ def cmd_ci_include(file: str) -> None: def log(*args: TV_Loggable, is_error: bool = False, once: bool = False, **kwargs: T.Any) -> None: if once: - return log_once(*args, is_error=is_error, **kwargs) - return _log(*args, is_error=is_error, **kwargs) + log_once(*args, is_error=is_error, **kwargs) + else: + _log(*args, is_error=is_error, **kwargs) def _log(*args: TV_Loggable, is_error: bool = False, |