ffi: bug: always using ffi_prep_cif_var.

* share/txr/stdlib/ffi.c (deffi): Fix misnamed variable.
The second value coming from sys:analyze-argtypes is the number of fixed
arguments, not the number of variadic arguments. Furthermore, if this
number is equal to nargs, we were supposed to have been passing nil
instead to ffi-make-call-desc, which indicates the ordinary non-variadic
function. We were always always passing a non-nil value, so always requesting
variadic. That is fixed now thanks to the change to ffi_make_call_desc.

* ffi.c (ffi_make_call_desc): Register the function as variadic if
either nfixed is specified as nil, or if it is equal to ntotal.

* txr.1: Document the convention change for ffi-make-call-desc.

1 files changed, 6 insertions, 3 deletions

diff --git a/txr.1 b/txr.1
index f34c1b06..80d28239 100644
--- a/txr.1
+++ b/txr.1
@@ -75185,13 +75185,16 @@ of arguments in the call.
 
 If the call denotes a variadic function, the
 .meta nfixed
-argument must be an integer between 1 and
+argument must be an integer at least 1 and less than
 .metn ntotal ,
 denoting the number of fixed arguments.
 If the call denotes an ordinary, non-variadic function, then
 .meta nfixed
-must be specified as
-.codn nil .
+must either be specified specified as
+.code nil
+or else equal to the
+.meta ntotal
+argument.
 
 The
 .meta rettype