-
Notifications
You must be signed in to change notification settings - Fork 590
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
Add support variadic templates #81
Comments
That's quite a complicated feature, but fortunately it doesn't look like even LLVM makes heavy use of it. Implementing something that assumes that only the last argument can be variadic should be good enough? |
I feel like I've created a lot of work for you over the last two weeks. I apologise, but appreciate the work you've done :) I think that will work -- only having the last argument as variadic... |
No problem, the other things didn't take long to fix. This might take a bit more time though... Anyway, I'm glad to have someone looking at the C++ API of LLVM :) Thank you for that! |
It just clicked today, but I figured we might not really need to support variadic templates for the current parser. The next big thing I'd like to try is to use Clang as parser and see how far we can go with this, see #51. To get there though, we first need access to the C++ API of Clang, so I've fixed the crash you found in the current parser. It's basically ignoring the templates, but it's also ignoring normal variadic functions, so nothing new here. Anyway, please let me know if you encounter any more crashes or problems with LLVM. We might not need the whole API for the portion of it that could be useful as a better parser for JavaCPP. :) And thanks for having reported the issues you found up until now! |
Add support for variadic templates
eg.
or
Currently this throws an NPE:
The text was updated successfully, but these errors were encountered: