From 0c875e16c4f79cad5440304ea89d44c9d600b6f0 Mon Sep 17 00:00:00 2001 From: Daniel Mensinger Date: Wed, 4 Dec 2019 16:59:10 +0100 Subject: [PATCH] lgtm: fix Non-iterable used in for loop Not sure why LGTM is complaining about this, but moving this from a static to a local context should fix this. --- mesonbuild/cmake/interpreter.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/mesonbuild/cmake/interpreter.py b/mesonbuild/cmake/interpreter.py index cb0416dee..d9f1e18f6 100644 --- a/mesonbuild/cmake/interpreter.py +++ b/mesonbuild/cmake/interpreter.py @@ -199,8 +199,6 @@ class OutputTargetMap: return '__art_{}__'.format(os.path.basename(fname)) class ConverterTarget: - lang_cmake_to_meson = {val.lower(): key for key, val in language_map.items()} - def __init__(self, target: CMakeTarget, env: Environment): self.env = env self.artifacts = target.artifacts @@ -240,7 +238,8 @@ class ConverterTarget: for i in target.files: # Determine the meson language - lang = ConverterTarget.lang_cmake_to_meson.get(i.language.lower(), 'c') + lang_cmake_to_meson = {val.lower(): key for key, val in language_map.items()} + lang = lang_cmake_to_meson.get(i.language.lower(), 'c') if lang not in self.languages: self.languages += [lang] if lang not in self.compile_opts: