X-Git-Url: http://jsfdemo.indexdata.com/?a=blobdiff_plain;ds=inline;f=ZOOM.xs;h=27721396efdeff9b30aa7c38e79532413cea54a0;hb=33ab99ed17f92d8c84e580f3708b9d6d27fa1bc7;hp=27b0bdd04866b6be1b21086be0e5b6220f2d2524;hpb=508ddb3e82081c0fc4a6077d137a5003c12f2e84;p=ZOOM-Perl-moved-to-github.git diff --git a/ZOOM.xs b/ZOOM.xs index 27b0bdd..2772139 100644 --- a/ZOOM.xs +++ b/ZOOM.xs @@ -1,10 +1,11 @@ -/* $Id: ZOOM.xs,v 1.14 2005-10-17 13:44:52 mike Exp $ */ +/* $Id: ZOOM.xs,v 1.24 2005-11-04 17:07:13 mike Exp $ */ #include "EXTERN.h" #include "perl.h" #include "XSUB.h" #include +#include /* Used by the *_setl() functions */ typedef char opaquechar; @@ -15,6 +16,64 @@ struct datachunk { 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 @@ -53,12 +112,14 @@ ZOOM_connection_option_get(c, key) 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 @@ -149,16 +210,11 @@ const char * ZOOM_diag_str(error) int error -# UNTESTED -int -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 @@ -171,31 +227,51 @@ void 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 @@ -203,21 +279,28 @@ ZOOM_resultset_record(s, pos) 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 + +# TESTED +void +ZOOM_resultset_sort(r, sort_type, sort_spec) + ZOOM_resultset r + const char* sort_type + const char* sort_spec # 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 @@ -237,28 +320,28 @@ ZOOM_record 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 @@ -339,88 +422,109 @@ ZOOM_package_option_set(p, key, val) const char * key const char * val -# UNTESTED +# 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_resultset_sort(r, sort_type, sort_spec) - ZOOM_resultset r - 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 +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 @@ -428,3 +532,8 @@ ZOOM_event(no, cs) int no ZOOM_connection * cs +# UNTESTED +int +ZOOM_connection_last_event(cs) + ZOOM_connection cs +