diff options
author | Eli Schwartz <eschwartz@archlinux.org> | 2021-10-19 19:13:59 -0400 |
---|---|---|
committer | Eli Schwartz <eschwartz@archlinux.org> | 2021-10-26 20:53:41 -0400 |
commit | 1979132a9d109daba67e1b2665342fc6506eb323 (patch) | |
tree | f013d890368a156304852cef4c76ded4604bde32 | |
parent | 0fea9965ada82845c1e8a908078c573a68a2029a (diff) | |
download | meson-1979132a9d109daba67e1b2665342fc6506eb323.zip meson-1979132a9d109daba67e1b2665342fc6506eb323.tar.gz meson-1979132a9d109daba67e1b2665342fc6506eb323.tar.bz2 |
do not save variable when all we want is the side effect of popping it
It's redefined on every loop iteration, and as the comment says, we just
want to make sure the next loop skips a value.
-rw-r--r-- | unittests/allplatformstests.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/unittests/allplatformstests.py b/unittests/allplatformstests.py index da461e4..878b8c8 100644 --- a/unittests/allplatformstests.py +++ b/unittests/allplatformstests.py @@ -919,7 +919,7 @@ class AllPlatformTests(BasePlatformTests): wcc = compiler_from_language(env, lang, MachineChoice.HOST) wlinker = detect_static_linker(env, wcc) # Pop it so we don't use it for the next detection - evalue = os.environ.pop('AR') + os.environ.pop('AR') # Must be the same type since it's a wrapper around the same exelist self.assertIs(type(cc), type(wcc)) self.assertIs(type(linker), type(wlinker)) |