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

Parser regression in 1.10beta1 #50703

Closed
nilshg opened this issue Jul 28, 2023 · 3 comments
Closed

Parser regression in 1.10beta1 #50703

nilshg opened this issue Jul 28, 2023 · 3 comments
Labels
parser Language parsing and surface syntax regression Regression in behavior compared to a previous version
Milestone

Comments

@nilshg
Copy link
Sponsor Contributor

nilshg commented Jul 28, 2023

Fredrik suggested that I link this JuliaSyntax issue here for consideration on the 1.10 milestone:

JuliaLang/JuliaSyntax.jl#332

In short, using Markdown; md"""$)""" errors in JuliaSyntax but used to parse fine in the old parser (i.e. works on 1.9 and earlier).

@fingolfin
Copy link
Contributor

This also affects us: oscar-system/Oscar.jl#2596

@KristofferC KristofferC added the parser Language parsing and surface syntax label Jul 28, 2023
@JeffBezanson
Copy link
Sponsor Member

Reduced:

julia> Meta.parse(")", 1, greedy=false, raise=true)

Should give an error of course, but gives an unexpected internal error. @c42f

@JeffBezanson JeffBezanson added the regression Regression in behavior compared to a previous version label Jul 31, 2023
@matthias314
Copy link
Contributor

Just a comment: As far as I can tell, one also runs into this error when building the Julia documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
parser Language parsing and surface syntax regression Regression in behavior compared to a previous version
Projects
None yet
Development

No branches or pull requests

5 participants