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

bug: lines are limited to 128 chars #6

Closed
2 tasks done
char0n opened this issue Apr 11, 2024 · 0 comments · Fixed by #8
Closed
2 tasks done

bug: lines are limited to 128 chars #6

char0n opened this issue Apr 11, 2024 · 0 comments · Fixed by #8
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@char0n
Copy link

char0n commented Apr 11, 2024

Did you check existing issues?

  • I have read all the tree-sitter docs if it relates to using the parser
  • I have searched the existing issues

Tree-Sitter CLI Version, if relevant (output of tree-sitter --version)

tree-sitter@0.22.0

Describe the bug

Line that contains key of the mapping cannot be longer than 128 chars. Lexical analysis stops on long line and doesn't continue any further.

More info in ikatyang#59

Steps To Reproduce/Bad Parse Tree

swagger: "2.0"
paths:
  /pet/adding-new-pets/testing-thelong-size-path-name/testing-thepath/testingthepathnameggggggggggggggggggggggggglllllllllllllla: {}
test: 3

Expected Behavior/Parse Tree

The object under paths path will parse as a block mapping key pair and parsing will continue with parsing test: 3.

Repro

ikatyang#59

@char0n char0n added the bug Something isn't working label Apr 11, 2024
@ObserverOfTime ObserverOfTime changed the title bug: bug: lines are limited to 128 chars Apr 29, 2024
@ObserverOfTime ObserverOfTime added the help wanted Extra attention is needed label Apr 30, 2024
@amaanq amaanq closed this as completed in #8 May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants