Publicité
+ 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 811
    Détails du profil
    Informations forums :
    Inscription : janvier 2005
    Messages : 2 811
    Points : 3 091
    Points
    3 091

    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 :
    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 :
    protected synchronized native void receive0(DatagramPacket p) throws IOException;
    Je regarde ce qui pourrait correspondre :
    Code :
    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 :
    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 :
    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 :
    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 811
    Détails du profil
    Informations forums :
    Inscription : janvier 2005
    Messages : 2 811
    Points : 3 091
    Points
    3 091

    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.

Liens sociaux

Règles de messages

  • Vous ne pouvez pas créer de nouvelles discussions
  • Vous ne pouvez pas envoyer des réponses
  • Vous ne pouvez pas envoyer des pièces jointes
  • Vous ne pouvez pas modifier vos messages
  •