On Jul 27, 2024, at 4:22 PM, Yuan Fu <casouri@gmail.com> wrote:



On Jul 25, 2024, at 7:05 AM, Ergus <spacibba@aol.com> wrote:



On July 25, 2024 9:42:10 AM GMT+02:00, Yuan Fu <casouri@gmail.com> wrote:


On Jul 23, 2024, at 5:38 AM, Ergus <spacibba@aol.com> wrote:

Hi:

After a while and looking that the ts modules are getting mature I was
planing to develop a `cuda-ts-mode'.

However after reading the documentation it is not totally clear to me
how to proceed in this specific case because cuda is a sort of
"extension" over C++, so I expect that the cuda-ts-mode somehow
"inherits" from c++-ts-mode.

My initial idea was to replicate how I was expecting that c++-ts-mode
should inherit from c-ts-mode, but it looks like this doesn't happen in
a clear way. The modes are threated as independent and there is a sort
of "mode" variable that is dynamically checked everywhere.

So the simple question is:

1. The approach of trying to inherit from c++-ts-mode is correct? (I mean,
is c++-ts-mode intended to be used like a "base" to inherit from?)

Some advise on this?

IIUC, tree-sitter-cuda is a separate grammar that is extended from tree-sitter-cpp, right? Then inheriting c++-ts-mode won’t work since it’ll setup everything using the cpp grammar. You’d need to delete the cpp parser and add your cuda parser (doable), and change everything that refers to ‘cpp to ‘cuda (very hacky).

I guess we can add some map which tells Emacs to use the cuda grammar in place of cpp. Kind of like treesit-load-name-override-list but more dynamic.

Once we’re able to do that, you can define a new mode that derives from c++-ts-mode and expect all the setup in c++-ts-mode to work.

Yuan

Hi Yuan:

Now I understand why c++-ts-mode does not inherit from c-ts-mode. This sort of inheritance is natural considering how tree sitter works. So I am a bit surprised that nobody faced this before. I saw some derived modes changes a while ago and I thought that it was related to this.

The extra features that the Cuda grammar adds to c++ are actually not many, but some are important to not break the indentation. 

<https://github.com/tree-sitter-grammars/tree-sitter-cuda/blob/master/grammar.js>

That's why, reimplementing the c++-ts-mode code substituting the entries seems as unnecessary and overkill to me. So I will wait for the new feature to be out before, then it will be more scalable and simple to do a Cuda-ts-mode.

Best,
Ergus

PD: Do you want me to open a bug report to keep track of this?

Yeah, that’ll be great. Thanks! I’ve implemented the feature; once you create a bug report I’ll push to master for you and others to try it out.

I have some other stuff that I want to push. So I went ahead and created bug#72388 and pushed the new feature to master. Now there’s a treesit-language-remap-alist.

See if that works for you, and we can continue discussion in the bug report :-)

Yuan