2

How can I convert this Clarion procedure declaration to C#? It's part of a 3rd Party DLL written in C, which doesn't have much documentation. I have listed the prototype for the method in Clarion that is working correctly. In C#, I'm not sure what type to use to replace *CString. I tried char[] like @DanielC suggested, but it didn't work. I also found that Clarion long is 32-bit (thanks @shf301).

Clarion:

SendRequest Procedure(*CString xData,Long DataLen,Long xTimeout),Byte,Virtual

C# (what I've tried, which doesn't work):

[DllImport("3RD_PARTY_API.dll")]
private static extern long SendRequest(ref string xData, int DataLen, int xTimeout);

When I call the SendRequest method in C# I get the standard PInvokeStackImbalance was detected error from VS2010. I think it's an issue of the parameter types and not something like CharSet or EntryPoint in the DllImport declaration. I'm really just stuck on how to convert *CString to a valid C# type.

Community
  • 1
  • 1
DanM7
  • 2,203
  • 3
  • 28
  • 46

4 Answers4

6

I found the answer. I wish I still worked with the developer who wrote this code so I could yell at them. They put a wrapper around the API so that the method I listed with 3 parameters then calls the actual API without the timeout value. The PInvokeStackImbalance exception came from the fact that I was adding in an extra parameter.

Here is the actual Clarion procedure as well as the correct C# syntax for this:

Clarion: 

SendRequest Procedure(*CString xData, Long DataLen), Byte, Virtual

C#:

[DllImport("NOVA_API.dll", EntryPoint = "SendRequest")]
private static extern byte SendRequest(string xData, int DataLen);

In summation, the Clarion *CString can be converted to the .NET string type in DLLImport declarations.

Many thanks to those who responded. Here is an additional resource I used in my search, which I found very usefull: pinvoke.net

DanM7
  • 2,203
  • 3
  • 28
  • 46
4

In C# a long is always 64 bits. The Long in Clarion is 32 bits. Change the long parameters in your p/Invoke declaration to int's.

Jesse Jashinsky
  • 10,313
  • 6
  • 38
  • 63
shf301
  • 31,086
  • 2
  • 52
  • 86
  • Good tip. Checked the help and you were right. `LONG: A four-byte signed integer.` Didn't solve the issue, but I'm one step closer... – DanM7 Oct 30 '12 at 13:49
2

According to wikipedia clarion can uses some weird TopSpeed "double fast call" where they use 4 reg for parameters instead of two. If code is compiled using this calling convention it will be impossible to call directly from C#.

TopSpeed / Clarion / JPI The first four integer parameters are passed in registers eax, ebx, ecx and edx. Floating point parameters are passed on the floating point stack – registers st0, st1, st2, st3, st4, st5 and st6. Structure parameters are always passed on the stack. Additional parameters are passed on the stack after registers are exhausted. Integer values are returned in eax, pointers in edx and floating point types in st0.

If this is not the case then probably one of your parameters is not the right size otherwise you would be getting an AV instead of corrupt stack error.

Can also try setting the calling convention on the dllimport. In addition to "TOPSPEED" clarion also supports stdcall and cdecl but "TOPSPEED" is the default. Below is a link to a tutorial on interop with vb6 and clarion.

http://pisoft.ru/verstak/insider/cw_vb.htm

AbdElRaheim
  • 1,384
  • 6
  • 8
  • I found out the DLL is written in C, but I appreciate the research. I'll keep that in mind for the future. I've clarified this in my question now: the DLL is written in C, the prototype in Clarion works, but the DLLImport in C# does not work. – DanM7 Oct 30 '12 at 14:10
  • Try setting `CallingConvention` on the `DllImport` attribute to `CallingConvention.Cdecl` – shf301 Oct 30 '12 at 15:50
1

Have you tried char[]? I am not sure that it would work but it is worth a shot. (I'd post this as a comment but I don't have the rights)

techvice
  • 1,315
  • 1
  • 12
  • 24
  • I tried this, but it didn't work. Thanks for the suggestion. I've updated my question accordingly. – DanM7 Oct 30 '12 at 14:02