-/* $Id: ZOOM.xs,v 1.14 2005-10-17 13:44:52 mike Exp $ */
+/* $Id: ZOOM.xs,v 1.22 2005-11-03 16:23:36 mike Exp $ */
#include "EXTERN.h"
#include "perl.h"
#include "XSUB.h"
#include <yaz/zoom.h>
+#include <yaz/xmalloc.h>
/* Used by the *_setl() functions */
typedef char opaquechar;
int len;
};
+/* Used to package Perl function-pointer and user-data together */
+struct callback_block {
+ SV *function;
+ SV *handle;
+};
+
+/* The callback function used for ZOOM_options_set_callback(). I do
+ * not claim to fully understand all the stack-hacking magic, and less
+ * still the reference-counting/mortality stuff. Accordingly, the
+ * memory management here is best characterised as What I Could Get To
+ * Work, More Or Less.
+ */
+const char *__ZOOM_option_callback (void *handle, const char *name)
+{
+ struct callback_block *cb = (struct callback_block*) handle;
+ int count;
+ SV *ret;
+ char *s;
+ char *res;
+
+ dSP;
+
+ ENTER;
+ SAVETMPS;
+
+ PUSHMARK(SP);
+ XPUSHs(cb->handle);
+ XPUSHs(sv_2mortal(newSVpv(name, 0)));
+ PUTBACK;
+ /* Perl_sv_dump(0, cb->function); */
+
+ count = call_sv(cb->function, G_SCALAR);
+
+ SPAGAIN;
+
+ if (count != 1)
+ croak("callback function for ZOOM_options_get() returned %d values: should have returned exactly one", count);
+
+ ret = POPs;
+ if (SvPOK(ret)) {
+ s = SvPV_nolen(ret);
+ /* ### `res' never gets freed! I think it is
+ * impossible to solve this problem "correctly"
+ * because the ZOOM-C option callback interface is
+ * inadequate. */
+ res = xstrdup(s);
+ } else {
+ res = 0;
+ }
+
+ PUTBACK;
+ FREETMPS;
+ LEAVE;
+
+ return res;
+}
+
+
MODULE = Net::Z3950::ZOOM PACKAGE = Net::Z3950::ZOOM PREFIX=ZOOM_
PROTOTYPES: ENABLE
struct datachunk
ZOOM_connection_option_getl(c, key, len)
ZOOM_connection c
- const char *key
+ const char* key
int &len
CODE:
- RETVAL.data = (char*) ZOOM_connection_option_getl(c, key, &RETVAL.len);
+ RETVAL.data = (char*) ZOOM_connection_option_getl(c, key, &len);
+ RETVAL.len = len;
OUTPUT:
RETVAL
+ len
# TESTED
void
ZOOM_connection_last_event(cs)
ZOOM_connection cs
-# UNTESTED
+# TESTED
ZOOM_resultset
ZOOM_connection_search(arg0, q)
ZOOM_connection arg0
- ZOOM_query q
+ ZOOM_query q
# TESTED
ZOOM_resultset
ZOOM_resultset_destroy(r)
ZOOM_resultset r
-# UNTESTED
+# TESTED
const char *
ZOOM_resultset_option_get(r, key)
- ZOOM_resultset r
- const char * key
+ ZOOM_resultset r
+ const char* key
-# UNTESTED
+# TESTED
void
ZOOM_resultset_option_set(r, key, val)
- ZOOM_resultset r
- const char * key
- const char * val
+ ZOOM_resultset r
+ const char* key
+ const char* val
# TESTED
size_t
ZOOM_resultset_size(r)
ZOOM_resultset r
-# UNTESTED
-void
-ZOOM_resultset_records(r, recs, start, count)
- ZOOM_resultset r
- ZOOM_record * recs
- size_t start
- size_t count
+# TESTED
+SV *
+ZOOM_resultset_records(r, start, count, return_records)
+ ZOOM_resultset r
+ size_t start
+ size_t count
+ int return_records
+ INIT:
+ ZOOM_record *recs = 0;
+ CODE:
+ if (return_records)
+ recs = (ZOOM_record*) xmalloc(count * sizeof *recs);
+ ZOOM_resultset_records(r, recs, start, count);
+ if (return_records) {
+ AV *av = newAV();
+ int i;
+ for (i = 0; i < count; i++) {
+ SV *tmp = newSV(0);
+ sv_setref_pv(tmp, "ZOOM_record", (void*) recs[i]);
+ av_push(av, tmp);
+ }
+ RETVAL = newRV((SV*) av);
+ } else {
+ RETVAL = &PL_sv_undef;
+ }
+ OUTPUT:
+ RETVAL
# TESTED
ZOOM_record
ZOOM_resultset s
size_t pos
-# UNTESTED
+# TESTED
ZOOM_record
ZOOM_resultset_record_immediate(s, pos)
- ZOOM_resultset s
- size_t pos
+ ZOOM_resultset s
+ size_t pos
-# UNTESTED
+# TESTED
void
ZOOM_resultset_cache_reset(r)
- ZOOM_resultset r
+ ZOOM_resultset r
# See "typemap" for discussion of the "const char *" return-type.
#
# TESTED
-### but should use datachunk
+### but should use datachunk for in some (not all!) cases.
const char *
ZOOM_record_get(rec, type, len)
ZOOM_record rec
ZOOM_record_clone(srec)
ZOOM_record srec
-# UNTESTED
+# TESTED
ZOOM_query
ZOOM_query_create()
-# UNTESTED
+# TESTED
void
ZOOM_query_destroy(s)
- ZOOM_query s
+ ZOOM_query s
-# UNTESTED
+# TESTED
int
ZOOM_query_cql(s, str)
- ZOOM_query s
- const char * str
+ ZOOM_query s
+ const char* str
-# UNTESTED
+# TESTED
int
ZOOM_query_prefix(s, str)
- ZOOM_query s
- const char * str
+ ZOOM_query s
+ const char* str
-# UNTESTED
+# TESTED
int
ZOOM_query_sortby(s, criteria)
ZOOM_query s
const char * sort_type
const char * sort_spec
-# UNTESTED
-ZOOM_options_callback
-ZOOM_options_set_callback(opt, c, handle)
- ZOOM_options opt
- ZOOM_options_callback c
- void * handle
+# We ignore the return value of ZOOM_options_set_callback(), since it
+# is always just the address of the __ZOOM_option_callback() function.
+# The information that we actually want -- the address of the Perl
+# function in the callback_block -- is unavailable to us, as the
+# underlying C function doesn't give the block back.
+#
+# TESTED
+void
+ZOOM_options_set_callback(opt, function, handle)
+ ZOOM_options opt
+ SV* function;
+ SV* handle;
+ CODE:
+ /* The tiny amount of memory allocated here is never
+ * released, as options_destroy() doesn't do anything
+ * to the callback information. Not a big deal.
+ * Also, I have no idea how to drive the Perl "mortal"
+ * reference-counting stuff, so I am just allocating
+ * copies which also never get released. Don't sue!
+ */
+ struct callback_block *block = (struct callback_block*)
+ xmalloc(sizeof *block);
+ block->function = function;
+ block->handle = handle;
+ SvREFCNT(block->function);
+ SvREFCNT(block->handle);
+ ZOOM_options_set_callback(opt, __ZOOM_option_callback,
+ (void*) block);
# TESTED
ZOOM_options
ZOOM_options_create()
-# UNTESTED
+# TESTED
ZOOM_options
ZOOM_options_create_with_parent(parent)
- ZOOM_options parent
+ ZOOM_options parent
-# UNTESTED
+# TESTED
ZOOM_options
ZOOM_options_create_with_parent2(parent1, parent2)
- ZOOM_options parent1
- ZOOM_options parent2
+ ZOOM_options parent1
+ ZOOM_options parent2
-# UNTESTED
+# TESTED
const char *
ZOOM_options_get(opt, name)
- ZOOM_options opt
- const char * name
+ ZOOM_options opt
+ const char* name
-# UNTESTED
-const char *
+# TESTED
+struct datachunk
ZOOM_options_getl(opt, name, len)
- ZOOM_options opt
- const char * name
- int &len
+ ZOOM_options opt
+ const char* name
+ int &len
+ CODE:
+ RETVAL.data = (char*) ZOOM_options_getl(opt, name, &len);
+ RETVAL.len = len;
+ OUTPUT:
+ RETVAL
+ len
-# UNTESTED
+# TESTED
void
ZOOM_options_set(opt, name, v)
- ZOOM_options opt
- const char * name
- const char * v
+ ZOOM_options opt
+ const char* name
+ const char* v
-# UNTESTED
+# TESTED
void
ZOOM_options_setl(opt, name, value, len)
- ZOOM_options opt
- const char * name
- const char * value
- int len
+ ZOOM_options opt
+ const char* name
+ opaquechar* value
+ int len
-# UNTESTED
+# TESTED
void
ZOOM_options_destroy(opt)
- ZOOM_options opt
+ ZOOM_options opt
-# UNTESTED
+# TESTED
int
ZOOM_options_get_bool(opt, name, defa)
- ZOOM_options opt
- const char * name
- int defa
+ ZOOM_options opt
+ const char* name
+ int defa
-# UNTESTED
+# TESTED
int
ZOOM_options_get_int(opt, name, defa)
- ZOOM_options opt
- const char * name
- int defa
+ ZOOM_options opt
+ const char* name
+ int defa
-# UNTESTED
+# TESTED
void
ZOOM_options_set_int(opt, name, value)
- ZOOM_options opt
- const char * name
- int value
+ ZOOM_options opt
+ const char* name
+ int value
# UNTESTED
int