+ Répondre à la discussion
Affichage des résultats 1 à 2 sur 2
  1. #1
    Expert Confirmé
    Avatar de ®om
    Inscrit en
    janvier 2005
    Messages
    2 817
    Détails du profil
    Informations forums :
    Inscription : janvier 2005
    Messages : 2 817
    Points : 3 099
    Points
    3 099

    Par défaut JNI : références locales réutilisables ou non ?

    Dans ce tuto sur JNI (et dans bien d'autres), il est écrit clairement qu'on ne doit pas garder une référence locale pour une réutilisation future :
    By default, JNI creates local references to ensure that they are liable for garbage collection. Because of this, you may unintentionally write illegal code by trying to store away a local reference so that you can reuse it later, as shown in the code sample below:
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    11
    12
    /* This code is invalid! */
    static jmethodID mid;
     
    JNIEXPORT jstring JNICALL
    Java_Sample1_accessMethod(JNIEnv *env, jobject obj)
    {
       ...
       cls = (*env)->GetObjectClass(env, obj);
       if (cls != 0)
          mid = (*env)->GetStaticMethodID(env, cls, "addInt", "(I)I");
       ...
    }
    Maintenant, je regarde les sources de java.net.PlainDatagramSocketImpl en Java, et je veux connaître l'implémentation native de :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    protected synchronized native void receive0(DatagramPacket p) throws IOException;
    Je regarde ce qui pourrait correspondre :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    $ grep -rl receive0 | grep '\.c$'
    jdk/src/windows/native/java/net/TwoStacksPlainDatagramSocketImpl.c
    jdk/src/windows/native/sun/nio/ch/DatagramChannelImpl.c
    jdk/src/solaris/native/java/net/PlainDatagramSocketImpl.c
    jdk/src/solaris/native/sun/nio/ch/DatagramChannelImpl.c
    Je trouve donc jdk/src/solaris/native/java/net/PlainDatagramSocketImpl.c.
    Dedans, il y a plein de jfieldID en tant que variables externes static. Par exemple :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    static jfieldID IO_fd_fdID;
     
    static jfieldID pdsi_fdID;
    static jfieldID pdsi_timeoutID;
    static jfieldID pdsi_trafficClassID;
    static jfieldID pdsi_localPortID;
    static jfieldID pdsi_connected;
    static jfieldID pdsi_connectedAddress;
    static jfieldID pdsi_connectedPort;

    Le début de l'implémentation du constructeur :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    11
    12
    13
    14
    15
    16
    /*
     * Class:     java_net_PlainDatagramSocketImpl
     * Method:    init
     * Signature: ()V
     */
    JNIEXPORT void JNICALL
    Java_java_net_PlainDatagramSocketImpl_init(JNIEnv *env, jclass cls) {
     
    #ifdef __linux__
        struct utsname sysinfo;
    #endif
        char *s;
        pdsi_fdID = (*env)->GetFieldID(env, cls, "fd",
                                       "Ljava/io/FileDescriptor;");
        CHECK_NULL(pdsi_fdID);
        pdsi_timeoutID = (*env)->GetFieldID(env, cls, "timeout", "I");
    Et voici le début de l'implémentation de la méthode receive0 :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    11
    12
    13
    14
    /*
     * Class:     java_net_PlainDatagramSocketImpl
     * Method:    receive
     * Signature: (Ljava/net/DatagramPacket;)V
     */
    JNIEXPORT void JNICALL
    Java_java_net_PlainDatagramSocketImpl_receive0(JNIEnv *env, jobject this,
                                                  jobject packet) {
     
        char BUF[MAX_BUFFER_LEN];
        char *fullPacket = NULL;
        int mallocedPacket = JNI_FALSE;
        jobject fdObj = (*env)->GetObjectField(env, this, pdsi_fdID);
        jint timeout = (*env)->GetIntField(env, this, pdsi_timeoutID);
    Précisément ce qu'il ne faut pas faire, non ?

    Qui a raison ? Le tuto ou l'implémentation des bibliothèques standard ?

    Merci de votre aide.

  2. #2
    Expert Confirmé
    Avatar de ®om
    Inscrit en
    janvier 2005
    Messages
    2 817
    Détails du profil
    Informations forums :
    Inscription : janvier 2005
    Messages : 2 817
    Points : 3 099
    Points
    3 099

    Par défaut

    Apparemment, c'est le tuto qui se trompe :

    Note that jfieldIDs and jmethodIDs are opaque types, not object references, and should not be passed to NewGlobalRef. The raw data pointers returned by functions like GetStringUTFChars and GetByteArrayElements are also not objects. (They may be passed between threads, and are valid until the matching Release call.)
    http://developer.android.com/trainin...bal_references

+ Répondre à la discussion
Cette discussion est résolue.