SDL_Init et SDL_InitSubSystem

**shenron666** · 10/08/2006, 11h38

Bonjour à tous,

initialiser la SDL se fait normalement avec SDL_Init et des flags qui indiquent quelles parties activer

[mode bourrin ON]
j'ai essayé un SDL_Init(0) avec succès
suivi d'un SDL_InitSubSystem(SDL_INIT_VIDEO) avec succès également

j'ai supprimé le SDL_Init(0) sans que cela pose problème
[mode bourrin OFF]

je me demandais si ce principe était convenable et ne risquait pas de poser des problèmes sur d'autres systèmes (je suis sous windows)

en fait, je cherche à mettre la gestion de l'affichage (création de la fenêtre, changement de résolution, passe en fullscreen...) dans un singleton
le constructeur du singleton fait appel à SDL_InitSubSystem(SDL_INIT_VIDEO) et son destructeur fait appel à SDL_QuitSubSystem(SDL_INIT_VIDEO)

visiblement ça ne pose pas de problème à la SDL
dites moi ce que vous en pensez

ps: je précise qu'un appel à SDL_Quit est effectué lorsque le programme se termine, à voir également s'il est nécessaire ou inutile, pour l'instant j'ai préféré le laisser

ci joint, une partie du code du singleton CDisplay :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
#include "Display.h"
#include "sdl/sdl.h"
 
 
// static declaration of display manager's unique instance
CDisplay* CDisplay::m_pInstance = NULL;
 
 
// constructor & virtual destructor
CDisplay::CDisplay(void)
{
    // initialisation SDL video
    if ( SDL_InitSubSystem( SDL_INIT_VIDEO ) < 0 )
    {
        printf( "Echec SDL_Init : %s\n", SDL_GetError() );
        return; // throw exception
    }
 
	// display parameters
	m_sTemp.nScreenX = DISPLAY_SCREENX_DEFAULT;
	m_sTemp.nScreenY = DISPLAY_SCREENY_DEFAULT;
	m_sTemp.nScreenBpp = DISPLAY_SCREENBPP_DEFAULT;
	// fullscreen flag
	m_sTemp.bFullscreen = DISPLAY_FULLSCREEN_DEFAULT;
	// OpenGL parameters
	m_sTemp.nZBits = DISPLAY_ZBUFFER_DEFAULT;
	m_sTemp.nStencilBits = DISPLAY_STENCIL_DEFAULT;
	// mise à jour de l'affichage
	Update(m_sTemp);
}
 
CDisplay::~CDisplay(void)
{
	SDL_QuitSubSystem( SDL_INIT_VIDEO );
}
 
 
// static method to retrieve unique instance of manager
CDisplay* CDisplay::Initialize(void)
{
	if(!m_pInstance)
		m_pInstance = new CDisplay;
 
	return m_pInstance;
}
 
void CDisplay::ShutDown(void)
{
	delete m_pInstance;
	m_pInstance = NULL;
}

**fearyourself** · 10/08/2006, 12h33

