João Matos

Results 370 comments of João Matos

Can you make the requested changes to merge this one?

Functions/methods AST entities provide a `CallingConvention` property, have you tried changing it for those declarations that you say are wrong? By the way, it would be good to know why...

Here is a way to do it: ```csharp class CallConvPass : TranslationUnitPass { public override bool VisitFunctionDecl(Function function) { function.CallingConvention = CallingConvention.StdCall; return base.VisitFunctionDecl(function); } } // In SetupPasses() driver.Context.TranslationUnitPasses.AddPass(new...

Try adding this to the pass: ```csharp public override bool VisitFunctionType(FunctionType function, TypeQualifiers quals) { function.CallingConvention = CallingConvention.StdCall; return base.VisitFunctionType(function, quals); } ```

Your best bet at this point is just to hardcode StdCall convention in the generator. Either that or try to get us a test case for your issue that we...

I don't think the problem is on the defines being @ddobrev, at least there is nothing so far that leads me to think that. I would say the issue is...

Parser triple is what controls which target architecture the parser is using. https://github.com/mono/CppSharp/blob/2a53e079aaecc2611b583d6e1dfccf3d467c6ac4/src/CLI/Generator.cs#L44

Hey @limenote135. I think we merged some VS fixes a little while ago. Could you check if there is still an issue with latest revisions?

@ddobrev We should definitely add a new CI test configuration to keep this tested. As for the testing assertion errors, perhaps the best way to fix them would be add...