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

Add Microsoft SEH extension to the grammar #164

Merged
merged 1 commit into from
Oct 16, 2023

Conversation

DennySun2100
Copy link
Contributor

This change is to make c/c++ parser be able to parse Microsoft SEH extension - __try/__except/__finally/__leave. Please refer to "SEH exception handling" in test/corpus/microsoft.txt for the test on this change.

@DennySun2100 DennySun2100 reopened this Aug 21, 2023
@DennySun2100 DennySun2100 changed the title Dev/dennys/merge changes denny/merge changes Aug 21, 2023
@DennySun2100 DennySun2100 changed the title denny/merge changes Add Microsoft SEH extension to the grammar Aug 21, 2023
Copy link
Contributor Author

@DennySun2100 DennySun2100 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@amaanq Could you help look at this change, it is adding Microsoft SEH extension.

@amaanq
Copy link
Member

amaanq commented Oct 16, 2023

Alright I collapsed it into one commit and tidied up the test, let me know if that looks good @DennySun2020

@DennySun2100
Copy link
Contributor Author

e know if that lo
Thanks Amaan, that looks good to me, should I close this PR now? Looks like I have no write access to merge this change to main branch?

@amaanq
Copy link
Member

amaanq commented Oct 16, 2023

no worries, was just letting you check it out then I'l merge

@amaanq amaanq merged commit 25371f9 into tree-sitter:master Oct 16, 2023
9 checks passed
@DennySun2100 DennySun2100 deleted the dev/dennys/merge_changes branch October 16, 2023 07:47
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.

3 participants