Discussion :

[bubkadk]

Bonjour a tous,

Cela fait maintenant 2 jours que je suis embêté pur fermer une fenetre SDL 1.3.

j'utilise la fonction winId=SDL_CreateWindowfrom(widgetId) pour lier SDL a un Widget Qt.

j'arrive ensuite a dessiner dans ma fenêtre SDL, mais quand j'utilise les instructions SDL_DestroyWindow(winId) et SDL_Quit(), je me retrouve avec un segfault.

voici mon code(trés basique)

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
 
MainWindow::MainWindow(QWidget *parent) :
        QMainWindow(parent){
    setupUi(this);
 
    if (SDL_Init(SDL_INIT_VIDEO) != 0) {
        printf("Error initializing SDL:  %s\n", SDL_GetError());
    }
 
 
    qlonglong window_id=reinterpret_cast<qlonglong>(widget->winId());
 
    fenetre=SDL_CreateWindowFrom((void*)window_id);
    if (fenetre == 0) {
        qDebug()<<SDL_GetError();
    }
 
}

Code :

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

1
2
3
4
5
MainWindow::~MainWindow()
{
    SDL_DestroyWindow(fenetre);
    SDL_Quit();
}

Voila rien que le code ci dessus fait planter l'appli sur fermeture de celle ci.

J'espère que quelqu'un pourra m'aider à trouver une solution.

Cyrille

[LittleWhite]

Bonjour,

Je pense que le problème est assez simple .

En fait, Qt va créer une fenêtre (window_id comme identifiant). Celle-ci sera utilisé par la SDL. Jusque là, rien de grave.
Sauf que lors de la destruction, votre widget demande à la SDL de détruire ce window_id, mais Qt, qui gère sa mémoire tout seul, va détruire aussi la fenêtre (il ne sait pas que celle-ci est utilisé par la SDL / et qu'elle est déjà détruite).

Donc, moi j'enlèverai le SDL_DestroyWindow.

Après, je ne peux pas confirmer, car je n'ai jamais vu de code SDL 1.3 et puis car la documentation ne parle pas de cela. Mais cela peut être une piste à suivre.

[bubkadk]

Je me doutais de quelque chose comme cela,
en fait j'ai ecrit a l amailing list de libsdl,

ils m'ont donné un patch qui corrige le probleme.

si cela interresse quelqu'un, voici le fix

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
 
# HG changeset patch
# User Ryan C. Gordon <icculus@icculus.org>
# Date 1311631422 25200
# Node ID 5ae512481858a6dd287e7bb6787447a9db447f86
# Parent  c8862e1bc41f3be30c7c67002b5a51d12e0eccb2
Windows: restore event procedure on destroy if we didn't create the window.
 
diff --git a/src/video/windows/SDL_windowswindow.c b/src/video/windows/SDL_windowswindow.c
--- a/src/video/windows/SDL_windowswindow.c
+++ b/src/video/windows/SDL_windowswindow.c
@@ -636,6 +636,15 @@
         ReleaseDC(data->hwnd, data->hdc);
         if (data->created) {
             DestroyWindow(data->hwnd);
+        } else {
+            /* Restore any original event handler... */
+            if (data->wndproc != NULL) {
+#ifdef GWLP_WNDPROC
+                SetWindowLongPtr(hwnd, GWLP_WNDPROC, (LONG_PTR) data->wndproc);
+#else
+                SetWindowLong(hwnd, GWL_WNDPROC, (LONG_PTR) data->wndproc);
+#endif
+            }
         }
         SDL_free(data);
     }