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: scanner: make state_new() static #17

Merged
merged 1 commit into from
Feb 23, 2024
Merged

fix: scanner: make state_new() static #17

merged 1 commit into from
Feb 23, 2024

Conversation

postsolar
Copy link
Owner

I suspect this is what's causing issues for multiple projects attempting to use this grammar:

Don't know why it wasn't static in the first place (in the original Haskell grammar), but I looked at what other grammars do and none export any functions other than those used by Tree-sitter.

I suspect this is what causing issues for multiple projects attempting to use this grammar:

- neurocyte/zat#1

- zed-industries/zed#7543 (comment)

Don't know why it wasn't static in the first place (in the original Haskell grammar),
but I looked at what other grammars do and none export any functions other than those used by Tree-sitter.
@postsolar postsolar merged commit 08cdbb8 into main Feb 23, 2024
@clason
Copy link

clason commented Feb 23, 2024

This seems to have broken the use of this parser on Neovim in Windows: https://github.com/nvim-treesitter/nvim-treesitter/actions/runs/8015659451/job/21898292793?pr=6179

(Unclear how exactly; I don't have access to a Windows box.)

@postsolar
Copy link
Owner Author

postsolar commented Feb 23, 2024

@clason Thank you for notifying. I don't have access to Windows either, unfortunately. Could it be related to dff8059 and 7566cbf? These seem to be more relevant to Windows.

@clason
Copy link

clason commented Feb 23, 2024

No; we don't use these.

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