search

stadelmanma / tree-sitter-fortran

Fortran grammar for tree-sitter
MIT License
30 stars 15 forks source link

feat: rewrite the scanner in C #83

Closed amaanq closed 1 year ago

stadelmanma commented 1 year ago

Hi @amaanq thanks! Can you tell me a bit more about why the changes were needed?

ZedThree commented 1 year ago

Looking at other repos where they've made a similar change, it's for projects/environments where only a C compiler is available. The official tree-sitter-cpp also had a similar PR, so I think it's probably worth doing here too -- after all, half the point of tree-sitter is that it only needs a C compiler and no other dependencies.

amaanq commented 1 year ago

pretty much what @ZedThree said, so projects using a pure C tool chain can leverage this and it makes it easier to integrate into wasm

stadelmanma commented 1 year ago

Sounds good to me, can’t argue with simplifying the necessary toolchain.