Some definitions are ignored

[ingeniarius]

I tried to generate .rb from libssh2.h (http://libssh2.org/download/libssh2-1.4.2.tar.gz)

But some definitions and macros are ignored.

Configuration for generation:

FFI::Gen.generate(
  module_name: "LibSSH2",
  ffi_lib:     "libssh2",
  headers:     ["libssh2/include/libssh2.h"],
  cflags:      `llvm-config --cflags`.split(" "),
  prefixes:    ["libssh2_", "LIBSSH2"],
  output:      "libssh2/libssh2.rb"
)

For example the following definitions were ignored:

#define LIBSSH2_CHANNEL_WINDOW_DEFAULT  (256*1024)

#define libssh2_channel_open_session(session) \
    libssh2_channel_open_ex((session), "session", sizeof("session") - 1, \
                      LIBSSH2_CHANNEL_WINDOW_DEFAULT, \
                      LIBSSH2_CHANNEL_PACKET_DEFAULT, NULL, 0)

[neelance]

Hi, thanks for using ffi-gen.

The first define should be easy to fix, I just need to parse more
arithmetic expressions. I'll look into that soon.

The second one is more difficult. Maybe ffi-gen could detect that this is
only a call to another function and generate a wrapper function for that.
But the arguments are also difficult to handle (using sizeof, etc.). So I
think the better solution is to write these wrappers by hand. Just reopen
the generated module after including it and add the methods. Are you okay
with that?

[ingeniarius]

Yes, it's ok!

But, is it possible to collect code which cannot be processed?
Thus, we have a chance to know what could be lost.

Thanks for this magic tool!

[neelance]

Good idea! I will add warnings on the console.

[neelance]

I've added support for more defines in 85abca7 and warnings for unsupported values in 7f4ef20. I think I will also add support for defines like "libssh2_channel_open_session" soon, because I noticed that they are quite common in many libraries.

[ingeniarius]

Good progress! Tnx!

[neelance]

I've added first processing of method-like defines in 329fda9. It is still quite rudimentary, but it does the job for most of libssh2's defines. I will close this issue now. Feel free to open new ones if you find problems concerning the generated methods.

[ingeniarius]

Thank you!