Inject code to process `#define`? about nimterop HOT 8 OPEN

I forked the repo and added in nnkTemplateDef to findOverrides in cimport.nim and added an extra case where findOverrides is used so nnkTemplateDef maps to nskTemplate. And modified plugin.nim so the nskTemplate exists in cOverrides. But I don't see it in my output file.

from nimterop.

I now see I have to modify ast2.nim to handle my use case.

from nimterop.

You don't need to cOverride - can just define it in your nim file per usual. nimterpo will skip it anyway since it is unsupported and won't conflict with your definition.

from nimterop.

I don't follow. If I just define TM_VERSION in my generator it doesn't appear in my output, and tm_api_registory_api_version still isn't defined in the output.

from nimterop.

cOverride only supports const, type and proc declarations which is why the template info is not propagated. Maybe if you make it a proc, it will work.

from nimterop.

Thanks, I switched the cOverride for TM_VERSION to a proc, and I see it in the output now. That gets me a little closer to what I want, but I still want exprparser.nim's processTSNode to handle call_expression so I can get the proper output for #defines. I want to be able to automatically wrap

#define tm_api_registry_api_version TM_VERSION(0, 3, 0)

And I'll probably have to work on a modification to allow templates in cOverride because The Machinery has lots of macros for example:

#define tm_get_api(reg, TYPE) \
    (struct TYPE *)reg->get(#TYPE, TYPE##_version

nimterop ignores these because of the call_expression, and templates would be the only way to support the token pasting operator ## and astToStr for token stringification.

from nimterop.

Although I'm not maintaining nimterop anymore, I'm totally open to PRs. However, the CI is not longer functional so if you are willing to take this on, hopefully you can help with the CI as well.

from nimterop.

I'm not sure I can take that on. I'm not familiar with the CI at all. I mean I only started looking at nimble because of nimterop. :) I was trying to upgrade treesitter to v0.20.0, but it wasn't as simple as swapping the branch. Where would I even begin with CI?

from nimterop.