From cb0265a6b28e74e0a20fe247cad7e82c68f9412b Mon Sep 17 00:00:00 2001 From: Dylan Baker Date: Mon, 14 Sep 2020 12:01:34 -0700 Subject: [PATCH] pylint: Catch cases of `if len(container)` which should be replaced by `if container` Unfortunately this doesn't catch other abuses of len(continauer) like, `len(container) 0`, see: https://github.com/PyCQA/pylint/issues/3751 --- .pylintrc | 4 +++- mesonbuild/backend/ninjabackend.py | 2 +- mesonbuild/build.py | 2 +- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/.pylintrc b/.pylintrc index 10483d33d..37c9f9f32 100644 --- a/.pylintrc +++ b/.pylintrc @@ -6,4 +6,6 @@ score=no [MESSAGES CONTROL] disable=all -enable=unreachable \ No newline at end of file +enable= + len-as-condition, + unreachable diff --git a/mesonbuild/backend/ninjabackend.py b/mesonbuild/backend/ninjabackend.py index e21da2939..66fc46489 100644 --- a/mesonbuild/backend/ninjabackend.py +++ b/mesonbuild/backend/ninjabackend.py @@ -2587,7 +2587,7 @@ https://gcc.gnu.org/bugzilla/show_bug.cgi?id=47485''')) def get_link_whole_args(self, linker, target): target_args = self.build_target_link_arguments(linker, target.link_whole_targets) - return linker.get_link_whole_for(target_args) if len(target_args) else [] + return linker.get_link_whole_for(target_args) if target_args else [] @lru_cache(maxsize=None) def guess_library_absolute_path(self, linker, libname, search_dirs, patterns): diff --git a/mesonbuild/build.py b/mesonbuild/build.py index f56e48469..30fa77a5e 100644 --- a/mesonbuild/build.py +++ b/mesonbuild/build.py @@ -2232,7 +2232,7 @@ class CustomTarget(Target): def get_dep_outname(self, infilenames): if self.depfile is None: raise InvalidArguments('Tried to get depfile name for custom_target that does not have depfile defined.') - if len(infilenames): + if infilenames: plainname = os.path.basename(infilenames[0]) basename = os.path.splitext(plainname)[0] return self.depfile.replace('@BASENAME@', basename).replace('@PLAINNAME@', plainname)