Effet de bord sur texture atlas.

[Cpl.Bator]

J'avoue ne pas te suivre.

- je récupère des pixels dans un tableau temporaire
- je les update dans une texture ( qui as une image )
- je passe à la prochaine itération et je recommence

en tout cas, pas de soucis à ce niveau là. pour moi il n'y a pas de problème.

[gostron]

Au moment de l'appel de cette ligne

const sf::Uint8 * pixels = t->getTexture().copyToImage().getPixelsPtr();

Il se passe ceci:
on appelle

getTexture()

qui te renvoie une référence vers la texture courante de t.

Puis on appelle

copyToImage()

qui va créer une image au moment de cet appel (qui est donc une fonction lente). L'usage normal de cette fonction est la construction d'une véritable image comme ceci

sf::Image myImage(t->getTexture().copyToImage());

Finalement, tu récupères le pointeur de l'image que tu viens de créer, et c'est celui ci que tu stockes. A aucun moment tu ne récupères réellement l'image.

A chaque appel de copyToImage() tu récrées une image à partir de la texture actuelle, avec à chaque fois une adresse différente.

[Laurent]

Demande-toi qui possède (crée / détruit) le tableau de pixels vers lequel tu pointes. Quelle instance de quelle classe. Et ensuite demande-toi si l'instance en question existe encore dans ton code au moment où tu utilises le pointeur.

[Cpl.Bator]

@gostron , je ne veut pas créer de véritable image.
j'ai compris que *pixels est temporaire a la fonction , je m'en sers uniquement pour récupéré les pixels que je colle dans une texture non temporaire a la fonction ( la texture atlas )
lorsque je passe *pixel a update , opengl m'en fait une copie avec glTexSubImage2D.
je ne veut pas gardé les portions que je découpe , j'en ai pas besoin par la suite.

[Laurent]

Il comprend pas...

Avant l'appel à copyToImage(), le tableau de pixels n'existe pas. Après l'appel à copyToImage(), le tableau de pixels n'existe plus. Donc quand tu arrives dans ton appel à texture.update, ça fait belle lurette (une ligne de code tout entière !) que les pixels vers lesquels tu pointes ont été détruits.

C'est comme de faire ça :

const char* ptr = std::string("blop").c_str();

C'est tout aussi incorrect, parce que le tableau de caractères est stocké dans le std::string, mais celui-ci n'est stocké nulle part et est donc détruit dès la fin de l'exécution de cette ligne de code.

[Cpl.Bator]

d'accord , pourquoi ça marche ? il faut donc que je passe par une image , et pas un raccourci comme je le fait donc.

[Laurent]

"Ca marche" est une manifestation tout à fait valide d'un comportement indéterminé. J'ai envie de dire que c'est la pire, puisque ça te cache complètement l'erreur. Avec un bon vieux crash, au moins, tu as le nez dans ton erreur directement.

Si tu veux des détails un peu plus techniques : tant que personne n'est allé réallouer et écrire par dessus la zone mémoire où se trouvaient les pixels, ceux-ci s'y trouvent toujours.

[Cpl.Bator]

ok , donc , si je m'amuse à modifier *pixel après mon update , je devrais voir le résultat.
Merci de ta patience