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.tar.gz | |
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 da461e449..878b8c849 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)) |