diff options
author | Daniel Mensinger <daniel@mensinger-ka.de> | 2019-12-04 16:59:10 +0100 |
---|---|---|
committer | Jussi Pakkanen <jpakkane@gmail.com> | 2019-12-05 00:22:10 +0200 |
commit | 0c875e16c4f79cad5440304ea89d44c9d600b6f0 (patch) | |
tree | ef9874146ec862f2b08ed0979329a05cac6a6d74 | |
parent | fe853ee516e1e7b392753a6e8f1f0b9cad6fb54f (diff) | |
download | meson-0c875e16c4f79cad5440304ea89d44c9d600b6f0.zip meson-0c875e16c4f79cad5440304ea89d44c9d600b6f0.tar.gz meson-0c875e16c4f79cad5440304ea89d44c9d600b6f0.tar.bz2 |
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.
-rw-r--r-- | mesonbuild/cmake/interpreter.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/mesonbuild/cmake/interpreter.py b/mesonbuild/cmake/interpreter.py index cb0416d..d9f1e18 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: |