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: correctly recognize multiple inheritance for @class #10

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

wookayin
Copy link
Contributor

@wookayin wookayin commented Dec 4, 2023

Problem: Inheritance from multiple parent types are not correctly
recognized, only the first parent class was parsed as identifier.

Solution: Use commaSep to recognize multiple parent types.

@wookayin
Copy link
Contributor Author

wookayin commented Dec 4, 2023

Preview: (Before / After)

image

@wookayin
Copy link
Contributor Author

wookayin commented Dec 4, 2023

I see a lot of style changes in src/tree_sitter/parser.h, this is probably due to the different tree-sitter CLI version used. I used 0.20.8; please let me know if this is causing any problem/conflicts.

@wookayin
Copy link
Contributor Author

@amaanq kindly bumping this up.

Problem: Inheritance from multiple parent types are not correctly
recognized, only the first parent class was parsed as identifier.

Solution: Use commaSep to recognize multiple parent types.
@amaanq amaanq force-pushed the class-inheritance branch from 0aacc64 to ef5a0bb Compare January 22, 2024 17:52
@amaanq
Copy link
Member

amaanq commented Jan 22, 2024

Thank you for the PR! And I'm sorry about being MIA

For future tree-sitter parser related PRs, make sure your cli is built from master, there's slight changes in the generation output

@amaanq amaanq merged commit 451cc56 into tree-sitter-grammars:master Jan 22, 2024
2 of 4 checks passed
@wookayin wookayin deleted the class-inheritance branch January 22, 2024 18:29
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.

2 participants