use of `define-foreign-record-type' expands into incorrect result type identifier

[felix winkelmann]

Shawn Rutledge reported this problem:

(module foo *

(import scheme chicken foreign)
(use foreigners)

(define-foreign-record-type (vtable "struct DBusObjectPathVTable")
  (c-pointer dbus_internal_pad4 vtable-dbus_internal_pad4 vtable-dbus_internal_pad4-set!))

)

gives:

Error: illegal foreign return type `#%void'

I assume that there is some bug in either the foreigners extension or the interplay of define-foreign-record-type
and the core FFI is wrong, or that the compiler does not
strip-syntax a type-identifier somewhere.

[felix winkelmann]

This is a chicken ffi/expander bug it seems as it can be reproduced by other means.

[Jim Ursetto]

I fixed this a few days ago in r14310 foreigners/trunk, but I didn't tag a new release. It's tagged now, try again.

It happens because you may not provide renamed type identifiers to foreign-lambda*, and 'void' was being renamed. I changed it not to rename. Now if this was actually supposed to work, then I will change foreigners back once it is fixed.

[felix winkelmann]

Should work with r14447.