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

feat: rewrite the scanner in C #221

Merged
merged 2 commits into from
Jun 23, 2023

Conversation

amaanq
Copy link
Member

@amaanq amaanq commented Jun 17, 2023

Closes #206
Closes #207
Pending #220

libFuzzer is happy as well

@clason
Copy link

clason commented Jun 17, 2023

@tausbn Having a C++ scanner prevents Neovim from bundling this parser since we only require a pure C toolchain. This (plus a release) would unblock this, so Neovim users can have tree-sitter highlighting for Python out-of-the box (which we already have for C).

@amaanq amaanq force-pushed the rewrite-it-in-c branch from 96d05b8 to 88519d1 Compare June 20, 2023 19:32
@maxbrunsfeld
Copy link
Contributor

Thanks!

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.

Deadlock/memory leak/crash when parsing a specific file Migrate scanner to C
4 participants