Discussion :

[coyotte507]

Bonjour,

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

Code :

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

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

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

Code :

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

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 :

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

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

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

Code :

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

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?

[Emmanuel Deloget]

Bonjour,

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

Code :

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

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

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

Code :

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

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 :

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

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

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

Code :

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

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 :

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

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.

[coyotte507]

Merci,

C'est ce que je voulais savoir

[Montag]

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).

[Emmanuel Deloget]

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.

[Montag]

Lors du :

Code :

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

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.

[coyotte507]

Salut,

moi aussi ça m'intrigue,

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