Search code examples
javacjava-native-interfaceswig

How do I convert to type SWIGTYPE_p_void in SWIG generated Java bindings?


I am developing some SWIG-generated Java bindings for a C library. The library contains functions that take parameters of type void *. On the C side these would typically be passed as a pointer to an array of type float or int cast to void *. In the generated Java bindings, this results in methods that take parameters of type SWIGTYPE_p_void.

What is the best way to construct an array of floats/ints in the Java bindings so that they can be passed as type SWIGTYPE_p_void to these methods?

At the moment I am defining a helper function in my example.i file:

void *floata_to_voidp(float f[])
{
    return (void *)f;
}

And then on the Java side doing something like this:

float foo[] = new float[2];
SWIGTYPE_p_void av = null;

// do something with foo

av = example.floata_to_voidp(foo);
example.myfunction(av);

This seems rather ugly, especially as I would also need an inta_to_voidp() etc in my SWIG interface file for each type conversion I want to support.

Is there a way to do this without helper functions and involving less extra code on the Java side to convert data types?

UPDATE (17/6/12): to give additional detail to the question: what I'm trying to do is take a set of C functions, with prototype int foo(const float *data, int N, const void *argv, float *result) and map them to methods on the Java side where an array of arbitrary type can be passed in as argv. Note that argv is const void * and not void *.


Solution

  • There's an alternative to this answer, it's very different and gives a more natural solution to this problem, closer to what you were looking for originally. The other suggestions were focused on adding overloads (tedious, manual) or making the array_classes implement a common interface one way or another.

    What it overlooks is that Object is a good match for void* in Java most of the time. Even arrays in Java are Objects. This means if you have SWIG map void* to Object it'll accept as inputs any arrays you might want to pass in. With a bit of care and some JNI we can then get a pointer to the start of that array to pass in to the function. Obviously we need to reject non array Objects with an exception though.

    We still end up writing some (private) helper functions to arrange extraction of the real underlying pointer and release it when done, but the nice thing about this solution is that we only have to do this once and then we end up with a typemap that can be used for any functions which take an array as void* like this.

    I ended up with the following SWIG interface for this solution:

    %module test
    
    %{
    #include <stdint.h>
    
    void foo(void *in) {
      printf("%p, %d, %g\n", in, *(jint*)in, *(jdouble*)in);
    }
    %}
    
    %typemap(in,numinputs=0) JNIEnv *env "$1 = jenv;"
    
    %javamethodmodifiers arr2voidd "private";
    %javamethodmodifiers arr2voidi "private";
    %javamethodmodifiers freearrd "private";
    %javamethodmodifiers freearri "private";
    %inline %{
    jlong arr2voidd(JNIEnv *env, jdoubleArray arr) {
      void *ptr = (*env)->GetDoubleArrayElements(env, arr, NULL);
      return (intptr_t)ptr;
    }
    
    void freearrd(JNIEnv *env, jdoubleArray arr, jlong map) {
      void *ptr = 0;
      ptr = *(void **)&map;
      (*env)->ReleaseDoubleArrayElements(env, arr, ptr, JNI_ABORT);
    }
    
    jlong arr2voidi(JNIEnv *env, jintArray arr) {
      void *ptr = (*env)->GetIntArrayElements(env, arr, NULL);
      return (intptr_t)ptr;
    }
    
    void freearri(JNIEnv *env, jintArray arr, jlong map) {
      void *ptr = 0;
      ptr = *(void **)&map;
      (*env)->ReleaseIntArrayElements(env, arr, ptr, JNI_ABORT);
    }
    %}
    
    
    %pragma(java) modulecode=%{
      private static long arrPtr(Object o) {
        if (o instanceof double[]) {
          return arr2voidd((double[])o);
        }
        else if (o instanceof int[]) {
          return arr2voidi((int[])o);
        }
        throw new IllegalArgumentException();
      }
    
      private static void freeArrPtr(Object o, long addr) {
        if (o instanceof double[]) {
          freearrd((double[])o, addr);
          return;
        }
        else if (o instanceof int[]) {
          freearri((int[])o, addr);
          return;
        }
        throw new IllegalArgumentException();
      }
    %}
    
    %typemap(jstype) void *arr "Object"
    %typemap(javain,pre="    long tmp$javainput = arrPtr($javainput);",post="      freeArrPtr($javainput, tmp$javainput);") void *arr "tmp$javainput"
    
    void foo(void *arr);
    

    This implements it for two array types, there's a small finite number and you could use fragments or macros to help with this too. Internally SWIG uses a jlong to represent pointers. So for each array type we need a function that returns a pointer for a given array and another one to release it. These are private and part of the module class - nobody other than the module needs to know how this works.

    There's then two functions which take the Object and use instanceof (ugly, but arrays in Java don't have any other common base or interface and generics don't help) and call the correct function to get/release the pointers.

    With these then it's just two typemaps to set up SWIG to use it for all void *arr arguments. The jstype typemap instructs SWIG to use Object for void* in these cases. The javain typemap arranges for a temporary local variable to hold the pointer (in a long) and then for it to be used to make the call and to be cleaned up once the call has succeed or failed.