Skip to content

Comments

Add erroneous for-loop test case for CFG#8957

Merged
charliermarsh merged 1 commit intomainfrom
charlie/cfg
Dec 1, 2023
Merged

Add erroneous for-loop test case for CFG#8957
charliermarsh merged 1 commit intomainfrom
charlie/cfg

Conversation

@charliermarsh
Copy link
Member

No description provided.

block2 -- "range(5)" --> block0
block2 -- "else" --> block1
block1 --> return
block0 --> return
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

block0 here (pass)should point toblock2 (for`).

In the next example, you can see that block1 (pass) gets pointed to black3 (for). But the structures are identical. It has to do with the lack of a statement following the loop.

@charliermarsh charliermarsh enabled auto-merge (squash) December 1, 2023 23:06
@charliermarsh charliermarsh added the internal An internal refactor or improvement label Dec 1, 2023
@charliermarsh charliermarsh merged commit 277cd80 into main Dec 1, 2023
@charliermarsh charliermarsh deleted the charlie/cfg branch December 1, 2023 23:11
@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2023

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal An internal refactor or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant