The clangimport is not working well at all. I get crashes for small simple example C code. This makes me think that it's not used by any users.
I do not feel that the ast-dump based approach is a proper approach it was an ugly hack. And I fear also that making the current ast-dump approach work fully would be a lot of work.
We should use json if we want to import the clang ast. That would not be so fragile. However a rewrite for json and making it handle C and C++ fully is a really big job.
I suggest that we remove the clang import.
Last edit: Daniel Marjamäki 1 day ago
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
The clangimport is not working well at all. I get crashes for small simple example C code. This makes me think that it's not used by any users.
I do not feel that the ast-dump based approach is a proper approach it was an ugly hack. And I fear also that making the current ast-dump approach work fully would be a lot of work.
We should use json if we want to import the clang ast. That would not be so fragile. However a rewrite for json and making it handle C and C++ fully is a really big job.
I suggest that we remove the clang import.
Last edit: Daniel Marjamäki 1 day ago
Seconded. There hasn't been any activity related to this feature in years.