Discussion :

[ToTo13]

Bonjour,

j'ai une question sur l'utilisation de Release<type>ArrayElements. Je sais qu'il faut l'appeler afin de faire un "commit" des modifications faites sur un tableau.
Mais j'ai dans certaines discussions (pas sur toutes), qu'il fallait toujours l'appeler, même si un tableau n'avait pas été modifié.

Dans ce petit code, j'aurai tendance à ne l'appeler que pour le tableau de double, car il est modifié. Je ne l'appellerai pas pour le tableau d'int, car ce n'est qu'un pointeur local qui devrait donc être supprimé à la sortie de la fonction.

Code C++ :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
JNIEXPORT jobject JNICALL Java_measures_miniball_Miniball_Compute2D (JNIEnv *env, jobject obj, jintArray Xarray, jdoubleArray Infos)
	{
	int length = env->GetArrayLength(Xarray) ;
	int* X = env->GetIntArrayElements(Xarray, 0) ; // On récupère un pointeur sur le tableau.
 
	double* infos = env->GetDoubleArrayElements(Infos, 0) ; // On récupère un pointeur sur le tableau.
	// Je ne modifie que le tableau de double
	infos[0] = 0.0 ;
	infos[1] = 1.0 ;
	env->ReleaseDoubleArrayElements(Infos, infos, 0) ; // Le commit...
	return NULL ;
	}

Ai je raison ? (Ca m'ennuierait d'avoir des fuites mémoire :p)
Merci par avance.

[tchize_]

D'après ce que je lit de la doc, je dirais que tu devrais l'appeler systématiquement. On y dit que le get peut retourner une copie du contenu du tableau. Même si tu ne modifie pas, sans signaler à la jvm que tu as fini, comment ces données peuvent être liberees.

[ToTo13]

D'après ce que je lit de la doc, je dirais que tu devrais l'appeler systématiquement.

Merci.

On y dit que le get peut retourner une copie du contenu du tableau.

Voilà le genre d'imprécision qui fait peur !!!
Je m'attendais à avoir quelque chose de fixe copie ou pointeur, mais pas un ou peut être l'autre !!!
D'autant plus que la copie fait perdre du temps.
Cela peut me poser de gros soucis, car je travaille sur des images (potentiellement grandes). Avoir un pointeur serait parfait, une recopie de mon image serait alors catastrophique au niveau performances.

Et je suppose qu'il n'y a aucun moyen pour savoir de manière certaine si on a un pointeur ou une copie... ?

[tchize_]

cf la doc:

http://docs.oracle.com/javase/1.5.0/...s.html#wp17314

NativeType *Get<PrimitiveType>ArrayElements(JNIEnv *env,
ArrayType array, jboolean *isCopy);
.....
If isCopy is not NULL, then *isCopy is set to JNI_TRUE if a copy is made; or it is set to JNI_FALSE if no copy is made.

isCopy contienra true ou false suivant que tu as obtenu une copie ou non.

L'idée quand on accélère des codes avec du JNI, c'est de ne pas interagir au maximum avec java: ton image doit être dans la partie native. Seulement à la toute fin du traitement tu la transforme en un tableau java. Chaque fois que tu utilie un point JNI / java, tu es soumis à différentes containtes de la JVM qui peuvent impacter tes performances.

[ToTo13]

Pourtant un peu plus loin on trouve :

RETURNS:
Returns a pointer to the array elements, or NULL if the operation fails.

Mais encore plus loin on retrouve pour "Release<PrimitiveType>ArrayElements Routines"

The elems argument is a pointer derived from array using the corresponding Get<PrimitiveType>ArrayElements() function. If necessary, this function copies back all changes made to elems to the original array.

Bref... pas très clair.
Par contre c'est la documentation pour Java 1.5 et il semblerait qu'elle n'ait plus évolué depuis. Plus d'évolution de JNI depuis ?

[ToTo13]

J'ai essayé de modifier mon code pour savoir si une copie est faite ou non:

Code c++ :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
	jboolean *j ;
	int length = env->GetArrayLength(Xarray) ;
	int* X = env->GetIntArrayElements(Xarray, j) ;
	if ( *j == JNI_TRUE ) printf("Oops copy\n") ;
	else printf("OK\n") ;

Ca compile, mais j'ai une magnifique "fatal error".
Par contre si je ne fais pas le "printf" (donc juste "if ( *j == JNI_TRUE ) ;") tout se passe correctement. Mais je peux quand même faire un printf dans mon code avant ou après sans erreur.
Là... je suis totalement perdu !!!