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

fix bug in parsing extended colon-like operators #29314

Merged
merged 1 commit into from
Sep 24, 2018
Merged

Conversation

JeffBezanson
Copy link
Member

#26262 added some operators to the : precedence level, but the parsing code for that is a bit non-standard so they did not actually parse. This fixes it. Should be non-breaking since trying to use the operators gave an error before.

@JeffBezanson JeffBezanson added parser Language parsing and surface syntax bugfix This change fixes an existing bug labels Sep 21, 2018
@stevengj
Copy link
Member

stevengj commented Sep 22, 2018

It looks like there is a problem with etcetera, too.

julia> Meta.parse("a ≔ b")
:($(Expr(:≔, :a, :b)))

Did we actually want to parse this as a call expression so that it could be overloaded? Or are these Expr(:≔, …) nodes mainly for use in macros?

Sorry, my bad.

@JeffBezanson JeffBezanson merged commit 99e0b3b into master Sep 24, 2018
@JeffBezanson JeffBezanson deleted the jb/colonops branch September 24, 2018 18:02
KristofferC pushed a commit that referenced this pull request Oct 6, 2018
KristofferC pushed a commit that referenced this pull request Feb 11, 2019
KristofferC pushed a commit that referenced this pull request Feb 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This change fixes an existing bug parser Language parsing and surface syntax
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants