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

Errors when using non-ascii identifiers in Fennel #30

Closed
alexaandru opened this issue Sep 6, 2021 · 6 comments
Closed

Errors when using non-ascii identifiers in Fennel #30

alexaandru opened this issue Sep 6, 2021 · 6 comments
Labels
bug Something isn't working

Comments

@alexaandru
Copy link

To reproduce:

  1. given f.fnl with content (local ⧔ 42)
  2. highlighting is broken
  3. using playground, it reports:
ERROR [0, 0] - [0, 14]
      ERROR [0, 7] - [0, 13]

Fennel itself can compile the above just fine:

$ fennel --compile f.fnl 
local _e2_a7_94 = 42
return nil

so it is valid code.

@TravonteD
Copy link
Owner

Thanks for the report, I'll work up a patch to get it fixed

@TravonteD TravonteD added the bug Something isn't working label Sep 8, 2021
@alexaandru
Copy link
Author

Many thanks for your work! Cheers! :)

@TravonteD
Copy link
Owner

I just merged a pretty large re-write, with a fresh install check if you still are getting the parser error

@alexaandru
Copy link
Author

I don't know how to test this directly, I was only using it indirectly, via https://github.com/nvim-treesitter/nvim-treesitter plugin. Could you please tell me how can I compile it? Keeps complaining it cannot find <tree_sitter/parser.h>.

I did download tree-sitter itself, and updated INCS to add -I../tree-sitter/lib/include but it still won't compile.

Thank you!

@otommod
Copy link
Contributor

otommod commented Sep 11, 2021

The new parser is now used by nvim-treesitter. You can update the plugin and then :TSUpdate and test.

@alexaandru
Copy link
Author

It works, errors are all gone now! :)

Thank you! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants