Utilisation de l'adresse d'un objet temporaire

Bonjour,

j'ai une fonction comme ceci, qui est déjà dans la bibliothèque:

Code:

---

SDL_FillRect(SDL_Surface *surf, SDL_Rect *dest, Uint32 color);

---

Et donc, j'ai fait une fonction pour créer un rectangle:

Code:

---

1 2 3 4 5

SDL_Rect SDL_CreateRect(Sint16 x, Sint16 y, Uint16 w, Uint16 h) { SDL_Rect r = {x, y, w, h}; return r; }

---

Et je fais ça:

Code:

---

SDL_FillRect(surf, &SDL_CreateRect(0, 0, 100, 100), 0);

---

Le problème c'est que j'ai un warning:

Code:

---

C:\Users\Dave\Documents\CODING\projets\teambuilder\teambuilder.cpp:553: warning: taking address of temporary

---

Je ne sais pas si ça peut causer une erreur, dans l'implémentation que j'en ai faite, et sinon comment faire disparaître le warning?

Citation:

---

Envoyé par coyotte507

Bonjour,

j'ai une fonction comme ceci, qui est déjà dans la bibliothèque:

Code:

---

SDL_FillRect(SDL_Surface *surf, SDL_Rect *dest, Uint32 color);

---

Et donc, j'ai fait une fonction pour créer un rectangle:

Code:

---

1 2 3 4 5

SDL_Rect SDL_CreateRect(Sint16 x, Sint16 y, Uint16 w, Uint16 h) { SDL_Rect r = {x, y, w, h}; return r; }

---

Et je fais ça:

Code:

---

SDL_FillRect(surf, &SDL_CreateRect(0, 0, 100, 100), 0);

---

Le problème c'est que j'ai un warning:

Code:

---

C:\Users\Dave\Documents\CODING\projets\teambuilder\teambuilder.cpp:553: warning: taking address of temporary

---

Je ne sais pas si ça peut causer une erreur, dans l'implémentation que j'en ai faite, et sinon comment faire disparaître le warning?

---

Ce que tu fait est proscrit: tu créer une variable temporaire stockée sur la pile qui est détruite à la sortie de SDL_CreateRect() (parce qu'on dépile la pile), mais tu l'utilise quand même ensuite. Si SDL_FillRect() utiliser ne serait-ce qu'une variable temporaire initialisée, son addresse risque fort d'être la même que celle de ton rectangle.

Tu peux faire ceci:

Code:

---

1 2 3 4 5

void fill_rect(SDL_Surface *surf, Uint16 x, Uint16 y, Uint16 w, Uint16 h, Uint16 color) { SDL_Rect r = {x, y, w, h}; SDL_FillRect(surf, &r, color); }

---

Je te déconseille aussi de créer des fonctions extérieures à une librairie mais dont le nom utilise le même préfixe. Si dans 1 an tu relis ton code tu ne sauras plus (sans avoir à faire d'allers-retours dans le code) si telle ou telle fonction est une fonction SDL ou non.

Merci,

C'est ce que je voulais savoir :)

Citation:

---

Ce que tu fait est proscrit: tu créer une variable temporaire stockée sur la pile qui est détruite à la sortie de SDL_CreateRect() (parce qu'on dépile la pile), mais tu l'utilise quand même ensuite. Si SDL_FillRect() utiliser ne serait-ce qu'une variable temporaire initialisée, son addresse risque fort d'être la même que celle de ton rectangle.

---

Je dirais plutôt que le temporaire est détruit à la sortie de SDL_FillRect. Non ? (J'ai toujours du mal avec la durée de vie des temporaires).

Citation:

---

Envoyé par Montag

Je dirais plutôt que le temporaire est détruit à la sortie de SDL_FillRect. Non ? (J'ai toujours du mal avec la durée de vie des temporaires).

---

Non: le temporaire est détruit à la fin du bloc qui l'a créé (durée de vie dite "automatique"). Dans ce cas, la variable est créée dans le corps de SDL_CreateRect(), elle n'existe donc plus à la sortie de cette fonction.

Lors du :

Code:

---

return r;

---

Une copie de r est créée, c'est donc son adresse qui est utilisée (je crois d'ailleurs que ce n'est pas autorisé par le standard...). Je pense qu'ensuite cette copie est détruite à la sortie de SDL_FillRect.

Salut,

moi aussi ça m'intrigue,

j'enlève le résolu pour le moment.