Mettre le code dans un singleton ne me semble pas être une mauvaise idée. (Bien que le mettre dans le main ne m'a jamais posé de problème...)

Par contre, il est bien spécifié qu'il faut d'abord faire un appel à SDL_Init.

Il se peut que sans cet appel, tu vas avoir des problèmes sur différents systèmes. Pour le jouer safe, je dirais que tu dois assurer que cet appel a été fait et avant le code du constructeur de ton singleton.

Jc

**fearyourself** · 10/08/2006, 12h43

Pour la peine, j'ai regardé le code de la SDL 1.2.11, voici le code de SDL_Init :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
int SDL_Init(Uint32 flags)
{
#if !SDL_THREADS_DISABLED && SDL_THREAD_PTH
        if (!pth_init()) {
                return -1;
        }
#endif
 
        /* Clear the error message */
        SDL_ClearError();
 
        /* Initialize the desired subsystems */
        if ( SDL_InitSubSystem(flags) < 0 ) {
                return(-1);
        }
 
        /* Everything is initialized */
        if ( !(flags & SDL_INIT_NOPARACHUTE) ) {
                SDL_InstallParachute();
        }
        return(0);
}

Comme tu vois, il y a tout de même une initialisation de la gestion des threads qui sera fait (si c'est voulu au moment de la compilation) et une remise à zéro des messages d'erreur...

Ensuite, on appelle effectivement SDL_InitSubSystem...

Du coup, si tu voulais faire ton singleton, cela dépend si tu utilises des threads ou non et il faudrait installer le système de parachute SDL...

Ceci confirme mes craintes -> pour avoir un code portable, tu vas devoir t'assurer qu'il y a un appel à SDL_Init avant de faire quoi que ce soit...

Enfin, ceci confirme que SDL_Init(0) fonctionne, au final, aucun SubSystem est mis en place...

Jc

**shenron666** · 10/08/2006, 14h06

Ok, donc je met un SDL_Init(0) au début de mon main et ensuite j'initialise le(s) sous-système(s) dont j'ai besoin

Envoyé par fearyourself

Pour le jouer safe, je dirais que tu dois assurer que cet appel a été fait et avant le code du constructeur de ton singleton.

c'est la raison pour laquelle j'ai modifié mon singleton
plutot que de déclarer :

Code :

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

1
2
3
4
5
CDisplay& CDisplay::Instance(void)
{
   static CDisplay instance;
   return instance;
}

j'ai préféré faire :

Code :

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

1
2
3
4
5
6
CDisplay* CDisplay::Instance(void)
{
   if(!m_pInstance)
      m_pInstance = new CDisplay;
   return m_pInstance;
}

vu qu'il faut que j'appelle le destructeur avant SDL_Quit
le destructeur faisant appel à SDL_QuitSubSystem

merci de m'avoir éclairé là dessus Jc

**fearyourself** · 10/08/2006, 14h33

Envoyé par shenron666

Ok, donc je met un SDL_Init(0) au début de mon main et ensuite j'initialise le(s) sous-système(s) dont j'ai besoin

c'est la raison pour laquelle j'ai modifié mon singleton
plutot que de déclarer :

Code :

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

1
2
3
4
5
CDisplay& CDisplay::Instance(void)
{
   static CDisplay instance;
   return instance;
}

j'ai préféré faire :

Code :

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

1
2
3
4
5
6
CDisplay* CDisplay::Instance(void)
{
   if(!m_pInstance)
      m_pInstance = new CDisplay;
   return m_pInstance;
}

vu qu'il faut que j'appelle le destructeur avant SDL_Quit
le destructeur faisant appel à SDL_QuitSubSystem

Ce n'est pas obligatoire :

- SDL_Quit va tout arrêter comme un grand

Par contre, si tu voulais vraiment faire proprement, il faut effectivement que SDL_Quit soit la dernière fonction SDL appelée.

- Si jamais il te vient à l'esprit de vouloir utiliser SDL_WasInit, ne le fait pas ! Cette fonction suppose qu'au moins un subsystem a été mis en place. Appeler SDL_Init(0) lui ferait penser que rien n'a été fait...

Jc

**shenron666** · 10/08/2006, 15h23

Envoyé par fearyourself

Ce n'est pas obligatoire :

- SDL_Quit va tout arrêter comme un grand

Par contre, si tu voulais vraiment faire proprement, il faut effectivement que SDL_Quit soit la dernière fonction SDL appelée.

Je vais peut-etre revoir ma copie mais comme j'ai mis SDL_InitSubSystem(SDL_INIT_VIDEO) dans le constructeur de CDisplay, ma logique veut que je mette SDL_QuitSubSystem dans le destructeur

Envoyé par fearyourself

- Si jamais il te vient à l'esprit de vouloir utiliser SDL_WasInit, ne le fait pas ! Cette fonction suppose qu'au moins un subsystem a été mis en place. Appeler SDL_Init(0) lui ferait penser que rien n'a été fait...

OK, c'est noté
je n'avais pas l'intention de l'utiliser et là du coup... encore moins

encore merci pour tes conseils

**fearyourself** · 10/08/2006, 20h33

J'ai réfléchi au problème et je pense avoir trouvé une solution élégante à ton problème.

On sait qu'à la construction, le subsystem est en place parce que sinon la fonction SDL_InitSubSystem aurait rendu un nombre négatif...

A ce moment là, lors de la destruction, tu pourrais faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
CDisplay::~CDisplay(void)
{
	SDL_QuitSubSystem( SDL_INIT_VIDEO );
        //S'il ne reste plus rien
        if(!SDL_WasInit(SDL_INIT_EVERYTHING)
            {
             //On peut fermer SDL, on sait qu'aucun autre singleton le fera 
             //(sauf cas multithreadé)
             SDL_Quit();
            }
}

Histoire de pousser le bouchon plus loin, on peut aussi augmenter le constructeur pour qu'en interne il fasse l'initialisation :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
 
// constructor & virtual destructor
CDisplay::CDisplay(void)
{
//Cette appel retourne un nombre non nul si un subsystem est initialisé
 if(!SDL_WasInit(SDL_INIT_EVERYTHING))
            {
             //On initialise SDL, on sait que ceci sera appele une fois vu qu'on verifie
             //Si ce singleton s'est bien initialise
             //(sauf cas multithreadé)
             if(SDL_Init(0)!=0) 
                  {
                    printf( "Echec SDL_Init : %s\n", SDL_GetError() );
                    return; // throw exception
                  }
            }
 
    // initialisation SDL video
    if ( SDL_InitSubSystem( SDL_INIT_VIDEO ) < 0 )
    {
        printf( "Echec SDL_Init : %s\n", SDL_GetError() );
        return; // throw exception
    }
 
 	// display parameters
	m_sTemp.nScreenX = DISPLAY_SCREENX_DEFAULT;
	m_sTemp.nScreenY = DISPLAY_SCREENY_DEFAULT;
	m_sTemp.nScreenBpp = DISPLAY_SCREENBPP_DEFAULT;
	// fullscreen flag
	m_sTemp.bFullscreen = DISPLAY_FULLSCREEN_DEFAULT;
	// OpenGL parameters
	m_sTemp.nZBits = DISPLAY_ZBUFFER_DEFAULT;
	m_sTemp.nStencilBits = DISPLAY_STENCIL_DEFAULT;
	// mise à jour de l'affichage
	Update(m_sTemp);
}

Je l'ai pas testé mais cela devrait fonctionner...
Jc

**shenron666** · 17/08/2006, 12h45

Merci pour ta réponse, elle est intéressante et a ses avantages tout comme ses inconvénients de la même façon que la solution précédente

au final j'en suis resté au même point pour ne pas me casser la tête des jours et des jours uniquement sur ça

j'avais repris une partie de ta proposition pour le destructeur mais ce qui reste peut-etre préférable c'est initialiser la SDL en dehors du constructeur

finalement je vais tester la SDL avec SDL_WasInit(SDL_INIT_VIDEO) et générer une exception le cas échéant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
CDisplay::CDisplay(void)
{
	// initialisation SDL video
	if (!SDL_WasInit( SDL_INIT_VIDEO ))
		throw "SDL_VIDEO not initialized"; // throw exception
 
	// display parameters
	m_sTemp.nScreenX = DISPLAY_SCREENX_DEFAULT;
	m_sTemp.nScreenY = DISPLAY_SCREENY_DEFAULT;
	m_sTemp.nScreenBpp = DISPLAY_SCREENBPP_DEFAULT;
	// fullscreen flag
	m_sTemp.bFullscreen = DISPLAY_FULLSCREEN_DEFAULT;
	// OpenGL parameters
	m_sTemp.nZBits = DISPLAY_ZBUFFER_DEFAULT;
	m_sTemp.nStencilBits = DISPLAY_STENCIL_DEFAULT;
	// mise à jour de l'affichage
	Update(m_sTemp);
}

et bien sûr dans ce cas, le destructeur ne fait rien

**fearyourself** · 17/08/2006, 13h17

D'accord et ceci t'oblige donc de créer ton singleton lors de l'exécution avec une allocation dynamique puisque tu dois attendre que la SDL soit initialisée, non ?

Jc

**shenron666** · 17/08/2006, 16h47

c'est exact,

d'un côté j'avais préféré un singleton avec allocation dynamique puisqu'on a plus de controle sur la création et la destruction de la fenêtre

mais si en plus la SDL doit être initialisée AVANT le singleton, pas d'alternative

je met un zip contenant le code source du singleton CDisplay et un main
ceux que cela intéresse peuvent l'utiliser mais surtout si vous avez des commentaires intéressants sur la façon dont c'est programmé n'hésitez pas

SDL_Init et SDL_InitSubSystem

SDL

Discussions similaires

Partager

Partager