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 crash when parsing malformed function(where #388

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

c42f
Copy link
Member

@c42f c42f commented Nov 10, 2023

When word operators are parsed as atoms (ie, identifiers), the kind should be remapped as such. This fixes a crash when parsing function(where because was_eventually_call assumes that a kind of K"where" implies an internal node.

Part of #380

When word operators are parsed as atoms (ie, identifiers), the kind
should be remapped as such. This fixes a crash when parsing
`function(where` because `was_eventually_call` assumes that a kind of
`K"where"` implies an internal node.
Copy link

codecov bot commented Nov 10, 2023

Codecov Report

Merging #388 (ef75191) into main (5757535) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #388   +/-   ##
=======================================
  Coverage   96.58%   96.58%           
=======================================
  Files          14       14           
  Lines        4185     4187    +2     
=======================================
+ Hits         4042     4044    +2     
  Misses        143      143           
Files Coverage Δ
src/parser.jl 98.10% <100.00%> (+<0.01%) ⬆️

@c42f c42f mentioned this pull request Nov 10, 2023
10 tasks
@c42f c42f merged commit a6b94d2 into main Nov 10, 2023
32 checks passed
@c42f c42f deleted the caf/fix-crash-function-where branch November 10, 2023 09:03
c42f added a commit that referenced this pull request Nov 12, 2023
When word operators are parsed as atoms (ie, identifiers), the kind
should be remapped as such. This fixes a crash when parsing
`function(where` because `was_eventually_call` assumes that a kind of
`K"where"` implies an internal node.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant