diff --git a/.pylintrc b/.pylintrc index 31ac9259b..423066a38 100644 --- a/.pylintrc +++ b/.pylintrc @@ -12,6 +12,7 @@ enable= bad-indentation, bare-except, compare-to-zero, + consider-iterating-dictionary, dangerous-default-value, deprecated-lambda, len-as-condition, diff --git a/mesonbuild/backend/xcodebackend.py b/mesonbuild/backend/xcodebackend.py index 01053c185..b1a89567d 100644 --- a/mesonbuild/backend/xcodebackend.py +++ b/mesonbuild/backend/xcodebackend.py @@ -1282,7 +1282,7 @@ class XCodeBackend(backends.Backend): def generate_pbx_sources_build_phase(self, objects_dict): - for name in self.source_phase.keys(): + for name in self.source_phase: phase_dict = PbxDict() t = self.build_targets[name] objects_dict.add_item(t.buildphasemap[name], phase_dict, 'Sources') diff --git a/mesonbuild/mparser.py b/mesonbuild/mparser.py index 10796827c..1617dd1fa 100644 --- a/mesonbuild/mparser.py +++ b/mesonbuild/mparser.py @@ -324,7 +324,7 @@ class ArgumentNode(BaseNode): self.arguments += [statement] def set_kwarg(self, name: IdNode, value: BaseNode) -> None: - if name.value in [x.value for x in self.kwargs.keys() if isinstance(x, IdNode)]: + if any((isinstance(x, IdNode) and name.value == x.value) for x in self.kwargs): mlog.warning(f'Keyword argument "{name.value}" defined multiple times.', location=self) mlog.warning('This will be an error in future Meson releases.') self.kwargs[name] = value