Discussion :

[kolodz]

Bonjour,

Sur du code legacy, il y a une manipulation de pointeur qui se trouve être particulièrement moche :

Code :

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

1
2
3
4
5
6
7
// Enregistrement du pointeur sous forme de chaine de caractère avec d'autres choses  avant
sprintf(sDescReq,"%s*%i",_sName,(int)Desc);
//Récupération de la partie pointeur de la chaine de caractère
// Note : Toujours pas compris l'utilité du +1 ici
sTempo = strrchr(sDescReq,'*')+1;
// Conversion barbare de la chaine de caractère en pointeur de truc.
Desc = (SQLDA *)atoi(sTempo);

Dans un optique de passage en 64 bits, il y a eu une correction rapide apporté et test sur une machine AIX :

Code :

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

1
2
3
4
5
6
7
// Enregistrement du pointeur sous forme de chaine de caractère avec d'autres choses  avant
sprintf(sDescReq,"%s*%L",_sName,(long)Desc);
//Récupération de la partie pointeur de la chaine de caractère
// Note : Toujours pas compris l'utilité du +1 ici
sTempo = strrchr(sDescReq,'*')+1;
// Conversion barbare de la chaine de caractère en pointeur de truc.
Desc = (SQLDA *)atol(sTempo);

Il se trouve que ce version ne fonctionne pas sous Windows. Car, le %L de rajoute rien dans la chaine de caractère.
Note : Je suppose que %L n'est pas reconnu par le compilateur Windows (32 bits) de la même manière que le compilateur AIX (32bits ou 64 bits)

J'ai fait un peu de recherche et je suis tombé sur une solution qui fonction pour Windows 32 bits qui utilise %lld Cependant, cela ne fonctionne pas sous AIX
Il m'est totalement possible de faire un #ifdef en fonction de l'OS ciblé, mais pas envie !

Je sais que l'ensemble est basé sur un principe à la *** et que l'implémentation n'est de toute façon pas valable. Je suppose que le plus logique serai d'utiliser %p. Mais, cela transcrit le pointeur en hex et je ne sais pas comment reconvertir cela en un pointeur propre de manière simple.

Donc, si vous avez une manière de rendre ce stockage d'adresse de pointeur plus propre et compatible AIX/Windows 32/64 bits, je prends !

Cordialement,
Patrick Kolodziejczyk.

[picodev]

Bonjour,
tes compilateurs sont-ils tous les deux au moins en C99 ? si oui, ne caste plus les pointeurs en int (de toute façon sous windows int=long en 32 et 64bit), mais utilises %p dans le sprintf. Pour récupérer les valeurs dans l'autre sens fait un sscanf. Ce devrait être suffisamment portable

[kolodz]

Je suis en C90 sur la machine Windows. Je suppose que je vais garder le cast...

[picodev]

Attention sous windows tu as sizeof(int)=sizeof(long)=4 en 32 et 64 bits, or en 64bits sizeof(void *)=8 ...

[gangsoleil]

Hello,

Question bête, mais as-tu essayé avec %ld ?

Sinon, le +1 s'explique assez simplement : le strrchr recherche le caractère '*' dans la chaîne, et pointe dessus. Le +1 permet d'avancer d'un caractère, c'est à dire de passer l'étoile.

Par contre, est-ce que Desc peut contenir un caractère '*' ou pas ? Parce que si la réponse est non, je ne vois vraiment pas l'intérêt du code...

Et pour être tatillon : tu dis être en C90, mais dans ce cas les commentaires doivent être en /* */

[kolodz]

Attention sous windows tu as sizeof(int)=sizeof(long)=4 en 32 et 64 bits, or en 64bits sizeof(void *)=8 ...

Non non,
le long en 64 bits fait une taille de 8 de même que le pointeur. D'ailleurs, quand tu as les options de compilations pour le 64 bits, tu as des warning quand tu converti un long en int, car tu perds de l'information.

Question bête, mais as-tu essayé avec %ld ?

Pas encore, je vérifie ça ! Mais, j'avoue que je n'y crois pas des masses.

Sinon, le +1 s'explique assez simplement : le strrchr recherche le caractère '*' dans la chaîne, et pointe dessus. Le +1 permet d'avancer d'un caractère, c'est à dire de passer l'étoile.

Ça semble logique au final.

Par contre, est-ce que Desc peut contenir un caractère '*' ou pas ? Parce que si la réponse est non, je ne vois vraiment pas l'intérêt du code...

Quand tu vois le reste du code, tu ne te pose pas de question sur ça...

Et pour être tatillon : tu dis être en C90, mais dans ce cas les commentaires doivent être en /* */

Les commentaires étaient juste pour explication pour le forum. Tout les commentaire de l'application sont bien en /* */Cordialement,
Patrick Kolodziejczyk.

Edit :
%ld fonction en AIX 32/64 bits et sous Windows 32 bits.

Problématique résolu, merci !