diff options
author | Dylan Baker <dylan@pnwbakers.com> | 2022-09-07 14:03:41 -0700 |
---|---|---|
committer | Eli Schwartz <eschwartz@archlinux.org> | 2022-11-29 23:26:05 -0500 |
commit | d5e899c76808d45854a06a4ba2b006da32480165 (patch) | |
tree | eb2f315230d2448edce06fa1a82666d683b2d6bb /mesonbuild/backend | |
parent | a5d547e0d9ccb523f0baab7fd32e782aa075fb42 (diff) | |
download | meson-d5e899c76808d45854a06a4ba2b006da32480165.zip meson-d5e899c76808d45854a06a4ba2b006da32480165.tar.gz meson-d5e899c76808d45854a06a4ba2b006da32480165.tar.bz2 |
pylint: enable the bad_builtin checker
This finds uses of deny-listed functions, which defaults to map and
filter. These functions should be replaced by comprehensions in
idiomatic python because:
1. comprehensions are more heavily optimized and are often faster
2. They avoid the need for lambdas in some cases, which make them
faster
3. you can do the equivalent in one statement rather than two, which
is faster
4. They're easier to read
5. if you need a concrete instance (ie, a list) then you don't have
to convert the iterator to a list afterwards
Diffstat (limited to 'mesonbuild/backend')
-rw-r--r-- | mesonbuild/backend/ninjabackend.py | 4 | ||||
-rw-r--r-- | mesonbuild/backend/xcodebackend.py | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/mesonbuild/backend/ninjabackend.py b/mesonbuild/backend/ninjabackend.py index aed4fd0..7f47823 100644 --- a/mesonbuild/backend/ninjabackend.py +++ b/mesonbuild/backend/ninjabackend.py @@ -209,8 +209,8 @@ class NinjaRule: return NinjaCommandArg(c) self.name = rule - self.command = list(map(strToCommandArg, command)) # includes args which never go into a rspfile - self.args = list(map(strToCommandArg, args)) # args which will go into a rspfile, if used + self.command = [strToCommandArg(c) for c in command] # includes args which never go into a rspfile + self.args = [strToCommandArg(a) for a in args] # args which will go into a rspfile, if used self.description = description self.deps = deps # depstyle 'gcc' or 'msvc' self.depfile = depfile diff --git a/mesonbuild/backend/xcodebackend.py b/mesonbuild/backend/xcodebackend.py index 9e101ce..c56036b 100644 --- a/mesonbuild/backend/xcodebackend.py +++ b/mesonbuild/backend/xcodebackend.py @@ -541,7 +541,7 @@ class XCodeBackend(backends.Backend): self.custom_aggregate_targets = {} self.build_all_tdep_id = self.gen_id() # FIXME: filter out targets that are not built by default. - target_dependencies = list(map(lambda t: self.pbx_dep_map[t], self.build_targets)) + target_dependencies = [self.pbx_dep_map[t] for t in self.build_targets] custom_target_dependencies = [self.pbx_custom_dep_map[t] for t in self.custom_targets] aggregated_targets = [] aggregated_targets.append((self.all_id, 'ALL_BUILD', |