Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Different list comprehension variable will share same place of variable #4843

Closed
Shylock-Hg opened this issue Nov 9, 2022 · 0 comments · Fixed by #4850
Closed

Different list comprehension variable will share same place of variable #4843

Shylock-Hg opened this issue Nov 9, 2022 · 0 comments · Fixed by #4850
Assignees
Labels
severity/minor Severity of bug type/bug Type: something is unexpected
Milestone

Comments

@Shylock-Hg
Copy link
Contributor

Please check the FAQ documentation before raising an issue

Describe the bug (required)

return [i in e where i > 1] Union return [i in e where i > 1]

The first list comprehension expression will share variable e in same place, because we don't provide mechanism for inner variable, and put inner variable to global too.
And #4498 change global unique list comprehension variables to be same.

Your Environments (required)

  • OS: uname -a
  • Compiler: g++ --version or clang++ --version
  • CPU: lscpu
  • Commit id (e.g. a3ffc7d8)

How To Reproduce(required)

Steps to reproduce the behavior:

  1. Step 1
  2. Step 2
  3. Step 3

Expected behavior

Additional context

@Shylock-Hg Shylock-Hg added the type/bug Type: something is unexpected label Nov 9, 2022
@Shylock-Hg Shylock-Hg self-assigned this Nov 9, 2022
@Shylock-Hg Shylock-Hg mentioned this issue Nov 9, 2022
11 tasks
@Sophie-Xie Sophie-Xie added this to the v3.4.0 milestone Nov 9, 2022
@jinyingsunny jinyingsunny added the severity/minor Severity of bug label Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity/minor Severity of bug type/bug Type: something is unexpected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants