Skip to content
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

spurious generation of variadic args rather than implicit void ie () argument list #38

Closed
Laeeth opened this issue Mar 18, 2015 · 3 comments

Comments

@Laeeth
Copy link

Laeeth commented Mar 18, 2015

converting the mathgl headers

I got:
double mgl_rnd_ (...);

but it should just be:
double mgl_rnd();

hope this is not something simple I am doing wrong. sorry, but I have not time right now to find the original header and try to replicate it.

@jacob-carlborg
Copy link
Owner

I'm using quite an old version of Clang now, but I can only get this to compile in C++ mode. DStep does not support C++.

@Laeeth
Copy link
Author

Laeeth commented Mar 18, 2015

It's one of those strange cases where there is a pure C API documented in the headers but it is mixed in with C++. So dstep did an okay job, but got confused on those small cases.

Sent from my iPad

On 18 Mar 2015, at 19:55, jacob-carlborg notifications@github.com wrote:

I'm using quite an old version of Clang now, but I can only get this to compile in C++ mode. DStep does not support C++.


Reply to this email directly or view it on GitHub.

@jacob-carlborg
Copy link
Owner

I'll need to investigate how AST in libclang looks like.

ciechowoj added a commit to ciechowoj/dstep that referenced this issue Jul 29, 2016
ciechowoj added a commit to ciechowoj/dstep that referenced this issue Jul 29, 2016
This was referenced Jul 29, 2016
ciechowoj added a commit to ciechowoj/dstep that referenced this issue Aug 6, 2016
ciechowoj added a commit to ciechowoj/dstep that referenced this issue Aug 8, 2016
jacob-carlborg added a commit that referenced this issue Aug 8, 2016
Fix #38: Spurious generation of variadic args.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants