support for-else and while-else #56153
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This adopts the semantics discussed in #1289, namely the `else` block is executed whenever the loop never runs. Unlike Python, `break` and `continue` are irrelevant. Multidimensional loops are not supported since there is some ambiguity whether e.g. ```julia for i in 1:3, j in 1:0 print(1) else print(2) end ``` should print 2 once, thrice or maybe not even at all. Currently only supported in the flisp parser, so requires `JULIA_USE_FLISP_PARSER=1`. I could use some guidance on the necessary steps to add this to JuliaSyntax as well - AFAIU this would also require #56110 first. closes #1289
simeonschaub
added
parser
simeonschaub
added a commit
to JuliaLang/JuliaSyntax.jl
that referenced
this pull request
Oct 14, 2024
simeonschaub
added a commit
to JuliaLang/JuliaSyntax.jl
that referenced
this pull request
Oct 14, 2024
|
downvote. IMO it's a bad idea to add not-obvious-what-it-does syntax that also happens to not mean the same thing at all as identical syntax in the world's most popular language. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adopts the semantics discussed in #1289, namely the
elseblock is executed whenever the loop never runs. Unlike Python,breakandcontinueare irrelevant. Multidimensional loops are not supported since there is some ambiguity whether e.g.should print 2 once, thrice or maybe not even at all.
Currently only supported in the flisp parser, so requires
JULIA_USE_FLISP_PARSER=1. I could use some guidance on the necessary steps to add this to JuliaSyntax as well - AFAIU this would also require #56110 first.closes #1289